evas gl rgba visual/rendering fix try ... again

so this is a re-try at the evas gl destination alpha fix. this is what
cedric tried, but done RIGHT. it required adding an ecore_x call to
create a window with correct visual/colormap. it requires doing
visuals totally correctly all the way from ecore_evas to the evas
gl_x11 core. nvidia drivers are very picky about visuals. i also had
to vid the egl/gles code too to do the same thing. nvidia gles/egl
drivers are also picky, mesa is not. this all requires a lot of code
changes. it's far from trivial

this isn't backported for a few reasons:

1. verify this fix doesn't break for anyone.

i tested:

nvidia glx + egl/gles
intel glx + egl/gles
radeon glx

it needs wider testing. nouveau, fglrx for starters and maybe
some other gles/egl drivers.

2. have some review time
3. time to settle before blasting to stable branches

@fix

ecore_exe_win32: CreateProcess was called with random flags.

CreateProcess() has a flags parameter which is being passed
"run_pri | CREATE_SUSPENDED".

The issue lies in the value of run_pri. It is best explained by the
following code somewhere else in the file:

   switch (run_pri)
     {
      case IDLE_PRIORITY_CLASS:
        return ECORE_EXE_WIN32_PRIORITY_IDLE;

The run_pri variable is supposed to store a value from the win32 API while
it was used to store one from the ecore API.

If I recall correctly, the windows one is equal to 32 and the ecore one to
9999. Meaning 9999 ended up used as flags so let's have a look at what that
actually enabled; the reference is "Process Creation Flags" from MSDN
http://msdn.microsoft.com/en-us/library/ms684863%28v=vs.85%29.aspx .

9999 gives 0x0000270F and this matches
  DEBUG_PROCESS | DETACHED_PROCESS | DEBUG_ONLY_THIS_PROCESS
  | CREATE_SUSPENDED | CREATE_NEW_PROCESS_GROUP | CREATE_SEPARATE_WOW_VDM
  | CREATE_UNICODE_ENVIRONMENT | <0x00002000 matches nothing>

Matches nothing? Weird. Well, maybe. Except that I stumbled upon this define
in the mingw-w64 headers:
  #define CREATE_FORCEDOS 0x2000

Mingw-w64 only has a #define, Wine has nothing (they don't do DOS anyway),
but ReactOS has some code about it:
  https://git.reactos.org/?p=reactos.git;a=blob;f=reactos/dll/win32/kernel32/client/proc.c;hb=f60941f8dc775427af04eb0a3c3e4d38160c7641#l3007

Overall the actual set of flags probably made very little sense and wasn't
working very well. :)

I also noticed the following in the mingw-w64 headers:
  #define INHERIT_CALLER_PRIORITY 0x20000

This should be a better match for what seemed to be the original intent of
inheriting the priority. I haven't tested it and it's only documented on
MSDN for Windows CE and similar so I'm really not sure about what it does.

MSDN however mentions that the child processes will have at most the
"normal" priority by default (same as its parent if the parent has less
than the default one) but I'm under the impression a process can raise its
own priority level... Anyway, "NORMAL_PRIORITY_CLASS" will do for now.

With this change and a couple others, elementary's theme builds properly
on Windows (_on_ Windows). I'll assess the usefulness of the other changes
in my tree over the next few days.

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_evas: Add since tag to new ecore_evas_gl_drm_new() function

ecore-wayland: Add since tag to new ecore_wl_window_xdg_surface_get() function

evas-gl-drm: Fix initialization from incompatible pointer type

Summary: The function prototype for eng_context_create has recently
changed in gl_common, however nobody thought it wise to update all
engines using it, so this commit fixes the function for the gl_drm
engine.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

eina_cxx: Include newly added header to unbreak make distcheck

[eina-cxx] Fixed EFL_CXX_THROW not defined

Fixed errors of EFL_CXX_THROW not defined when #include'ing specific
eina C++ headers.

[cxx] Fixed eina::value get for string

The get function for string in eina::value wronfully was freeing the
string returned from eina_value_get. Which crashed applications that
used it.

ecore_wayland: Making _ecore_wl_shutdown return int instead of EINA_BOOL.

Summary:
_ecore_wl_shutdown should return int instead of EINA_BOOL. So changing the function prototype.

@fix

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1574

ecore-wayland: Add error message for calling shutdown without calling
init

Summary: If someone calls ecore_wayland_shutdown without first calling
ecore_wl_init, then the init count is wrong. Warn the caller.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm: Add error message for calling shutdown without calling init
first.

Summary: If someone calls ecore_drm_shutdown without first calling
ecore_drm_init, then the init count is wrong. Warn the caller.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Revert "tests/eina: Enable eina_condition_timedwait() now that it should work."

This has to work until we really fixed up eina_condition_timedwait()

This reverts commit 5c9024424f7102abd6c7d858628b52da6065cf9a.

Revert "eina: Fix _timedwait to handle the given timeout on top of the absolute time"

Not ready yet as it uses _eina_time_get which is internal only right now. Compiling
works fine for efl alone as the private header is in teh include search part but it
blows up when compiling elementary.

Need to think a bit more about this. Maybe exposing _eina_time_get as API but that
should wait until after the release.

This reverts commit f0a02a92beff49879eba84b4d4eee265fe156511.

ecore_drm: Making _ecore_drm_init_count not to go below zero in _ecore_drm_shutdown.

Summary:
If _ecore_drm_init_count goes below zero, then when next time ecore_drm_init is called, it won't do the initializations which it is supposed to do. So preventing this scenario by not making it go
below zero in _ecore_wl_shutdown function.

@fix

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1573

ecore_wayland: Making _ecore_wl_init_count not to go below zero in shutdown.

Summary:
If _ecore_wl_init_count goes below zero, then there would be problem if someone calls ecore_wl_shutdown 1st and then ecore_wl_init later. So fixing this issue in ecore_wl_shutdown.

@fix

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1571

ecore-drm: Added documentation for various APIs in ecore_drm_output module

Summary:
Added documentation for ecore_drm_outputs_create, ecore_drm_output_free,
ecore_drm_output_cursor_size_set, ecore_drm_output_size_get functions

@fix

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1572

evas/cserve2_slave: Fix build race with linking to eo but not depending on it.

I have not been able to reproduce this myself but I have seen a build log where
the binary tries to link to libeo and fails due to the missing file.
A similar problem was "fixed" in 0e4b847deb79c957dbc042ea0c6a98282aead6e0, but
this really makes me wonder where the linking against eo comes from for cserve2
which is not using eo as far as I can see.

Eina Test : Added test case for convert float in eina_test_value

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric, devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1567

ecore-drm: Added documentation for various APIs in ecore_drm_device module

Summary:
Added documentation for ecore_drm_device_fd_get, ecore_drm_device_window_set
and ecore_drm_device_name_get APIs in ecore_drm_device module.

@fix

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1568

ecore-drm: Fix formatting

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm: Cleanup backlight and drm device in output_free function code

Summary:
Added code to cleanup backlight structure and to close the drm device and
delete it.

@fix

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1566

eina: Add a note about potential use of CLOCK_MONOTONIC

Note it down so we can think about this at some later point.

tests/eina: Enable eina_condition_timedwait() now that it should work.

eina: Fix _timedwait to handle the given timeout on top of the absolute time

The pthread man page clearly states that pthread_cond_timedwait() takes an
absolute time parameter. So far we always passed it epoch plus timeout in
seconds. This would never trigger the timeout.

Making sure we fill out timespec struct with the current time before adding
the timeout as offset now. Also handling the t < 0 error case.

Various version worked up together with Jean-Philippe Andre <jp.andre@samsung.com>

Fixes T1701

eet: shutup stupid compiler warning.

evas-gl-drm: Cleanup formatting

NB: No functional changes

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm: Check for valid crtc before calling free

Summary: We cannot call drmModeFreeCrtc with an invalid crtc, so check
that it is set inside the output structure before trying to make this
call

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm: Cleanup/Unify output_free function code

Summary: There is no point in having different code in each output
free function (internal one and API exposed one), so let's unify the
code here.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

emotion: Add wma as possible playable extension

This allows to preview (listen) a wma file in efm for example.

Fixes T1568

evas: tests only for extension compiled in.

ecore_drm: Added internal function to check if drm device is hotplug device

Summary:
Added _ecore_drm_output_device_is_hotplug API to check if the
drm device is hotplug device. It returns EINA_TRUE if device is
hotplug else returns EINA_FALSE

@feature

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1559

Eo id: Add regression tests to eo id sanity checks

Relevant commit: 0b86334a858d9ba88a1642ec5671185292e75ce5

Eo id: Fix id security checks for invalid objects.

In some cases, invalid object ids (e.g 0x1) would pass validation and
represent completely different objects (0x80...01). This happened because
we weren't properly checking a given object id is actually an object id.

@fix.

Eina: Check for errors after eina_tls_new

Fix CID 1248913.

Evas filters: Remove useless source_count and cow checks

Evas filters: Feed some cow to the image objects

Same as for the text objects.
Fixes T1725.

Cedric, please run again your memory test, please.

Evas filters: Move filter COW to evas global stuff

It will be shared between TEXT and IMAGE objects.

Evas filters: Use a cow to reduce memory usage of text objects

As pointed out by Cedric, the memory usage of basic evas objects
has increased a lot in recent versions of EFL, in part due
to this excessive use of filters data.

This is a partial fix for ticket 1725.

Evas GL: Don't pass shared context for GLX with 1.1 API

Not sure if this is very relevant, since GLX does not support
GL-ES as such, anyways... We should be using the extension
GLX_EXT_create_context_es_profile to create proper contexts.

Note: GLX + OpenGL-ES 1.1 crashes at any function call on my
machine (binary bloc driver), while EGL + GLES1.1 is fine.

Evas GL: Remove useless #ifdef/#endif

Evas GL: Fix unused variable

CID 1248912

eldbus: Fix wrong comments.

evas: let's handle dds also during tests properly.

evas: tests more file format loader.

NOTE: It seems that the xpm one is broken. If somebody could give it
a better look and check if it work for him or not, before opening a bug
in phab.

evas: don't redefine macro in tgv saver.

release: Update NEWS and bump version for 1.12.0-alpha1 release

tests/eolian: Move eo to EXTRA_DIST assignment out of the EFL_ENABLE_TESTS case

When running distcheck with release mode enabled these would not end up in the
tarball. Better make sure these always end up at the right place.

evas-wayland-egl: Fix function prototype for eng_context_create

Summary: Apparently, somewhere down the line, the function prototype
for eng_context_create was changed in the EVGL_Interface, but nobody
decided to fix All engines. This commit fixes the function prototype
for evgl_eng_context_create

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas: Evas_Box - layout_flow_horizontal and layout_flow_vertical fix

Summary:
Now minimum width hint (for horizontal layout) and minimum height hint (for vertical)
are set to largest item size rather than largest row/column.

@fix

Test Plan:
1. Run "elementary_test"
2. Maximize window
3. Unmaximize it
ExR: button boxes should return to start state.

Reviewers: cedric, seoz, Hermet

Subscribers: cedric, reutskiy.v.v

Differential Revision: https://phab.enlightenment.org/D1511

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_wayland: Added test case to check ecore_wl_init by passing socket name.

Summary:
Added a test case for ecore_wayland to test ecore_wl_init by passing the socket name. Wayland display is created and a socket is added to the display, then this socket is passed to ecore_wl_init to connect. It should successfully connect. Then
ecore_wl_shutdown is called to verify if it closes.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1560

ecore-drm: Create udev watch for each output, and delete the watch
when we free the output.

Summary: This commit fixes an issue where a hotplug watch was Only
being added for One input, and also deletes any created watch during
output_free.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm: Default output to NULL and add safety check for NULL output
after the device loop.

Summary: If there are no outputs created during the loop, we would end
up crashing here by checking output->watch. This fixes that situation
without leaking resources because we still end up doing our cleanup.

NB: Thanks Cedric for the report !! :)

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Eo composite: Fix composite object functions to be eo functions.

For some reason, they were normal functions instead of eo functions,
which makes them harder to bind, less safe, and just wrong.
This commit fixes that.

tests/evas: Add missing files to EXTRA_DIST and remove duplicates

Various files form the tgv and 3d tests have been missing in EXTRA_DIST
and thus failing make distcheck. On the other hand we had duplicates in
the list. Clear this all up.

Evas-3D Tests: fix Eo usage.

There are still Valgrind (invalid read) issues but at least, it doesn't
segfault more.

Revert "evas: GL_X11 context need to always be with alpha or it will fail to change."

Need to also revert this in master. Cedric seems busy with other things and this
should not end up in 1.12  alpha1

This reverts commit 4ea75113caeeec6bbc047f2406f8d1c89870360b.

Eet: Fix eet jpeg image load

Commit 101dee79b0d75fd15 introduced critical errors, as standard
error callbacks now fail.

Apparently, these error callbacks depend on the installed version
of libjpeg (8 vs. 9).

I couldn't start E without this commit.

Evas: Fix some debug output formatting

Remove trailing newline.

Evas-3D: invoke eo_do_super in destructors.

3D Warnings--

[example][evas] add error message, you cannot use this example without the image.

[ecore_input] fix build break - the semicolon

eet: enhance error messages with libjpeg

@fix

evil: simplify implementation of localtime_r

@fix

evil: formatting.

Merge branch 'devs/captainigloo/doc'

Improve documentation accross all EFL libraries. Work was done on
behalf of Samsung by Cadoles, Openwide and Zefla.

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eet: add some more documentation.

eldbus: update missing eldbus_proxy API documentation.

eldbus: update missing eldbus_pending.h API documentation.

eldbus: update missing eldbus_object API documentation.

eina document eina mempools' API.

Except for the weird _Eina_Mempool_Backend_ABI1/ABI2 work-around which is
completely internal and probably is of interest to noone (the only person
on Earth who will be interested in it in a decade will hate me for skipping
it).

eina: remove @addtogroup in eina_inline_mempool.x: the .h already has it.

It looks like doxygen doesn't appreciate having a group inside a group of
the same name and simply ignores any documentation coming from the parent
one.

Since the .x is included from the .h, it is always already inside the
group so simply remove the directive from the .x.

This fixes the documentation for all the functions which are declared and
documented in the .h and implemented in the .x.

eldbus: update missing eldbus_message API documentation.

eldbus: update missing eldbus_freedesktop.h API documentation.

eldbus: update missing API documentation.

eldbus: add missing API documentation in Eldbus.h.

ecore_ipc: update missing API documentation.

emotion: updating API documentation.

ethumb: update API documentation.

 - ethumb_generate
 - ethumb_exists
 - ethumb_dup
 - ethumb_cmp

etumb: updating API documentation.

 - ethumb_thumb_compress_set
 - ethumb_thumb_compress_get
 - ethumb_video_start_set
 - ethumb_video_start_get
 - ethumb_video_time_set
 - ethumb_video_time_get
 - ethumb_video_interval_set
 - ethumb_video_interval_get
 - ethumb_document_page_set
 - ethumb_document_page_get

ethumb: update API documentation.

 - ethumb_frame_set
 - ethumb_frame_get
 - ethumb_thumb_dir_path_set
 - ethumb_thumb_dir_path_get
 - ethumb_thumb_category_set
 - ethumb_thumb_category_get

eio: add doc for returns of eio_file_{mkdir,move,copy}, eio_dir_{move,copy}.

ethumb: document most functions from ethumb.

ecore_audio: add @brief doc for struct Ecore_Audio_Vio.

It's simply a copy of the one given in the following typedef:
  typedef struct _Ecore_Audio_Vio Ecore_Audio_Vio;

evas: add documentation about EVAS_MAJOR/MINOR_VERSION.

eio: documentation for return values.

 - eio_file_copy
 - eio_file_move
 - eio_dir_copy
 - eio_dir_move
 - eio_dir_unlink

evas: documentation on missing apis

edje: document part of edje_container.h done before seeing all of edje_container.c is #if 0’d.

edje: documentation for edje_convert.h.

edje_convert: updating function documentation.

edje: update typedef documentation.

edje: update typedef and callbacks documentation.

evas: add documentation for enums in Evas_Gl.h.

evas: more documentation of Evas_Common. Fix inline comments

eet: update callback API documentation

Adding some documentation on callbacks typedefs for :
- Eet_Node_Walk

eet: update callback API documentation.

Adding some documentation on callbacks typedefs for :
- Eet_Descriptor
- Eet_Dump

ecore_input: update missing Ecore_Input.h API documentation.

ecore: update Ecore documentation main page.

Add missing groups and remove groups that are already in subgroups in
order to have a nicer Ecore main page in the documentation.

ecore: update the doxygen groups to compile forgotten parts of the API.

Some parts of the API documentation where not compiled at all by doxygen
because of missing '@{' and '@}' tags. This commit adds the missing tags
in Ecore_Getopt.h, Ecore_Con_Eet.h, and Ecore_IMF.h headers.

ecore_file: remove ecore_file_ls_iterator function from header.

It has no implementation in the ecore_file.c file.
This has been forgotten in commit d4059c7 when moving this function to
Eina library.