eina: as we use string.h function in this inline function, we better include it.

Reported by @netstar, thanks.

ecore-evas-x: Remove blank line

NB: No functional changes

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

efl_debugd: only compile this on Linux.

Summary:
Well, the build is broken again on BSD and Windows.
efl_debugd is full of lots lovely Linuxisms. Just
don't compile.

This unbreaks the build for FreeBSD and others.

Reviewers: cedric, raster, jpeg

Reviewed By: jpeg

Subscribers: JackDanielZ, jpeg

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

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

ecore-drm2: Fix potential SIGFPE when adding output mode

If we receive bad crtc info from libdrm, then we could end up with a
SIGFPE here due to division by zero if info h/v total are not set.

@fix

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

ecore-drm2: Add API function to set pointer rotation

This patch adds a new API function which can be used to swap x & y
pointer axis and invert them according to rotation angle. Mouse input
events occur according to canvas coordinates so this can be used when
a canvas is rotated.

@feature

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

elput: Add API function to swap dx & dy axis from pointer motion event

Small patch which adds an API function that can be called to swap x
and y axis and invert them according to rotation angle.

@feature

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

elput: Minor formatting fixes

NB: No functional changes

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

elput: Minor formatting fix

NB: No functional changes

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

elput: Get touch event slots using slot not seat_slot

@fix

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

ecore-evas-wayland: Fix potential null dereference

Quick patch to fix a potential null derefence in render_flush_pre
callback.

@fix

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

evas-gl-drm: Fix uninitialized variable

Small patch to fix an uninitialized variable when using
eglGetConfigAttrib()

@fix

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

evas-gl-drm: Be sure to initialize ncfg variable

This could lead to an uninitialized variable or use of ncfg variable
without a value.

@fix

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

widget: Remove unnecessary define

Looks like a copy & paste went wrong :)
Thanks @bu5hm4n

edje - fix signal propagate crash

fixes segv when using boxes/tables and sub groups with signal
propagation working in _cb_signal_repeat() when name is NULL (not
found/there)

@fix

elm_test:ui.image.zoomable: Allow image click to enable/disable animation

its better to have animation start/stop on image click rather
than clicking a button.

Efl.Ui.Image.Zoomable: inherit elm.widget as class

this fixes a mistake done in cb1d53c835ef465
Thanks to jp for notifying it.

ref T5352

Efl.Ui.Image.Zoomable: implement efl.play.get/set

ref T5352

Efl.Ui.Image.Zoomable: implement efl.player.playable.get

ref T5352

edje: pass whole text to IMF in case of selection

Summary:
This patch partially reverts D2951.
_edje_entry_imf_retrieve_surrounding_cb() function has to pass
whole text nearby the entry's cursor. If IMF needs to ignore
selected text when the entry has selection, IMF can check
selected text by calling _edje_entry_imf_retrieve_selection_cb().
So, we don't need remove selected text before passing it to IMF.
@fix

Test Plan: N/A

Reviewers: woohyun, jihoon, subodh6129

Reviewed By: subodh6129

Subscribers: cedric, jpeg

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

efl.gfx.map: support for map points as multiples of 4

This patch adds support for map with more than 4 points.
However, we only support points with multiples of 4.
This also adds efl_gfx_map_point_count_set/get APIs to
set number of points.

@feature

evas_map: support map with number of points as multiples of 4

Currently, in evas map, we only support map with 4 points.
This patch adds support for map with number of points as
multiples of 4.

@feature

edje: Fix warnings introduced in recent fix

See b9b0177d5e0f2b3a03f77e04c64de534b2c786ac
See d3420cd35c692a808c0a626c8e54a0c85af3a4d4

elm_panel: set hidden property when panel is close/open

Summary:
 - elm_panel has a property named hidden which stores
   open/close status.
 - This is updated when:
    1. bring_in animation is done(anim_stop_cb).
    2. mouse_up on panel.
    3. API is called. (elm_panel_toggle, elm_panel_hidden_set)
 - In case 3, API changes hidden, and starts bring_in animation
   which will call anim_stop_cb() which will update hidden again.
 - If bring_in animation is canceled (eg: sizing_eval),
   anim_stop_cb will be called and calculate hidden status
   which will not guarantee updated hidden state by APIs.

Test Plan:
   1. Call any APIs which will call elm_layout_sizing_eval(panel)
      right after calling elm_panel_toggle()/elm_panel_hidden_set().
   2. Delete content of panel during "toggled" cb.

Reviewers: jpeg, eunue, cedric

Subscribers: conr2d, cedric, jpeg

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

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

evas - font dir fonts.alias - limit alias parse length

a rouge fonts.alias file in a font dir/path could contain long strings
that buffers dont have space for, so limit length of string.

@fix

elm spinner - handle more digits for format string + handle invalid fmt

handles format strings like %0.234567f even tho before it was just %2f
- single digit after . ... not that it's useful handling more...

@fix

evas xpm loader - add more rgb.txt file locations...

2 more. /etc/X11/rgb.txt /usr/share/vim/vim80/rgb.txt  ...what i do notice
is that this file seems to have vanished from modern systems... so we'll have
lots of un-fun loading old xpm's with colornames if we cant figure out what
color names map to what colors...

edje_cc handle line too long for buffers

if a line from the cpp processor is too long (more than 4k) then abort
compilation as something is wrong asnd our buffers can't handle it
anyway.

@fix

elm_prefs_cc: handle line too long for buffers

if a line from the cpp processor is too long (more than 4k) then abort
compilation as something is wrong asnd our buffers can't handle it
anyway.

@fix

evas-gl-drm: Fix potential memory leak

If/When we exit _evas_outbuf_egl_setup function, we should be freeing
the allocated cfgs variable else we leak it.

@fix

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

ecore-wl2: Use output height when calculating dpi

When calculating output dpi, we should also be considering output
height in the calculation.

@fix

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

ecore-wl2: Add API function to set surface buffer transform

Small patch to add an API function which can be called to set a buffer
transformation on a given window.

@feature

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

ecore-wl2: Add API function to return an output transformation

Small patch to add a new API function that can be used to retrieve the
current transform value for a given output.

@feature

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

ecore-wl2: Add missing doxygen groups for documentation

@fix

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

ecore-wl2: Add API function to find an output for given window

Small patch which adds an API function that can be used to find the
output where a given window resides.

@feature

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

ecore-wl2: Add event for output transformation

Some servers may need to do special animations or other effects when
an output transformation happens, so raise an event in the output
geometry callback when this happens.

@feature

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

ecore-evas-wl: Don't check window visibility on raising

As we won't get a request from a server to raise a window which is not
visible anyway, this check is useless.

@fix

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

Efl.Ui.Image: fix efl.player.playable.get API

an edje obj is always playable. return TRUE always.

@fix

scroller : Call mirrored_set function after edje_obj is initialized

Summary:
Problem that scroller is not mirrored in RTL mode has been fixed in D4908.
However, I found a problem in init process.
I will revert D4908 commit(https://phab.enlightenment.org/D4908).

After edje_obj is initialized, mirrored_set is called.
When wanted_region_set is called, x-coordinate flip normally and acts as RTL.

Test Plan: scroller test on elementary_test.

Reviewers: raster, woohyun, SanghyeonLee, akanad, eagleeye, cedric, singh.amitesh, eunue

Subscribers: jpeg, cedric

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

evas: Set locks and modifiers for all input events

See also eb27724eb9c32a954c2f75b45307a207d0 which didn't fix
the reported issue. Not quite sure yet how to implement those
lock & modifiers for fake events (i.e. artificially created
by the app itself).

Fixes T5510

edje: Remove external param_type from EO

This is not exactly Legacy API but at the same time doesn't
belong to EO (see previous commits).

Revert "scroller : fix mirrored contents_pos_set call"

Summary:
This reverts commit de313d6296ac8a2d9a3c0f9762179ffd42a833ff.

Because x-coordination mirroring is called properly from wanted_regison_set.
There was a problem in scroller init process.
Before edje_obj was initialize in elm_interface_scrollable_objects_set, mirrored_set was being called.
So Move mirrored_set after elm_interface_scrollable_objects_set
it can fix this issue(https://phab.enlightenment.org/D4944)

Reviewers: raster, woohyun, SanghyeonLee, akanad, eagleeye, cedric, singh.amitesh, eunue

Subscribers: jpeg, cedric

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

docs: Fix typos and some wrong expressions in Ecore, Ecore_Audio, Ecore_Cocoa, Ecore_Con, and Ector API reference doxygen.

Summary: I had fixed some typos and wrong expressions, such as capital letters, $simbols in .eo and singulars in Ecore, Ecore_Audio, Ecore_Cocoa, Ecore_Con, and Ector API reference doxygen.

Test Plan: Doxygen Revision

Reviewers: Jaehyun_Cho, stefan, jpeg, cedric, raster

Reviewed By: Jaehyun_Cho

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

ephysics: set proper geometry when setting an object to a face

Summary:
It was missing a break statement in a switch.

Fixes T5547

@fix

Reviewers: cedric

Subscribers: jpeg

Maniphest Tasks: T5547

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

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

eina: enable fallthrough warning suppression only on GCC 7 and above.

ecore-drm2: Don't use Atomic functions to set DPMS levels

This was initially an experiment in trying to use Atomic properties to
set dpms on/off, however it does not turn off backlight support when
triggered so it is useless.

Fixes T5462

@fix

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

ecore-drm2: Fix output backlight value type definitions

When we go to set output backlight level we use doubles, so lets just
store these values as doubles in the structure.

@fix

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

ecore-drm2: Add internal functions to get backlight values

Small patch to add an internal function which can be used to retrieve
backlight values on output creation.

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

eina: fix clang compilation with EINA_FALLTHROUGH

btw, using clang 4.0.0, I can't produce fallthrough warnings using
-Weverything, -Wall or -Wimplicit-fallthrough

ecore_drm2: Fix error handling in _output_dpms_atomic_set

This was skipping the error path on failure and setting some state as if
it was successful.  Then the next attempt at a page flip was actually
setting this state.

So _output_dpms_atomic_set (which has always been broken) wasn't actually
the function that successfully disabled dpms.

This is confounding attempts to debug why dpms isn't coming back on
properly.

Now it won't turn *off* either, because it really never should have.

Ref T5462

scroller : fix mirrored contents_pos_set call

Summary:
Fixes that a scroller can not be moved normally even if it is changed to mirrored.
In elm_scroller, _mirrored_set is called when sizing_eval is called. because
When content size is changed, it should be scrolled based on mirrored coordinates.
Also In elm_interface_scrollable,
elm_interface_scrollable_content_pos_set of _elm_scrollable_mirrored_set
to be called regardless of mirrored state.

Test Plan: scroller test on elementary_test.

Reviewers: raster, woohyun, SanghyeonLee, akanad, eagleeye, cedric, singh.amitesh, eunue

Subscribers: cedric, jpeg

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

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

elc_multibuttonentry: role of MBE item, MBE updated, and state_set_get API added.

Summary:
Reason for role change:
MBE items do not  act like push buttons, they need to maintain their status and at a time only one item can be selected, they act like radio buttons.
Role change of MBE from entry to panel, because MBE is just a container and entry is one of the children, when entry gets access frame, it will work as how entry works, but making MBE role as entry does not work as expected and also MBE should not receive highlight as a whole but only its children should receive highlight one more reason why MBE role cannot be entry.
state_set_get API added to get items state.

Test Plan:
When atspi mode is enabled the access frame has to
navigate directly on item and not on MBE as a whole and state has to
be read out

Reviewers: kimcinoo, cedric

Reviewed By: cedric

Subscribers: cedric, govi, rajeshps, jpeg

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

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

ecore_evas: use strbuf instead of PATH_MAX.

elm_config: password mode change should reflect on already running applications.

Summary:
When password mode is changed, the change has to reflect on already running
apps as well. config_sub_apply will not be called for an already running app.

Test Plan:
1. 2 running process
2. Change the config value of password last show in one process
3. Check whether the change has reflected in previous process.

Signed-off By: Shilpa Singh<shilpa.singh@samsung.com>

Reviewers: jpeg, cedric, thiepha

Subscribers: govi, rajeshps

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

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

eldbus: add test suite for eldbus (eldbus_signal_handler*)

Summary:
add tests for next API:
eldbus_signal_handler_add()
eldbus_signal_handler_del()
eldbus_signal_handler_connection_get()
eldbus_signal_handler_interface_get()
eldbus_signal_handler_member_get()
eldbus_signal_handler_path_get()
eldbus_signal_handler_sender_get()
eldbus_signal_handler_ref()
eldbus_signal_handler_unref()
eldbus_signal_handler_free_cb_add()
eldbus_signal_handler_free_cb_del()

Reviewers: cedric, NikaWhite, myoungwoon, raster, artem.popov

Subscribers: jpeg

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

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

elementary: refactor day handling in calendar to just use unsigned char.

edje: clear up left over warning on fallthrough switch case statement.

edje: use strbuf instead of hard coded PATH_MAX.

ecore_evas: reuse code by relying on switch case statement fallthrough for key offset case.

evas: reuse code for colorspace logic by using switch case statement fallthrough.

elementary: reuse code by relying on switch case statement fallthrough.

evas: use switch case statement fallthrough to handle colorspace logic reuse.

evas: TGA does use switch case statement fallthrough to factorize code.

evas: silence warning for SSE3 optimization.

draw: silence warning in a tricky loop/switch statement.

edje: object have small difference and we use switch case fallthrough to reuse code.

eet: silence properly reused code through switch case statement fallthrough.

edje_cc: when first parameter is a string and the second one a range, just reuse code.

emile: SSL state should fallthrough to avoid duplication of logic.

emile: fallthrough is legal here as we handle fallback case when caller doesn't know about GRY8.

eina: add EINA_FALLTHROUGH to disable warning on useful case serie with no break.

GCC has started introducing a detection for series of case in a switch statement
without break for each case. We do use that trick a lot to reduce our code base.
Even if in most case we have documented this so that people using coverity don't
try to fix it. Now with GCC we need to silence it properly to avoid future
problem.

ecore_wl2: silence gcc warning.

Code is correct, but gcc can not follow the branch if and believe it
is worth triggering a warning.

eina: silence gcc warning.

Code is correct and won't lead to a case where size is > 0 and buffer
isn't set, but gcc can't see the relation between both variable.

evas/examples: add intro for evas-event-filter.c example

Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

evas/examples: fix minor typo 'whe'

Reviewers: cedric

Subscribers: jpeg

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

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

examples/evas: add tutorial commentary for evas-init-shutdown

Summary:
This is the most basic of the Evas examples and serves as the starting
point for new Evas users.  Since this targets neophytes, we can afford
to be much more detailed in commentary.

Reviewers: cedric

Subscribers: jpeg

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

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

examples/evas: add a README

Summary:
Categorize the examples by topic, and identify certain examples as
introductory; these can be commented more verbosely than the other
examples.

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

Reviewed By: cedric

Subscribers: jpeg

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

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

eldbus: add test suite for eldbus (eldbus_message*)

Summary:
add tests for next API
eldbus_message_arguments_vget()
eldbus_message_arguments_append()
eldbus_message_error_get()
eldbus_message_from_eina_value()
eldbus_message_iter_struct_like_to_eina_value()
eldbus_message_iter_arguments_vappend()
eldbus_message_iter_arguments_vget()
eldbus_message_signature_get()
eldbus_message_method_return_new()
eldbus_message_error_get()
eldbus_message_iter_container_new()
eldbus_message_iter_fixed_array_append()
eldbus_message_method_return_new()
eldbus_message_error_new()
eldbus_message_iter_del()
eldbus_message_iter_fixed_array_get()
eldbus_hello()

Reviewers: cedric, raster, NikaWhite, myoungwoon, artem.popov

Subscribers: jpeg

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

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

evas: revise grammar for memory allocator error status

Summary:
Wordsmith phrasings to be a bit more concise.  In example code,
distinguish better between the OOM and Low Mem conditions that it
is demonstrating.

Reviewers: cedric

Subscribers: jpeg

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

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

popup: fix unintentional object deletion when change scrollable

Summary:
If popup scrollable change from FALSE to TRUE after content or text is added,
content_area is deleted, so it cause null pointer problem.
main_layout has content_area in its CONTENT_PART, but in scrollable_set API,
tbl set into main_layout's CONTENT_PART, so content_area is deleted.

On the contrary, if scrollable change to FALSE,
content object in content_area is deleted, because tbl set into content_area's CONTENT_PART.

So if some object set into other object,
unset previous object and set it into properly part after that.

A case using item_append should be fixed by other patch.

@fix

Test Plan: Change scrollable repeatedly after adding content or text to popup.

Reviewers: jpeg, singh.amitesh, conr2d, cedric, raster

Reviewed By: cedric

Subscribers: herb

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

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

evas: document the wayland ecore_evas constructors

Reviewers: cedric

Subscribers: jpeg

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

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

ecore_evas: correct error message reporting incorrect engine name

Summary:
_ecore_evas_wl_common_new_internal() creates both wayland_egl and
wayland_shm backed windows, so reporting that the failure was in looking
up "Wayland_Shm" could be misleading.

Also, this routine can be called with any arbitrary string as
engine_name, so including what was received in the error message might
be helpful for diagnosing bugs.

Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

calendar: fix elm_calendar_interval_set() API not working.

Summary:
The API was not working because of an internal logical error.

@fix

Test Plan: Calling elm_calendar_interval_set API in elementary_test.

Reviewers: jpeg, cedric, Hermet, woohyun

Subscribers: cedric, jpeg

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

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

examples: fix efl_net_control_example.c on BSD.

Reviewers: cedric, stefan_schmidt

Subscribers: stefan_schmidt, jpeg

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

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

elput: fix sp. usefull

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

ecore_imf: add ecore_imf_context_mime_type_accept_set API

Summary: Added a new api to send the mime type of entry to IME.

Test Plan: Tested in Tizen device

Reviewers: woohyun, id213sin, jihoon, cedric

Reviewed By: cedric

Subscribers: cedric, jihoon, jpeg

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

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

efl_debugd: don't compile this on macOS

Well, the build is broken again on macOS :(
efl_debugd is full of platform-specific code: network and epoll.
To make it compile (not work as expected, just compile), we would need
to comment out most of efl_debugd's code. So instead of having a
crippled, useless binary, just don't compile it on macOS.

This unbreaks the build for macOS.

eina: attempt to make the code more portable

This fixes the build on macOS.

efl_debugd: enforce void parameter when no arguments are required

Eina_Debug: Fix missing include of signal.h

Elm layout: fix a few objects after move to efl_part

Fixes elm_layout_text_set/get bugs that were introduced in
3eb649b180eb98f7be829de5aed7e8c7f1ceb35a.

Layout/legacy: rename object type

Enlightenment and terminology compilations fail because Elm_Layout is
not known.

Eina_Debug: Fix config.h usage

It was used inside eina_debug.h that is a public header.
Terminology compilation was failing when including its own config.h.

tests:efl.ui.image.zoomable: add a test case for icon APIs

efl.ui.image: fix icon_get API

If icon set is failed, icon_get should return NULL,
not the old icon name.

@fix

tests:efl.ui.image: Add test case for icon api

This test case shows a bug in icon get API. If icon set is failed,
icon_get should return NULL, not the old icon name.

Merge Eina Debug feature

Eina Debug is a layer aimed to bring a way to debug EFL applications by
providing a transport channel between a debug tool and the applications
of a device.

In order to be interrupted by EFL core as less as possible, the communication
is done in a separated thread.
The Ecore loop is not used there, as well as the Ecore helpers (socket...).

Debugging operations can be registered easily by any layer of the application.

To use it:
- Launch efl_debugd
- Launch the application to debug
- Launch Clouseau (the new version will be pushed soon) or another debug tool
supporting Eina Debug

Fix Windows compilation

Thanks to vtorri for his help on this delicate issue named Windows

Set Eina Debug APIs as Beta

Support endianness

Support opcodes registration for Windows

A standard static array with symbols whose addresses are only known at
runtime is not supported in Windows.