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.

evas: Add ExtraLight, ExtraBold for font weight.

Summary:
Evas supports UltraLight, UltraBold as font weight.
These terms have same weight value as ExtraLight, ExtraBold.
Some applications, for example, fontforge, use ExtraLight, ExtraBold terms for these weight values.
So, it would be better to support these terms, too.
@feature

Test Plan: None

Reviewers: tasn, woohyun, herdsman

Reviewed By: herdsman

Subscribers: cedric

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

edje entry: return correct selection

Summary:
In entry, when selection_get function is called, selection is not
always returned the current selection.

Scenario:
- In select mode, entry has selection (e.g, by double click).
- When selection handler is moved, set the cursor the current coordinate
  with edje_object_part_text_cursor_pos_set() API.
- Edje emits "selection,changed" signal.
- Elementary gets current selection and stores it.
- Elementary does not get selection as visual selection (e.g, text is
   highlighted with "entry test", but the selection returned
   from edje is "entry").
- If we copy and then paste to the entry, the pasted text is not same as
   selected text.

Reason:
- In _edje_entry_cursor_coord_set function, if entry has selection, we only
  emit "selection,changed" signal without freeing en->selection.
- When _edje_entry_selection_get is called, we check the en->selection,
  since it is existed, we just return it which is not updated one.

This patch clears en->selection, so that it is updated at _selection_get,
and the updated selection is returned to caller.

@fix

Test Plan:
In mobile profile, open entry
- Right click, choose select, double click -> selection handlers are shown.
- Drag selection handlers to change selection.
- Right click, do copy.
- Right click, do paste.
- See the pasted text is not same as selection.

Reviewers: raster, tasn, herdsman

Subscribers: seoz, JackDanielZ, cedric

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

build: rename Eolian helper cmake file to mark it as input

Align it with the rest of our cmake support files. Adding it to configure as
well so the cmake file gets generate from the cmake.in

Without this I had distcheck failing with a missing target for it.

cmake: Include module per default

if someone uses eolian its likly that he wants to install some rules

cmake: Add a helper script which defines generator rules for eo files

eo_rule_create adds rules to generate to .eo.x .eo.h .eot.h files.
It also monitors its deps.

@feature

efreet_icon: Fix wrong inherit selection

Summary:
The list of elem->paths is given with the actual icon at position 0 and
the bigger the index is the lower the icon in the inherit structure is.

Due to the for loop beginning at 0 walking to the end this
direction is flipped. So the last r is returned, which is the lowest
icon in the inherit structure.

This is fixed by returning if the first valid path is found.

@fix

Test Plan: run jesus or efm with a custom icon theme beore the wrong icons are taken, now the correct ones are taken

Reviewers: raster, cedric

Subscribers: DaveMDS

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

Evas: Temporarily hack OT_SUPPORT to fix textgrid

See T2865.
Since Harfbuzz 1.1.0, terminology displays fonts funnily aligned to
the top. This is apparently because until 1.0.6 the y_offset was
always 0 for all glyphs, but since 1.1.1 the offset is actually
set.

This is a TEMPORARY fix. There might be an underlying issue left
here.

Harfbuzz changed behaviour in this commit:

  commit 44f82750807475aa5b16099ccccd917d488df703
  Author: Behdad Esfahbod <behdad@behdad.org>
  Date:   Wed Nov 4 20:40:05 2015 -0800

      [ft] Remove font funcs that do nothing

ector: cleanup freetype raster

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

ector: support fill rule for shape object.

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

efl: added fill_rule property in efl_gfx_shape class

Open question, should fill rule be part of the public data ? Let's
say no for now.

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

efl: make path stroke related api as non virtual.

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

efl: quick code cleanup.

ector: updated the freetype raster with faster line renderer.

Merged from freetype repo with 'smooth', faster, alternative line renderer.

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

ector: fix the rounding issue when applying transformation to shape data in freetype backend.

@fix

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

evas: avoid double eet_close call while caching GL shader.

eet: improve error message during eet_close.

Edje: Add example for hyphenation style

A small example how hyphenation is set as a style in TEXTBLOCK parts.

Evas textblock: add hyphenation tests

Evas textblock: add support for hyphenation wrap style

We now support hyphenation in style. Use "wrap=hyphenation" to use this
wrap option. It will hyphenate based on explicit SOFT HYPHEN (&shy;)
placement in the text, and with the (optional) assistance of dictionaries
compatible with Hunspell's "hyphen" library.

This wrap mode favors breaking at hyphen positions in a word, over moving
the whole word to the next line. It will put an additional "-" at the
break position if it was hyphened.

Enabling the hyphen dictionaries is done by adding these configure
options:
  --enable-hyphen (requires Hunspell's "hyphen" library installed)
  --with-dictionaries-hyphen-dir=DIR (specifies the install location of
          the actual .dic dictionary files e.g. /usr/share/hyphen)

Note that dictionary files are expected to be in the form of "en_US.dic"
or anything that ends with it e.g. "hyph_en_US.dic" (this how they are
        named in Arch Linux).

@feature

Eina unicode: add eina_unicode_unicode_to_utf8_range

Required some special treatment to get words (substring) out of an
input Unicode string to a utf8 one. This saves the trouble converting
the whole string: you input an offset in the Eina_Unicode array, and
provide the required length. That's is, now you can extract words and
whatnot in utf8 form.

To save code I wrapped the original one to get the whole length.

eina: Applied NULL check

Summary:
In _eina_list_setup_accounting function
{
...
...
if (!list->accounting)
     goto on_error;
...
...
on_error:
   _eina_list_mempool_list_free(list);
}

_eina_list_mempool_list_free function deference the "list->accounting"
variable which is already NULL.

Reviewers: JackDanielZ, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

Evas 3d: Initialized local variables

Summary:
The OpenGL functions  assigns a value to the local variable. However,
in case of error it would take the uninitialized garbage value. So, the
correct solution would be to initialize it to 0.

Signed-off-by: Deepjyoti Dutta <deepjyoti.d@samsung.com>
Reviewers: Hermet, singh.amitesh, raster, jpeg

Reviewed By: jpeg

Subscribers: yashu21985, mvsovani, alok25, sachin.dev, cedric

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

autotools: detect IPV6 support on Windows

The detection in configure.ac includes netinet/in.h, for linux, but this header
file does not exist on Windows. So guard this header.

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

eina: simplify tests logic for base64.

eina: always terminate the string with a '\0'.

Eina Example: Fix memory leak of variable buf

Summary: Fix memory leak of buf in eina_test_simple_xml_parser.c as reported by static analysis tool, Cppcheck.

Reviewers: singh.amitesh, Hermet, jpeg

Reviewed By: jpeg

Subscribers: alok25, mvsovani, sachin.dev, cedric

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

evas: Fix text effect which has shadow direction

Summary:
If text object has outline and shadow effect (OUTLINE_SHADOW or OUTLINE_SOFT_SHADOW), and has shadow direction except BOTTOM_RIGHT then outline is not displayed.
So fix logic of style checking.

OUTLINE_SOFT_SHADOW effect of text object is different with text block, so fix it.

Test Plan: Make text obect with OUTLINE_SHADOW or OUTLINE_SOFT_SHADOW effect with any direction except BOTTOM_RIGHT

Reviewers: cedric, woohyun, herdsman, tasn

Subscribers: id213sin

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

evas_fb: Handling memory leak on realloc failure.

Summary:
I thought its better to fail and return null if realloc fails than to
continue. So returning by closing all openend file.

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

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

Evas 3d: Remove left over debug message from recent commit

Edje edit: Free textblock styles with the proper function

Edje: Remove duplicate NULL reassignment

Summary:
Remove duplicate code of variable 's' assignment to NULL in edje_edit_style_del.

Signed-off-by: Yeshwanth Reddivari <r.yeshwanth@samsung.com>
Reviewers: Hermet, alok25, mvsovani, singh.amitesh, jpeg

Reviewed By: jpeg

Subscribers: sachin.dev, cedric

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

eet: Fix warning in eet file

Summary:
variable 'total' is of unsigned long long but %lli was used to print..so fixed
warning

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

Reviewed By: jpeg

Subscribers: cedric

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

evas: fix PLY loader and saver.

Summary:
Use less memory by indexation vertices and not keeping more than one copy of it (from task T2713).

[Fix]

Reviewers: cedric, raster, Hermet

Subscribers: artem.popov

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

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

evas: remove useless part of code in evas_convex_hull_get function

Summary:
@fix
CID: 1339788

Reviewers: raster, cedric

Reviewed By: cedric

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

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

evas: fix uninitialize variable in convex_hull_vertex_set function

Summary:
A bit of useless claim, becouse parameter coord always have value 0 or 1 or 2.
May be for escape problem in future.
@fix
CID: 1339781

Reviewers: raster, cedric

Reviewed By: cedric

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

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

evas: fix possible accsess to NULL pointer in Evas.Canvas3d.

Summary:
@fix
CID:1339784

Reviewers: raster, cedric

Reviewed By: cedric

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

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