Evas GL common: whitespace fixes

Evas masking: Implement line masking in GL

This commit also introduces a few whitespace changes...

Evas GL generic: Simplify "scaled" images (used for masking)

Invert the meaning of scaled (w,h), so that im->(w,h) corresponds
to the final scaled size, and the original size is stored directly
in the texture itself.

This simplifies code a little bit.

Also, lift the limitation on the maximum texture size, as those
virtual textures are not limited by GPU texture size.

Evas masking: Refactor GL code

Use the same method as for map masking:
- Pass absolute geometry of the mask to the push functions,
- Compute absolute position in the shader (like gl_Position)
  and apply scaling factor + offset to sample the mask.

Masking now uses a single vec4 attribute instead of hi-jacking
other vertices.

This way, all masking shaders share the same code and are way
simpler. On the other hand, the vertex shaders have a little bit
more work to do.

ecore_file - fix nasty memory issues in ecore_file_app_exe_get()

valgrind was most unhappy with ecore_file_app_exe_get(). like:

==8331== Invalid write of size 1
==8331==    at 0x68DE90A: ecore_file_app_exe_get (ecore_file.c:994)
==8331==  Address 0x1348e58f is 0 bytes after a block of size 31 alloc'd
==8331==    at 0x4C28C20: malloc (vg_replace_malloc.c:296)
==8331== Invalid write of size 1
==8331==    at 0x68DE948: ecore_file_app_exe_get (ecore_file.c:1000)
==8331==  Address 0x1348e599 is 10 bytes after a block of size 31 alloc'd
==8331==    at 0x4C28C20: malloc (vg_replace_malloc.c:296)

etc. etc. - so i rewrote it cleanly using strbuf to save code and
effort. cleaner now and ACTUALLY works correctly... and no valgrind
complaints.

@fix

edje object swallowing no longer forces a full recalc

this (and/or related prior commits) seems to have hugely improved
edje performance across the board. particularly noticeable in the
settings pane animations for terminology

 #aprilfools

edje object scale factor changes now trigger deferred recalcs

as with the previous commit, any user who wants geometry immediately
will be calling calc functions, so this is just going to be redundant
except in cases where calc is not wanted, in which case it is unnecessary
overhead

edje group part init no longer forces synchronous hint updating during load

hints will be updated later and any user who wants these hints is going to
be calling edje calc functions anyway, causing this to be a redundant calc

edje calc no longer forces proxy part source calcs for currently-hidden proxies

this saves a bit of calc time when a proxy part is not currently visible

edje calc no longer performs calculations for group parts of fixed size

if the min/max of a part are identical and > 0, the part's min size is guaranteed
to be this size. there is no need to perform expensive recursive calcs here

edje: add edje_file_iterator_new().

This list the actually opened Edje file by the current process.

edje: fix build break.

edje: add edje_mmap_color_class_iterator_new().

This function make it possible to list the Color class of a specific Edje file.

edje: add option to dump gnu style include dependencies in edje_cc.

Summary:
We were facing one problem in tizen sdk's build system as it does not trigger build for edc file
if only sub-edc files are changed. During analysis, we found that there is no option in edje_cc
for dumping include dependencies which other compiler (clang/ gcc etc) does have. We can do other
hack to solve this problem but it will be great if edje_cc can emit gnu style include dependency
target.

This patch will add support to generate gnu format include dependency file while compiling edc file.
similar to what gcc generates with option '-MMD -MF=<dep_file> -MT<dep_file>'
https://gcc.gnu.org/onlinedocs/gcc-4.9.2/gcc/Preprocessor-Options.html

Test Plan: no failure in existing test

Reviewers: raster, cedric

Reviewed By: cedric

Projects: #efl

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

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

evas/software_x11: Environment can influence the flow when the value is 1 not else.

Summary: Do not try swapbuf init only if value of env 'EVAS_NO_DRI_SWAPBUF' is 1.

Reviewers: jpeg

Subscribers: spacegrapher, cedric, wonsik

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

edje: get the color class value by iterating over all possible value provider.

edje: add edje_color_class_active_iterator_new()

This function make it possible to get a list of active Edje_Color_Class
in an application. Think about Enlightenment color class configuration,
but can now be done on any application dynamically.

@feature

edje: make Edje_Color_Class a public structure.

Evas 3D: Fix shadow map shader

Summary:
Silly mistake after this b9b5ced501f4642f03d8bcc8441fd3f7b55a7f52 commit.
Seems uninitialized components of vector gl_FragColor

Reviewers: cedric, Hermet, jpeg

Subscribers: cedric

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

edje: force NULL initialization of desc pointer.

edje: this desc pointer is actually never set.

ecore_drm: use get_vt instead of parsing the tty attribute

Summary: logind has a seperate function to get the vt number.

@fix

Reviewers: devilhorns

Subscribers: cedric

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

ecore/idler - simpler checking.

eina: include blank lines in file iterator T2237

any line that would not have been included before has length == 0

ecore_con: fix ftp upload function to follow documentation.

Summary:
While creating a Ecore_con_url object, the url is given in this format "ftp://ftp.example.com". While uploading a file, this function was prefixing "ftp://" to this url which resulted in DNS failure, and upload fail. So corrected the issue.

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

Reviewed By: cedric

Subscribers: cedric

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

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

evas: fix gl_cocoa build after recent changes in evas_common_draw_context_font_ext_set.

Now evas_common_draw_context_font_ext_set takes 3 more parameters, fix the build
by adding NULL for the function pointers. We really need to switch to gl_generic
here, it would avoid this kind of problems.

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

eina: Change newline parsing to handle crlf better

Adjust test to demonstrate failure, existing code only worked for
windows newlines on blank line.
@fix T2236

edje: edje_edit - recalculate object after chenge aspect preference

ecore: trigger events on FD_READ, FD_CONNECT and FD_ACCEPT.

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

ecore_con: do not try to load /etc/resolv.conf when on windows.

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

[Evas: Evas_3D] Improve shaders math

Reviewers: cedric, jpeg

Subscribers: cedric

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

On many platforms the built-in mix(x, y, a) function faster
than its equivalent x * a + y * (1 - a)

Ecore: Use calloc() to initialize Ecore_Event_Key fully

Some fields (eg. data) may not be properly initialized.
Use calloc() instead of malloc() will ensure proper initialization
even if we add more fields.

These fields (data) would even be passed down to Evas as
Evas_Event_Key_Down for instance.

Build: Fix horribly slow compilation times

Cedric, our dear b0rker, introduced changes in the CFLAGS
generation when merging Emile. While the changes seem to make sense
at first sight (add the -I flags for the lib our new package depends on),
they were actually a terribly bad workaround.

The number of CFLAGS args would grow exponentially, slowing down libtool
a lot, which is known to be slow when it has a lot of arguments.

edje: add of textures to Edje 3D node.

Summary: Adding textures to edje_cc and some topblock keywords that I have forgotten to add in previous commits.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric, artem.popov

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

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

eina: add eina_list_data_idx().

Summary:
Now the developer has to iterate the whole list to find the
index of the first occurence of the data.
I see that it is possible to get the index of the item for
the genlist widget, but for the eina_list not.

With these APIs it will be easier to implement *index_get
functions for the rest of widgets which contain items (Elm_List,
Elm_Ctxpopup etc.).

These functions returns the index of the given data or node in the eina_list.

Reviewers: Hermet, cedric

Subscribers: raster, cedric

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

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

ecore_con: add example for ftp upload.

Summary:
Added example for ftp upload. In the .gitignore only 2 files added which were missing. The differences it is showing is cos of reordering. I did ls and redirected the file to gitignore. So the files got reordered.

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

Reviewed By: cedric

Subscribers: cedric

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

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

evas: fix logic of 180 degree image rotation in software_generic backend.

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

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eet: fix variable naming

Summary: this file name is used for input, not for output.

Reviewers: cedric, seoz, Hermet, raster

Subscribers: cedric, reutskiy.v.v

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

Wayland shm: Fix shadow variable warning

edje: add text_class_get() APIs

Summary: No APIs to get text_class for global hash and object hash

Reviewers: woohyun, cedric, raster

Reviewed By: cedric, raster

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored shadow map shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored parallax shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

Edje: Edje_Edit - code generation for TABLE part (description.table section)

Edje: Edje_Edit - fix typo in code generation of BOX part

@fix

ecore_idler: + null check.

like timer, animator we do check the idler validation
and prevert crash there just in user misusage case.

@fix

evas: fix use after free.

Thanks to Thanatermesis for tracking this in T2135.

ecore-drm: Use better output names

Summary: As there are various DVI and HDMI output names, update the
output connector types area to be more accurate so we get better
output names

@fix

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

Evas 3D: Print SHADERS instead of SHADERS_3D at compile time

This is just a cosmetic change.

Evas 3D: Add forgotten changes after generation of shaders

Evas GL: Add missed GLES 3.0 functions.

Summary:
Initialise glVertexAttribDivisor & glVertexAttribIPointer
EvasGL function pointers.

Reviewers: wonsik, spacegrapher, jpeg

Reviewed By: jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored normal map shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored  color pick shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored diffuse shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored vertex color shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Refactor shader system: refactored flat shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

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

[Evas: Evas_3D] Fix shader code for using with GL_ES

Summary: GL_ES variables precision added.

Reviewers: cedric, jpeg

Subscribers: cedric

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

ecore_drm: Add event of ECORE_DRM_EVENT_SEAT_ADD.

Summary: To support multi-seat, ecore_drm need to issue the event to notify compositor of seat is added.

Test Plan: N/A

Reviewers: zmike, raster, gwanglim, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore-drm: Fixed null pointer free

Summary:
Fixed null pointer free
         when sd_session_get_seat() fails, it does not allocate string.

Reviewers: zmike, cedric, raster, gwanglim, devilhorns

Subscribers: cedric

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

ecore_imf: Add NULL check before accessing evas device API

evas: Add safety check for NULL pointer of clipper.

Summary: It caused crash when clipper is NULL and it makes evas test-suite fail.

Test Plan: Run evas test-suite. (make check)

Reviewers: woohyun, tasn, herdsman, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

ecore-imf: Added the device information in the Ecore_IMF_Event structs

Summary:
Added the device information in the Ecore_IMF_Event structs.
Applications may require to know actual source of key event to handle it differently depending on device.
Even if key events are currently generated from different devices, application couldn't distinguish it.
This patch will support device information for the Ecore-imf key events.

Reviewers: jihoon

Reviewed By: jihoon

Subscribers: cedric

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

evas/gl_x11: add env variables for disable partial rendering

Summary: add 'EVAS_GL_PARTIAL_DISABLE' environment variables for disable partial rendering

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: mer.kim, spacegrapher, cedric, wonsik

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

Signed-off-by: Jean-Philippe ANDRE <jpeg@videolan.org>
jpeg added atoi(s) and copied the code for gl_drm and wayland_egl.

evas/gl_x11: Separate gl symbols check

Summary:
There is a restriction for some gpu drivers that
eglGetProcAddress must be called after eglMakeCurrent.
So separate egl/glx extensions check from gl_symbols
to be called inside eng_window_new.

Test Plan: egl and glx backend tests

Reviewers: cedric, jpeg

Subscribers: cedric, mer.kim, wonsik

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

Evas: Fix invalid test case

No idea how it worked before. It shouldn't. It was invalid.
But somehow evas_suite decided to start crashing on this test.

[Evas: Evas_3D] Refactor shaders: used m4 for removing duplication of code.

Reviewers: Hermet, raster, cedric, jpeg

Subscribers: cedric

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

Revert "evas/gl_x11: Optimize out calls to make current"

This reverts commit 0585540bb3558b2ff834a1bb730e897c6301a224.

This broke Evas 3d examples. I also suspected some weird things and
wasn't 100% confident with this patch.

Closes T2215.
Thanks for the report.

emile: meaningful initialization of some pointers.

ecore_con: clean up, refactor and unpdate documentation.

Summary:
1. Have refactored code in ecore_con_local.c
2. Changed env variable from "TMP" to "TMPDIR".
3. Added check for negetive port number when type is ECORE_CON_LOCAL_USER.
4. Added check to check TMPDIR before assuming /tmp as temp directory.
5. Updated documentation in Ecore_Con.h explaining about local socket and port number.

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

Reviewed By: cedric

Subscribers: cedric

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

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

ecore_exe: fix ecore_exe_pid_get on windows

Summary:
ecore_exe_pid_get returns exe->pid value so it should be filled

@fix

Reviewers: raster, seoz, cedric, Hermet

Subscribers: NikaWhite, cedric, reutskiy.v.v

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

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

eina: add test cases for various eina xattr data type functions

Summary:
Added test cases for eina_xattr_string_set, eina_xattr_int_set and
eina_xattr_double_set functions

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

Subscribers: cedric

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

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

ecore-drm: Fix issue of not getting updated for output hotplug events

Summary: When setting up the Eeze_Udev_Watch for output hotplug
events, we should also be listening for the "change" event. As eeze
will now check the HOTPLUG property for drm events, we don't need to
check that here.

@fix

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

eeze: Fix issue of eeze_udev_watch_add not getting events for drm

Summary: This fixes eeze_udev_watch_add for DRM events, Previously, we
were not getting a watch callback fired due to using the wrong
subsystem device type in
udev_monitor_filter_add_match_subsystem_devtype.

@fix

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

evas: prevent crash.

if user delete the interceptor in the interceptor callback,
the interceptors would be NULL.
So we need to check whether it's valid or not.

@fix

emile: fix build on NEON enable system.

T2214

evil: fix access to a file mapping object

Patch from Jorge Luis Zapatta (turran), a bit modified
by me

@fix

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

ecore-drm: Fix build error caused by missing function declaration

Summary: This is mainly just a build fix. This issue occurred due to
the function declaration being After a call to use it while not having
a function prototype. This just adds the function prototype

@fix

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

ecore-drm: Add more debug info when output is created or updated

Summary: This patch adds more information so we can debug output
hotplug events better. It fixes an issue where any new outputs
from _ecore_drm_update_outputs where not getting the eeze udev watch
event created, and also addresses the issue of disconnected outputs
not sending the ecore_drm_event (both minor fixes)

@fix

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

evas-drm: Enable more information during flip error

Summary: This adds a bit more output during crtc flip errors so we can
debug multi-monitor setups

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

eo: fix wrong check libaddr dependency.

move checking libaddr to eo step to valid it.

Wayland: Fix compilation

Bad merge?

ecore_wayland:  Check for nulls before wayland calls

Summary:
Wayland API routines by policy do not check input parameters for invalid
values.  In particular, many calls are wrappers through
wl_proxy_marshal, which derefs its first argument without a check.

So, for all wayland calls, always null check the first argument.

Reviewers: zmike, cedric

Subscribers: cedric

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

ecore_wayland: Check for null inputs

Summary:
The other functions in this module check inputs; this one should as
well.

Reviewers: zmike

Subscribers: cedric

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

ecore_wayland: Add some comments

Summary: Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>

Reviewers: zmike

Subscribers: cedric

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

Revert "evas-3d: materials in .mtl format are deleted"

This reverts commit 59271716ba5443bcba215f5dfbc190fc107ae4ab.

This patch submitted twice.

evas render2 - restructure it to be an explicit api call - cleaner to do

evas-3d: materials in .mtl format are deleted

Summary: Materials are not used in any loader and will not be used in nearest future.

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

Conflicts:

src/examples/evas/Makefile.am

evas: fix description about Evas_Device_Subclass

Fix wrong drawing of shadows for some blending modes

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

emile: Remove unnecessary check in _emile_jpeg_head function

Summary: Already checked it by "else if ((opts->w > 0) && (opts->h > 0))"

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

evas: fix build failure with old version freetype.

Summary:
Fix build failure with old version freetype.
It is caused for supporting colored font.

Reviewers: raster, jpeg, woohyun, Hermet

Subscribers: cedric

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

evas - font - new color bitmap support - rename funcs to match file

evas-wayland-shm: Fix redrawing issues that were causing 'flashing' in latest engine code.

Summary: As reported by derek & zmike, the previous engine refactor
left some issues where the app would 'flash' during redraws due to
buffer handling. This commit fixes that issue along with flashing
during resize.

@fix

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

evas-software-generic: Allow future rendering calls if evas has to skip a frame

Summary: If we do not get a new_region_for_update from an engine, it
could mean that the engine has skipped a frame. In that case, we still
want to try and render any future frames. Before this commit, evas
would never try to re-render anything again once it hit a failed
frame. To fix this, we test for a valid returned region, and if we
don't get one, mark this frame as failed so we can try again.

NB: Thanks for the help Cedric !! :)

@fix

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

evas: use switch instead of a serie of if for cleaner example.

evas: remove unused variable.

evas: remove useless line.

elua: cast data, not cbs (data stores the cb id)

Revert "elua: more reliable eoid->lua number conversion"

This reverts commit 8bb13b626f3a5d284f50af493f3f7b5566e06586.

Seems like this method is not so reliable after all :) Results
in precision loss for Eo IDs, so revert to the old hacky way.

evas-gl-common: Fix building when --with-opengl=es

Summary: If we are building for only gles 2.0, then we cannot use
glAlphaFunc else we get compile errors. This adds an #ifndef around
that block to fix building

@fix

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

Edje: edje edit - ability to remove last item in BOX/TABLE

Removing last item in BOX/TABLE part is actually successful
but then we shouldn't realloc an array of items for 0 items.
That's ridiculous
(and because of that function removes EINA_FALSE,
so user could think that it's unable to remove last item).

So simply array can be set into NULL.

@fix