e_start_main: refactor (reduce scope depth, separate ifdefs, cleanup)

The main reason for this change is to clean it up for proper non-Linux
platform support, but also to make it cleaner in general.
remove unused variable
fix indentation
e-start: Fix 'control reaches end of non-void function'

Summary: Seems one of the Daniels did not check their compiler output
;) _e_ptrace_attach function should return an integer value...

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>
e_start_main: Close fd only if we actually succeeded opening it.

In the error case fd would be -1 here. Closing needs to be handled in the
cases where fd != -1.

CID: 1256492
ibar: Fix "additional action"s and seperator placement.

Summary: This moves the additional actions to the top of the menu and the seperator right underneath.

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1694
backlight: visual feedback when adjusting value through keybindings

The backlight module has an action to show the value of the backlight brightness
by showing the gadget's popup. Added a parameter to this action to adjust the
brightness at the same time.
backlight: remove code duplication

Move repeated blocks of code to a static function
AUTHORS: Add Luis to authors file.

Thanks for your contributions, let's hope they are the first of many. :)
AUTHORS: Add Marcel to authors file.
print more descriptive error messages when scanner connection fails, also don't exit() here

ref T1612
don't exit e_fm when eeze init fails, send RASTER mode instead
store eeze_scanner connection poller to avoid creating an infinite number

ref T1612
only attempt to start eeze_scanner a couple times before giving up

ref T1612
attempt to run eeze_scanner on failure to connect, not successful connect

ref T1612
everything: terminate bc instead of quit

On some systemd SIGQUIT causes the logger to save a coredump.
Quiting bc here does not need save a backtrace here, the termination is
not a crash its a "closing" of the program.
ensure clients are visible before triggering shape/render queue
fix X crash when trying to apply desktop window profile to client without desk
add failure count for client rendering, drop clients after FAILURE_MAX tries

this traps clients which are showing and hiding themselves extremely quickly and prevents them from crashing us when we try to draw them

fix T1820
redo shaped client checks to use shape flags instead of rect pointer, fix shapeless clients

according to the shape extension protocol, the number of rectangles returned should be checked to determine a client's shape. if 0 is returned, the client has no shape, meaning that it either should not be drawn or should have no input region. this improves behavior with various client window types such as tooltips

ref T1820