Ector: Move cairo surface back to ector

The original plan was to have two different surfaces for GL and SW,
but this is probably not going to happen anytime soon. So, move
the implementation back to lib/ector. This avoid a file duplication.

Ector: Rename C files to match their EO counterparts

Ector: Another minor code cleanup

Remove DATA8, DATA16, DATA32
Remove empty data structure
Remove unnecessary typedef

Ector: Move drawhelper to static_libs

Rename a few things:
 - draw helper -> efl_draw
 - Ector_Rop -> Efl.Gfx.Render_Op
 - ECTOR_ bla bla -> DRAW_ bla bla (base pixel ops)
 - ector_memfill -> draw_memset32 (and invert arg order to match memset)

The main rasterizer file is now draw.h in static_libs/draw
This is a non functional change, simple code refactor.

Ector: Use Ector Buffer inside SW and Cairo renderers

Ector Surface now inherits from Ector Buffer, and the current
two renderers (SW and Cairo SW) use Ector.Software.Buffer
implementations for pixel surfaces.

Basic pixel handling is merged and will allow easy extension
(color conversion, etc...).

Buffer classes are Mixins to be fully implemented by the final
class, such as: Ector.Software.Buffer, Ector.Software.Surface
or Ector.Cairo.Surface.

This is a large ugly commit. Sorry.
The code is quite a mess right now.

Ector: Implement pixel buffer support

The objective of this patch is to propose a standardized
format for pixel buffers to use within Ector and Evas.

The basic EO API provided here is not meant to be the
fastest path for all operations, simply the most convenient
to generalize. Performance will be achieved by implementing
(or porting) custom draw functions.

This implements support for:
- Generic pixel buffers
- Generic buffer renderer to draw images with ector
- Software engine pixel buffers, ie. malloc buffers
- Software buffer renderer

Cairo support has not been implemented yet.

The only renderer is still extremely limited, as it does not
support Fill modes, Scaling, etc... yet.

Not a single line from this patch has been tested yet.
It compiles. That's pretty damn good for a start!

@feature

Eio: Fixes T2831, eio xattr tests failing

Summary:
@Fix
The eio_file_xattr test was failing in jenkins, so added a new callback
and moved the check condition to callback. The condition is checked
only when it is successfully executed

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric, stefan_schmidt

Subscribers: jpeg, cedric

Maniphest Tasks: T2831

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

Edje entry: emit "selection,reset"

Adding a check for the selection range on extend/preextend.
If there is no range, then emit "selection,reset".
Expected handling on the theme end for this signal is to show the
cursor.

This improves cursor visuals, as it will show the cursor again once the
selection range ends up being 0.

evas: rewrite gl image transformation logic in a very simple way

all we need to do for image transformation such as rotation or flipping in gl
is to map the vertices of source image to destination in a changed order.
this commit not only enhances readability but also fixes bugs in rotation.

@fix

edje: fix edje RTL description in case of custom state of a part

Summary:
In case of RTL, the "custom" state properties does not apply. It happened because we don't copy the latest src to
dst in set_state(PART:.., "custom", 0.0); in case of dst is already populated.
We should copy the updated src to dst whenever we set the new custom description.

@fix

Reviewers: cedric, raster, jpeg, zmike, jaehwan

Subscribers: kimcinoo, seoz, jpeg

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

evas: set visible texcolorpick texture in 3D shaders

Summary:
Add uniform varialble uColorTexture. Generate sampler count for textcolorpick
and pass to renderer for bind additional texture unit to gl.
It can be used for different 3D effects - blur, wave distortion, heat haze, etc.
T2761

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

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

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

edje: add missing EAPI keyword for edje_color_class_del

Reviewers: jpeg

Subscribers: cedric

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

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

ecore_evas: fix disappearing socket image

Summary:
In case there is connection between a server and some client,
show/hide does not work properly.

If there are clients need to show,
this patch make it hide operation will not work even if get the hide signal

@fix

Reviewers: jypark, Hermet, cedric

Subscribers: Hermet, cedric

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

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

evas: fix bug with logic with eina_matrix in evas_3d_utils

Summary: Also need to fix logically dead code in coverity.

Reviewers: raster, Hermet, cedric

Subscribers: jpeg

Maniphest Tasks: T2832

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

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

edje: Remove redondant array in Evas 3D edd (part2)

Summary: There are more redundant arrays to be deleted. In fact I don't think string name of descriptor shouldn't never change, I was testing some 2D examples and this is not true.

Reviewers: raster, jpeg, cedric

Subscribers: artem.popov, cedric

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

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

evas: properly clean up all memory.

evas: fix double free.

The ownership of this list and its content belong to the iterator. This code
was actively double freeing and our test suite doesn't crash or have any issue...
Well actually it was complaining that the list had error, but due to our other
false positive test that do trigger eina log, we continuously ignored this error.

NOTE: The fact that the test rely also on the container of the iterator instead
of the iterator itself is not that great to. I kind of feel bad now for having
allowed access to the container. We should have been able to allow changing the
logic to generate those rectangle step after step instead of in one go. Now this
seems difficult.

Evas textblock: fix height on line wrapping

Line advance should finalize the line with its last item, and not the
item we're currently wrapping. This fixes T1583, where some line
wrapping cases would look different than their equivalent <ps>
versions.

@fix

fix efreetd to scan more subdirs for desktop and icon files

we miss desktop files for apps and stuff because we dont monitor too
deep a tree. this ups our tree depth to 10 levels.

@fix

efreet desktop tracking - fix monitoring of dirs of custom desktops

@fix

this is wrong - start monitoring every/any dir in which a desktop file
exists that we load a desktop file from. imagine you browse directories
in efm with lots of desktop files in them - we end up monitoring lots
of directories that we then rememebr and don't un-monitor. this
disables monitoring of dirs from which we load a .desktop file from to
fix this.

build: fix distcheck after model saver and loader rework

32c33ed64dda542c7cfc952fc656bb711260441b broke distcheck here as the header
files would not be included in the generated tarballs. This is the second or
third time I fixed soemthing like this after a loader/saver rework. I would
appreciate if the people involved would run distcheck on their own.

Evas Textblock: Fix text disappear issue when text is made up with multiple items.

Summary:
Text is disappearing when we resize a singleline Evas Textblock with ellipsis.
It is happened by putting a Text item at logical_items list without considering about logical position.
It is only happended the text is made up with multiple items.
@fix

Test Plan:
1. Run elementary_test
2. Click Label Ellipsis
3. Resize the window dynamically and see the result.

Reviewers: woohyun, tasn, herdsman

Subscribers: jpeg, subodh6129, shilpasingh, cedric

Maniphest Tasks: T2709

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

evas_gl_shader: refractor shader/program compile check code.

Summary:
    Call respective shader/program functions on shader/program target only.

Reviewers: cedric, raster, wonsik, spacegrapher, jpeg

Reviewed By: jpeg

Subscribers: alok25, sachin.dev, cedric, seoz

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

edje cc fix - we parsed floats not intsfor min and max desc size

thanks conr2d for pointing this out - we uses the float parse func and
not in parse. we should use int parse as the min and max sizes are
just ints and not floats. :)

@fix

evas: refactor model's savers and loaders.

Summary:
Move common part to a separated document.
Make code more readable using smaller functions. (from Task T2713)

Reviewers: cedric, raster, Hermet

Subscribers: artem.popov

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

efl: fix build failure when it builds with coverage.

Summary:
When src/bin/efl/ builds with [--with-tests=coverage], it fails to find gcov lib.
The gcov/lcov related options has to be passed when it is builded.
And it only contained in EFL_CFLAGS, EFL_LIBS.
@fix

Test Plan:
Be sure the your enviroments to build src/bin/efl/.

1. Run ./autogen.sh --with-tests=coverage
2. make or make check
3. See the build errors.

Reviewers: raster, jpeg, cedric

Reviewed By: cedric

Subscribers: herdsman

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

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

evas: fix a NULL dereference issue in font.

Summary:
eina_list_remove returns Eina_List pointer.
It could be NULL if the last list item is removed.
And the returned Eina_List pointer could be different from the given list.
So, calling free for fdir->data after fdir's address is changed is dangerous.
@fix

Test Plan: Run expedite or test app with evas_font_path_append() API.

Reviewers: stefan_schmidt, jpeg

Reviewed By: jpeg

Subscribers: stefan, jiin.moon, cedric, jpeg

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

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

edje: add mising closing brace and adjust indents and spacing in test edc

Summary: A closing brace is missing, and indent is not aligned.

Reviewers: cedric

Subscribers: jpeg

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

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

evil: better check of Windows 64 bits

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

ecore_cocoa: fix live resize of windows

This was a tricky little bastard!
When a window is live resized, the NSWindow which is the target of
the live resize will wait for a kevent from the window manager,
until live resizing is done. So... live resizing is synchronous
and blocks the main thread... hence ecore_main_loop.

- When live resize starts, the Ecore_Timer which polls NSRunLoop
is paused.
- When the window is resized, the ecore_main_loop is run manually
with ecore_main_loop_iterate() to process Ecore events (mostly
Ecore_Evas)
- When live resize finished, the Ecore_Timer which polls NSRunLoop
is resumed.

@fix

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

ecore_cocoa_app: don't override init

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

ecore_cocoa: don't make ecore_cocoa_feed_events() a public API

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

ecore_cocoa: NSApp must be configured in the App wrapper

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

ecore_cocoa_window: cosmetic changes

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

ecore_cocoa: don't export Ecore_Cocoa_Keys.h

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

ecore_x: factorize multiple calls to strlen()

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

ecore_cocoa: factorize code

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

evas_gl_cocoa: make sure focus is always unlocked after locking it

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

ecore_cocoa_cnp: make API thread-safe

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

ecore_cocoa_cnp: clear the clipboard only if writable

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

ecore_cocoa: NSEvent modifiers are of type NSUInteger

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

ecore_evas_cocoa: factorize code

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

ecore_cocoa: report error when [super init] fails

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

ecore_evas_cocoa: remove dead code

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

ecore_cocoa: remove EINA_UNUSED because parameter is used

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

ecore_cocoa: remove debug log

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

ecore_cocoa: reindent file

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

ecore_cocoa: add more safety checks

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

ecore_cocoa: remove EINA_UNUSED because parameter is actually used

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

ecore_cocoa: prevent Ecore_Cocoa_Window to be re-defined

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

ecore_cocoa: remove ECORE_COCOA_EVENT_EXPOSE

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

ecore_cocoa: remove unused events

ecore_cocoa: start doxygen

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

ecore_evas_cocoa: focus: handle ignore_events

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

ecore_cocoa: don't track NSAppKitDefined event

Currently, it was used only to get track of focus, howver focus was
already handled (and better) in ecore_cocoa_window (with
NSWindowDelegate).
This led to the LOST_FOCUS event to be posted twice, with different
values, which messed up the focus stack.

@fix

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

ecore_evas_cocoa: don't reset the init count if negative

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

ecore_evas_cocoa: use Eina macro to determine the size of a static array

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

ecore_evas_cocoa: don't shutdown ecore_event_evas() twice

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

ecore_evas_cocoa: EAPI cannot be set for WIN32

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

ecore_cocoa: fix types

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

ecore_evas_cocoa: remove dead code

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

ecore_cocoa: copy and paste support

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

ecore_cocoa: silent warning: unused parameter

ecore_cocoa: use EAPI in implementation

ecore_cocoa: improve const-correctness

ecore_cocoa: don't specify inline

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

ecore_cocoa: add support for system cursors

- Ecore_Cocoa_Cursor enum which references system cursors;
- API to show/hide cursor: ecore_cocoa_window_cursor_show();
- API to set system cursor: ecore_cocoa_window_cursor_set();
- Ecore_Evas interface to get Ecore_Cocoa_Window from Ecore_Evas.

@feature

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

ecore_win32: add support of X11 shaped cursors

@feature

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

ecore_win32: add clipboard management

@feature

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

eina: use eina_file_path_join() and eina_environment_tmp_get() when needed

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

ecore_file: fix ecore_file_file_get() on Windows

@fix

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

efl: change efl_gfx_shape_cubic_to() api signature to follow other API (cairo and freetype).

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

efl: updated efl_gfx_shape_dup() implementation to use direct access to internal data.

As I don't see the use of inheritance for stroke anymore, we will use internal access
rather than inherited get/set function. This behavior can be reverted if anyone find
a real use case for it.

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

efl: add bookkeeping for convex shapes.

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

efl: update append_rect implementation.

This now generate more optimized path by reducing the use of arc and
switching to only line whenever possible.

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

efl: update append_circle implementation to use _efl_gfx_shape_append_arc.

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

efl: add alternative append_arc implementation

This allow to reduce the number of generated arc, but doesn't follow
SVG specification. This is just used internally and can't be used by
outside call.

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

eina: add new api eina_bezier_on_interval()

I couldn't come up with a better name, if anyone has a suggestion.

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

eina: fix bad use of calloc.

eina: fix missing header

This caused functions from eina_util to be undefined at compile time.
The compiler would make implicit casts of return values into ints,
which had major side effects (e.g. segfault edje_cc)

@fix

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

evas_3d_utils: do not bitwise AND against 0 in condition

We need to check against the state here and if the compilers assignes 0 to the
first item in an enum we are screwed here as the bitwise AND will always
evaluate to false.

This is a re-incarnation from a486671bce76e405a8d7261f19b0f67ca678f0c4

evas_font_dir: do not passed free'd memory to eina_list_remove()

Free the actual memory after we removed it from the list.

Evas: use LC_MESSAGES on Windows to fix compilation

Compilation is broken on Windows because LC_MESSAGES does not exist on this platform
Include Evil.h to provide support of LC_MESSAGES

@fix

Evil: add support for LC_MESSAGES for setlocale()

LC_MESSAGES is an extension to C ANSI and does not exist on Windows.
So add LC_MESSAGES API and overload setlocale() to support it

@feature

ecore-drm: Add logical pointer x, y variable in seat for reflecting multiple pointer's movement

Summary: When one pointer moves, we should update the position of other devices.

Test Plan:
(1) Two pointer devices are connected.
(2) Move the cursor to (x, y) position using "device 1".
(3) When you move the cursor using "device 2", the cursor doesn't start from (x, y) position. This causes discontinuous mouse motion.

Reviewers: raster, zmike, gwanglim, stefan_schmidt, devilhorns, ManMower

Reviewed By: devilhorns, ManMower

Subscribers: cedric, Jeon, input.hacker, jpeg

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

emotion: correct argument order for calloc()

calloc() expects count first and the actual size to allocate as second
argument. Say Thank You to smatch for finding this issues for us.

examples ecore_buffer: correct argument order for calloc()

calloc() expects count first and the actual size to allocate as second
argument. Say Thank You to smatch for finding this issues for us.

evas-3d: Fix missing field initializers

Clang spits warnings here about missing field initializers for
Evas_Vec3 (missing y and z fields), so we will explicitly initialize
them to 0 (matching the x field).

@fix

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

Evil: fix warning

Use the USERPROFILE environment variable instead of deprecated evil_homedir_get
function. Also set the shell to cmd.exe if the SHELL var is not found

eolian: fix eo makefile example of --eo option.

Summary: eolian: fix eo makefile example of --eo option.

Reviewers: cedric, q66

Reviewed By: q66

Subscribers: jpeg, q66

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

Revert "Evas: Temporarily hack OT_SUPPORT to fix textgrid"

Fixed in upstream, see T2865.

This reverts commit 8ccea8233c144f723470da09a01487484c269440.

eina_matrix: optimize multiply and inverse calls by actually using the shortcut

We had this nice shortcuts for multiply and inverse with the identity matrix.
Pity we never used it! The EINA_MATRIX_TYPE_IDENTITY is coming from an enum
without and direct assignments to its internals. Being the first item in the
enum it is most likely will be 0 which makes the whole bitwise AND zero and thus
the optimized path will never get called. If our compiler now decides hew wants
to handle enums differently and does not assign the 0 to the first item this
bitwise operation will be even more screwed. What we really want is to check is
if the type we get for the matrix matches EINA_MATRIX_TYPE_IDENTITY. So better
do this. Made me look into matrix multply and inverse. Fun!

Thanks to smatch for poiting this out.

rg_etc: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

evas software_x11: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

evas gl_common: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

evas_common3d: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

eolian: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

ector: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

evas_cserve2: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

edje_decc: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.