eo: fix warning.

eo: adjust test suite to take invalidate state into account.

ecore: rely on Efl.Object.invalidate instead of a hack for Efl.Model.Container.Item.

eo: introduce invalidate and noref state.

This is just a first step. All user of destructor should be updated to
move the code that rely on their efl_parent and on efl_provider_find to
invalidate. Then we will be able to change the way efl_add and efl_del
work to properly refcount things. efl_noref won't be triggered at the
moment until both efl_parent_set(obj, NULL) and the last user ref are
set to NULL. This is not what we want, but due to how user refcount is
accounting parent at the moment, until all the code is move to rely
on invalidate we can not fix this.

ecore_wl2: cleanup input code to string logic to reduce compiler warning and useless memory access.

elementary: fix backward compatibility bug introduced by 9c8749b99a03d3601321da6d16071dd7b631d1ae.

ecore-wl2: Check for valid string before passing to strlen()

As strlen() cannot accept NULL (segfaults), we should check for valid
key, keyname, and compose strings here before passing to strlen().

@fix

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

ecore-wl2: Reduce calls to strlen

Small patch to reduce calls to strlen when sending key events. This
patch is loosely based on Phab D5567

@fix

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

eolian: get rid of old APIs for typedecl retrieval

eolian: remove old variable retrieval APIs

eolian: remove old class retrieval APIs

efl_ui_focus_manager_calc: performe refocus after the node is deleted

if node is the last element, the element will be focused again, and
later deleted, without cleaning up the history

examples: cxx: fix the compilation for slider example.

efl check - fix env var for CK_FORK=no

elementary: entry - recover cursor position when theme is changed

Summary:
When theme is applied text is reset and cursor position is set to 0.
This patch is recover the position to previous after text is reset.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Reviewers: cedric, jpeg, woohyun, herdsman

Reviewed By: cedric

Subscribers: cedric, jpeg

Tags: #efl

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

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

elementary: bg - keep file path and key string for legacy bg widget

Summary:
If a file path and key string was passed to elm_bg, we could get
the same file path and key string. Even if it failed to load the image file.
And the file path also remained its original form.
ex) Setting file path "~/image.png" => Getting file path "~/image.png"
                                       (Not "/home/user_name/image.png")

@fix

Test Plan: Included in elementary test suite.

Reviewers: jpeg, cedric, raster

Reviewed By: cedric

Subscribers: woohyun

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

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

elementary: entry - fix to generate "activated" and "aborted" event in single line entry only

Summary:
"aborted": The escape key was pressed on a single line entry. (since 1.7)
"activated": The enter key was pressed on a single line entry.

Reviewers: id213sin, jpeg, cedric, herdsman

Subscribers: cedric, jpeg

Tags: #efl

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

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

elementary: add missed type for legacy.

Summary:
We've been missed type of class name.

Add legacy class name to maintain compatibility.

@fix

Reviewers: jpeg, woohyun, Jaehyun_Cho

Subscribers: cedric, jpeg

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

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

elemetary: spinner - free the longpress_timer when unfocused

Summary:
It needs to delete when widget unfocused.

@minkyu
Gets from tizen.
0bc1a19703dc204c8d347794bae0290006f4bde5

Reviewers: jpeg, Jaehyun_Cho, minkyu, cedric

Subscribers: minkyu, cedric, jpeg

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

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

elm: fix location of legacy include for elm_code.

Summary:
Instead of including the legacy header on the main header, just include
it where it is actually used (diff_widget.c), like in elm_code_widget.c.

Test Plan: Try to compile edi with the tarball from `make dist`. It should not fail with elm_code related errors.

Reviewers: raster, stefan_schmidt, felipealmeida

Subscribers: cedric

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

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

efl_ui_multibuttonentry: maintainance legacy function type.

Summary:
I did it wrong.

The "Elm_Multibuttonentry_Format_Cb" should be maintain for legacy.

efl_ui_XXX has to use efl_format interface.

@fix

Test Plan: elementary_test->multibuttonentry sample.

Reviewers: cedric, woohyun, Jaehyun

Subscribers: cedric

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

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

evas: increase offset by 4 to do work for next map points

Summary:
Increasing offset as 2 for next map points is wrong.
If evas tries to draw for wrong combination of map points,
it can cause wrong results. Actually, every drawing code for
map points use and increase offset as 4.

@fix

Test Plan:
A test case for textpach is modified for testing this issue.
1. Run elementary_test with sync render mode.
   ex) ECORE_EVAS_FORCE_SYNC_RENDER=1 elementary_test
2. Open textpath test.
3. Set a short text by clicking newly added check box.
4. (It will show another issues... So,) change slice number to update textpath properly.
5. See some noises at top-left side of text.
   It is drawn from the two of end map points to the two of empty(not used) map points.

Reviewers: raster, cedric, jpeg, jypark

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

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

eina: tighten grammar in eina_hash API docs.

Summary:
Improve the verbage in the doxygen comments.  Refer to the value being
changed as the 'previous' rather than the 'old' value, to be more
precise.  Drop the phrase 'keeping API sane' as it is unnecessary and
is an odd thing to say.  Try to avoid referring to 'your program' as we
shouldn't assume the reader's situation.

Reviewers: cedric

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

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

eina: define the in/out params for eina_bezier

Reviewers: cedric

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

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

eina: use better parameter name in eina_rectangle

Summary:
'thiz' is not commonly used in EFL, more commonly used is a word or
abbreviation that is descriptive of the object being used ('hash' for
Eina_Hashes, 'str' for Eina_Strings, 'array' for Eina_Arrays, etc.)
Follow this convention by using 'rect' (as used already in various
places) instead of 'thiz' or 'r'.

Reviewers: cedric

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

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

eina: specify error returns in @return for eina_list API docs

Summary:
For all routines that can return NULL on error, mention this in the
function's @return docs.  In cases where a small number of situations
result in this return, move the docs to the @return; in other cases just
state the NULL return briefly and leave the elaboration in the body.

Reviewers: cedric

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

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

efl_wl: Don't create touch resources with pointer interfaces

Fixes a forced disconnect error.

ecore - go back to args 0 being the command and 1+ being actual args

so this is then inconsistent with efl.exe and efl.thread, so go back
to being normal with 0'th arg being the binary itself jsut to make
sure we have conistent usage.

ecore - efl threads - add thredio class

this adds a simple indata and outdata void ptr to begin that you can
set on efl.thread objects (set the indata) and get the outdata too to
get results. then on the efl.appthread side the indata is set on the
efl.appthread before it runs and on quit the thresad can set the
outdata on the appthread, and this appears back on the efl.thread
object in the parent thread.

so you can basically share pointers to anything in and out this way on
start/exit in addition to string args etc.

the reason i made it an extra class (mixin actually) is for future
expansion. sharing more complex data - eina values maybe or objects as
long as they are shared objects, and perhaps acting as an interface
for calling a function at the other end like ecore_thread_async_call
etc.

ecore efl exe/task/thread - move stdin/out flags to task class

also use them in efl thread to determine if io handlers are set up and
pipes created at all etc.

Efl.Ui.Slider: implement Efl.Ui.Range.range_step_set/get()

Also move elm_slider_step_set/get() as legacy APIs.
remove eo api step.set/get() as well since range intf
already implements it.

evas cache - fix return val to fix warning

fix return val if cache is null (something bad happening)

evas cache image - protect against null cache ptrs

ok. i can't find the root cause because all i have is a backtrace from
ApBBB and he says he can't reproduce it and i know im->cache is
null... if i could reproduce ... i'd be hunting the root cause. but
the best i can do is check for null im->cvache ptrs and be safe.
crashes are bad. especially for end users.

Elm_code test: fix compilation

Reviewers: cedric, raster

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

efreet_xml: make sure we unmap the right pointer.

Seems Linux would munmap a lump of coal without failing. Make
sure the pointers match. Again bogus unmap not detected by
valgrind and not failing.

@fix T5949

efl loop - remove commented out code left over from work on theads etc

efl.task - add an api to clear environment

efl exe - fix indent

efl loop promises - cleare out promise data to null

so there is something broken in the complect efl promise/loop promise
that the clear of promises on loop destroy is clearing
promises/futures that have already triggered (loop timer ones). i've
spent enough time figuring out that it is happening.
_efl_loop_timeout_del() simple doenst ensure the future in
pending_futures for that promise is removed from the list. getting the
future from the promise handle is an exercise in pain... so i'm not
continuing with that path and will just ignore it.

but for now filling the promise data with null at least means if the
menory is re-used after free it wont see garbage freed ptrs and get
nulls so its easier to track.

ecore thread queue tests - reduce size and craziness and fix 1

1 test was wrong. it didn't wait for the thread to exit before checking
msg count recieved. fixed. race condition here.

also reduce the sheer message counts sent - it makes the suite take a
lot longer than is sane and als consume massive amounts of log space
in /tmp as a result.

ecore file test - change url to a file we control.

rthe ecore file download test was downloading from sf.net ... and i
noticed sf.net refusing thus the ecore tests suite failing... i
changes it to grab a file (rss.php which is disabled for us but there)
so at least enlightenment.org has it.

better would be to spawn a webserver and test against that locally.
but thats a whole other level of work.

tests - eo - found eo_signals test is wrong... amazing it passed before

so it was listening for cb adds and dels... and or del of any cb
except the cb add/del catcher was done.. it would fail...

but ... the test actually added other cbs than this ... like:

   efl_event_callback_array_priority_add(obj, _eo_signals_callbacks(),
-100, (void *) 1);

a while array of cb's:

{ EV_A_CHANGED, _eo_signals_a_changed_cb },
{ EV_A_CHANGED, _eo_signals_a_changed_cb2 },
{ EV_A_CHANGED, _eo_signals_a_changed_never },
{ EFL_EVENT_DEL, _eo_signals_efl_del_cb });

none of which were _eo_signals_cb_added_deled. i am amazed it passed
before...

now switch its checks to check for itself and then check for anything
BUT itself...

put efl app test back with mods to match app as superclass

ecore - a different take on efl.app class as a super class to efl.loop

so the MAIN loop is actually an efl.app object. which inherits from
efl.loop. the idea is that other loops in threads will not be efl.app
objects. thread on the creator side return an efl.thread object.
inside the thread, like the mainloop, there is now an efl.appthread
object that is for all non-main-loop threads.

every thread (main loop or child) when it spawns a thread is the
parent. there are i/o pipes from parnet to child and back. so parents
are generally expected to, if they want to talk to child thread, so
use the efl.io interfaces on efl.thread, and the main loop's elf.app
class allows you to talk to stdio back to the parent process like the
efl.appthread does the same using the efl.io interfaces to talk to its
parent app or appthread. it's symmetrical

no tests here - sure. i have been holding off on tests until things
settle. that's why i haven't done them yet. those will come back in a
subsequent commit

for really quick examples on using this see:

https://phab.enlightenment.org/F2983118
https://phab.enlightenment.org/F2983142

they are just my test code for this.

Please see this design document:

https://phab.enlightenment.org/w/efl-loops-threads/

Revert "cxx: Fix manual code after efl_app change."
This reverts commit 135154303bea691c6f7f9472a5dec32d9103c38d.

Revert "efl: move signal events from efl.loop to efl.app"
This reverts commit 3dbca39f98288580c62a43c179ac11621433ec88.

Revert "efl: add test suite for efl_app"
This reverts commit 3e94be5d73256a7f5c02d3a9474173226be7beff.

Revert "efl: create Efl.App class, the parent of Efl.Loop"
This reverts commit 28fe00b94e55575c15684959b89a614d5a579309.

Go back to before efl.app because I think this should be done with
superclassing here not a parent object. reasons?

1. multiple loops per single thread make no sense. so if multilpe loop
objects they wont be contained in a single app object and then deleted
like this.
2. the app object is not really sharable in this design so it cant be
accessed from other threads
3. it makes it harder to get the main loop or app object (well 2 func
calls one calling the other and more typing. it is longer to type and
more work where it is not necessary, and again it can't work from
other threads unless we go duplicating efl.app per thread and then
what is the point of splittyign out the signal events from efl.loop
then?)

etc.

eolian cxx,csharp: Update after eolian changes

Closes D5829.

elua: add missing APIs to eolian bindings

elua: objectify unit and redo class retrieval

eolian: remove old APIs for path retrieval

elua: update path retrieval APIs

eolian: remove old directory_scan/file_parse APIs

elua: remove old dir scan/parse APIs

elua: add new API prototypes in eolian bindings

eolian*: replace various directory_scan/file_parse

eolian: replace directory_scan and file_parse in tests

Pyolian: new APIs for typedecl lookups

Pyolian: new variable lookup APIs

with adjusted tests

Pyolian: new APIs for class retrieval

Also updated tests, generator and gendoc accordly

efl file interface - fix reyturn if no file set to return a null file

eolian: new APIs for typedecl lookups

eolian: new variable lookup APIs

eolian: new APIs for class retrieval

evas: Check for NEON via eina_cpu_features if possible

On linux we can do this test without firing a SIGILL and trapping it,
if getauxval() is present.

ref T6711

eet: add doc note for EET_DATA_DESCRIPTOR_ADD_VAR_ARRAY_STRING

elm config: hide theme and profile config in gui when running in enlightenment

put some small effort into preventing the user from destroying their config

elm: fix config upgrades for user profiles

loading the system profile only works if the current profile has the
same name as a system profile

theme: add fallback matching when referenced theme is not found

if done properly, this should never occur, but at least find some
layout to use if one is available

theme: introduce theme overlay/extension matching

after this commit, efl base themes should now specify:

data.item: "efl_theme_base" "theme_name";

and overlays/extensions which match a given theme should use:

data.item: "efl_theme_match" "theme_name";

this will cause overlays and extensions with the data.item to only
be loaded when the corresponding theme is in use. note that this
should not be specified for theme-independent overlays/extensions
as it will completely block loading of themes

elm_theme: deduplicate theme apply code

the "default" style fallback code here was identical, so just call again
with "default" instead of copy and pasting the same code

elm_theme: massively simplify internals

instead of maintaining separate lists for the file and the edje file,
maintain a single list of structs containing both of these

also dynamically manage a string list of files to preserve compat with
existing (bad) functions which return this directly

elm: set default theme name internally when applying config

for whatever reason this is only generated in elm_theme_get(), so call
that whenever doing theme string parsing

@fix

edje: add ability to reference images from other edje files

this uses the just-added "id" property to allow referencing images
by name from that theme. example:

=FILE1=
id: "myfile";
images.image: "someimage.png" COMP;

=FILE2=
requires: "myfile";
images.image: "someimage.png" EXTERNAL "myfile";

FILE2 will now load someimage.png from FILE1 at runtime if FILE1 is
currently opened in edje, and FILE1 will be kept open until FILE2 is
closed

@feature

edje_cc: add "id" toplevel property

this can be used by edje files to identify themselves

edje: reformat bin/ c files

a lot of this was unreadable due to mixed tabs/spaces or just random
formatting

edje: EDJE_IMAGE_SOURCE_TYPE_EXTERNAL -> EDJE_IMAGE_SOURCE_TYPE_USER

no functional changes, just a confusing define rename

Revert "wayland_imf: Fix bug in shutdown"

This reverts commit fb01a697dd099f17a6a30c74781b6cfeb72bc512.

The problem this commit was anticipating never happened.

Now the next version of wayland will allow us to make protocol symbols
private, so it makes more sense to have this stuff back where it was
in the first place.

Pyolian: add APIs to retrieve units from a state

Also fixed a declaration error from previous commit

Pyolian: add API to get file name of a unit

Pyolian: add API to get children of a unit

Pyolian: reorder the header file

No functional changes, just reordered the declarations

Revert "elm: make elm_view_list legacy"

This reverts commit d2d63a8c239cf40b7b828303cca34a0d0dde97ec.

This broke compilation of efl applications, since elm_view_list.eo.h
is required, however, this again leaves the installed .eo files broken.

eina/bezier: use FLT_EQ marcro for float equal comparison.

Reviewers: SanghyeonLee, jpeg, cedric

Subscribers: cedric

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

ecore_wl2: Disable horizontal resize optimization

We had a "clever" optimization that would keep a buffer on resize
if it was resizing up horizontal and fit within the previously
allocated stride.

Unfortunately, there still needs to be a buffer reconfigure between
client and compositor that wasn't taking place.  Remove this for now.

Pyolian: rename Eolian to Eolian_State

Pyolian: eolian_file_parse -> eolian_state_file_parse

Pyolian: rename path/filename retrieval APIs

Pyolian: rename all_files_parse functions

Pyolian: new APIs for directory scanning

eolian_directory_scan => eolian_state_directory_add
eolian_system_directory_scan => eolian_state_system_directory_add

eio_file: fix some indentation.

Make this easier to read.

eolian: rename path/filename retrieval APIs

eolian: rename all_files_parse functions

eolian: eolian_file_parse -> eolian_state_file_parse

eolian: add API to get file name of a unit

eolian: add API to get children of a unit

eolian: add APIs to retrieve units from a state

eolian: new APIs for directory scanning

eolian: eolian_* -> eolian_state_*, Eolian -> Eolian_State

evas: minor formatting fix

NB: No functional changes

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

ecore-drm2: Add API function to retrieve output user data

With the rework of Ecore_Evas drm engine, we need to get the
ecore_evas itself in the pageflip callback so we have to reassign
output->user_data to the ecore_evas which we can then use to retrieve
via this function

This is needed as the pageflip callback will pass us the output on
which the pageflip completed.

@feature

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

ecore-drm2: Free stringshare for relative output during output_free

When we destroy a given output, we should be freeing the
eina_stringshare for relative.to

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