genlist: Fix rare jump in prepend insert

If an item is selected, and another item is insert before
the selected item, then we try to lock the genlist view (pan)
around the selected item (if it's visible). Unfortunately,
every 16 inserts cause a jump by one line in the scroller.
That's because the scroll math assumes the block position is
known, but since it's a new block it wasn't known.

This patch fixes this issue by precalculating the block position.

Test scenario:
 elementary_test -to "Genlist Tree, Insert Relative"

Select an item, clikck 50 times on "+ before".
The view should not jump.

This does not fix fileselector's craziness.

@fix

genlist: (Mostly) fix item index odd/even styles

This fixes the internal item order index.

Note that groups don't reset the odd/even styles. The
original code wasn't very clear on the intent (setting
to 0 in one case, not increasing the counter in another,
but that was not consistent all over the place). I believe
resetting the odd/even styles at a group boundary would
look great, but this might be for another patch :)

This amends part of another commit, but keeps its feature:
 b40a6eb85bf44a genlist: implement list position signals.

See T3086

PS: I've discovered more odd/even issues with the
    fileselector in particular. Still working on it...

@fix

Revert "elm_genlist: when appending items to the parent, prepend to the parent"

This reverts commit 43d82e567a2d655a089b6ca3f2d913e6ec52f1dc.

I don't understand this commit. It broke the logical order of
items, as the internal list wouldn't match the order on screen.
Other places in the code didn't seem to make this assumption
that parents come after their children. And for sure my recent
fixes require the parent to come before.

This commit was one of the many reasons why odd/even styles
look often wrong.

See T3086

genlist: Remove macro GL_IT()

It was used to hide "it->item"... but was used less than it->item
itself. Explicit code here is not longer, and just as readable.
This macro I think was harmful to readability.

Simple sed, no real change at all.

evas gl common - fix nv12 text upload with upack row length

we segv'd because our testure width was wrong... i was row span
length in bytes and we use luminance+alpha texture... this fixes it.

@fix

ecore: do not access 'timer_thread' global from animator thread

@fix

ecore-evas: use ee->prop.window in ecore_evas_input_event_register()

this allows window events to match correctly

@fix

ecore-evas: handle pointer_warp() for buffer canvas

this was missing and would do nothing when called

ecore-evas: handle pointer_xy_get() for buffer canvas

this was missing and would do nothing when called

ecore-evas: handle screen_geom_get() for buffer canvas

this was missing and would do nothing when called

docgen: fix broken inheritance graphs

docgen: more useful event listing

docgen: disregard override info for displaying property get/set

docgen: minor speed optimizations and cleanups

docgen: merge overridden and other categories for methods/properties

elementary: Fix compilation for SDL

Not sure how long this has been broken, but the variable name changed
in this routine to "is_gl_accel"..."is_hw_accel" is no longer used, so
change variable name here to fix compilation with SDL.

@fix

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

docgen: initial support for inherited events listing

docgen: add "Others" section with all remaining callables to class pages

docgen: merge methods and properties in listing

docgen: properly handle overridden property docs

elm: Restore ABI compatibility (elm_pan_gravity)

elm_pan_gravity_{set,get} are functions that were generated as
legacy APIs (in other words EAPI), but were never actually exposed
to applications as they were protected behind EFL_EO_API_SUPPORT
(see elm_interfaces.h).

This patch restores the ABI compatibility with elementary 1.18.

emotion XXX - make nv12 709 streams work even with slightly off colors

ther eis no 709 nv12 support support in evas but there is 601, so use
that for now until we add the feature of 709 nv12 support.

efl-js: Fix deprecated Set feature of setting non-primitive value

efl_js: Export efl.Loop

efl_js: Use efl_add_ref for saner refcount.

efl_add was messing up the refcount in objects with parents on the JS
side.

js: Update examples, including window api changes.

Window now inherits from Container instead of Pack.

Also removed some deprecated stuff.

js: Raise exception to js instead of crashing

eolian_js: Avoid errors when getting function scope

efl_js: Update register functions.

js: Change test eos to ptr()

docgen: add a way to get numerical id of a function

docgen: more useful function listings in class pages

Now there are C signatures visible.

genlist: Simplify code (remove one argument)

Parameter "qadd" is always true when calling _item_process_post(),
so simplify the code by removing it.

genlist: Fix more tree issues (expanded state)

This fixes a lot of cases where a genlist node (of any type)
with children may have appeared in the invalid expanded or
contracted state.

Before this patch, the test case "Genlist tree, Relative insert"
looked like below (all items are programmatically added):

> A
  > 1
  > 2
> B
  > 3
  > 4

The problem above is that A and B have visible children but
still believe they are in contracted state. This patch ensures
that A and B will be marked as expanded, but will do so without
firing an "expanded" signal and definitely without the
"expand,request" signal.

After this patch, the test case will look like this:

v A
  > 1
  > 2
v B
  > 3
  > 4

Which is more correct. Note that this test case does not handle
any expand/contract signal.

NOTE: This is a behaviour break!

genlist: Fix insert after with a tree

This fixes a case where inserting item "C" after item "B" in this
tree would go wrong:

A
B
- 1
- 2

Before this patch, 1 and 2 lose their parent:

A
B
C
- 1
- 2

After this patch, 1 and 2 retain their parent:

A
B
- 1
- 2
C

Insert before worked by luck, no need to fix it.
Note that this patch may require the next one to actually
work (ensuring expanded state flag).

NOTE: This is a behaviour break!

genlist: Fix sorted_insert with tree

This fixes the test case "Genlist Tree, Insert Sorted".
This is a pretty ugly patch... but the genlist code is already
pretty ugly, as it keeps a flat inlist of items (sd->items)
as well as a tree structure in parallel.

Before this patch, the following configuration led to issues:

 1
 3
 - A
 - B

Adding item "2" led to a crash. Adding item 4 led to this:

 1
 3
 4
 - A
 - B

Items A and B lost their parent "3". Subsequent sorted inserts
would lead to insane bahaviour, where for instance "8" would
appear before "3".

This patch fixes all sorted inserts, at the cost of performance
(an optimized code path is avoided). Subsequent patches will
increase the robustness of the tree structure.

NOTE: This is a behaviour break!

Fixes T4850

edje_edit: fix scripts compilation

@fix

docgen: inherit overridden method/property brief descriptions on class pages

slider: theme - fix warnings

Fixes warnings related to  "disabled_visible" states
not present in "limits" part.

fixes T5123

elementary gengrid: fix for working item reorder mode correctly

Summary:
Change the item indexing to start 1 after reordering animation.
Change the logic of edge checking to get the row or col correctly.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Test Plan:
elementary_test -to gengrid2
append 6 items
enable the reorder mode
check reordering is working properly (4 to 1)

@fix

Reviewers: singh.amitesh, cedric

Subscribers: jehun.lim, jpeg

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

dayselector: Fix DaySelector widget

An invalid strcmp() led to always reject the swallowed
widgets, which were Elm.Check instead of Elm_Check.
Use efl_isa() instead.

Fixes T5034

edje: Fix a major issue with recalc loop

With the new Efl.Ui.Clock widget, which implements Datetime,
most parts are relative to each other, and trigger a near-inifinite
recalc of all the parts. Indeed edje was recalculating a part even
if it had already calculated it (flag is set).

This seemingly simple patch fixes Datetime and probably a lot
of other edje widgets.

Fixes T4909

@fix

evas: Optimize out some nop calls to evas text

Some property set calls to Evas Text are NOPs if the property was
already set to the same value. Avoid doing any work then.

This adds a missing intercept call (size_set as it's overriden
by Evas.Text).

elm: Make clock widget more resilient to invalid config

I was in a situation where my elm_config had the proper version
but did not contain a proper module name for the clock widget.
This resulted in not being able to create any field object in
the clock widget... in other words it made for a very useless
clock :)

Ideally the config should be updated at this point. I'm not
sure how to proceed exactly, though :(

evas: Fix module memleak during evas_shutdown

This fixes a minor memory leak during shutdown.

Note: This does NOT perform the dlclose (and still leaks the
Eina_Module descriptor). Calling dlclose leads to a whole lot
of other issues, so we avoid it.

evas: Fix make check (evas_suite compilation)

This fixes issues where Evas_Object_Intercept_Cb_Type was not
defined, as this is defined in Evas_Legacy.h (unfortunately...
as it's used by elementary), but the private headers defining
EFL_CANVAS_OBJECT_PROTECTED were included after Evas_Legacy.h.

evas: Fix compilation warning (unused variable)

evas: Avoid data_scope_get in internal function

evas: Inline part of clip_dirty

clip_dirty is called A LOT. Unfortunately this patch does not
result in very measurable improvements.

**********************
Note about this merge:

After this series of patches, the CPU usage for a certain test
case has significantly gone down:

Based on c0e6a8d698c17fc16f9b67fc9 (upstream before git push):
NS since frame 2 = 28910806786 , 2937 frames = 9843652 / frame

After this patch:
NS since frame 2 = 19218592951 , 2928 frames = 6563727 / frame

1.18:
NS since 2 = 13105584220 , 2961 frames = 4426066 / frame

As we can see, 1.18 remains *much* better than 1.19. I'm still
struggling trying to figure out why (clip_recalc is called more,
but the call tree is hard to decypher).

The test case is:
EINA_FREEQ_BYPASS=1 ELM_TEST_AUTOBOUNCE=100 \
  elementary_test -to "Scroller 2"

EFL was compiled with GCC 6.3.1 with -O3 -g

@optimization

evas: Inline part of clip_recalc

This function was moved out of inline (see d7c6fca6c00a0bfb05) but
unfortunately the early checks at its beginning are likely
to result in an early return. Inline this part so we get back a
better performance. Inlining the whole function does not improve
the performance, as GCC simply gives up with inlining.

Note: Between 1.18 and master the number of calls to clip_recalc
has simply blown out. It is thus crucial to find out where those
calls come from but also micro-optimize the function itself. This
patch does the latter only.

@optimize

evas: Optimize out some calls to data_scope_get

Within evas when we know an object is marked as is_smart, then
we know it's a smart object. :)

@optimize

evas: Avoid more calls to efl_isa in clip_set

No need to verify that the clip is an evas object unless we're
actually going to change said clipper. This has a minor effect,
removing ~10% of all calls to efl_isa (0.1% of everything).

After all those efl_isa patches the number of calls has dropped
from 1.9M (~3.05%) to ~570k (0.76%). More can still be done,
surely. Now efl_isa accounts for ~0.05% of my CPU usage according
to perf top (down from ~0.7%).

If edje uses EO APIs instead of legacy image APIs, then many of
the remaining calls will vanish.

@optimization

evas: Assume object is valid and avoid call to efl_isa

In an internal function we should be able to safely assume an
object is indeed a smart object.

@optimization

evas: Optimize out more calls to efl_isa

In relative smart move, we only need to check once that the object
is indeed a valid smart object. This drops the call count from ~1.02M
to ~785k.

@optimization

evas: Avoid call to EAPI inside internal code

This avoids calling efl_isa and locking the async mutex.

In callgrind analysis, this reduces the count of calls to
efl_isa from 1.96M to 1.02M.

@optimization

evas: Avoid useless calls to eo_data_scope_get

Use an internal variant of the function for evas when we already
have the protected data pointer.

eo: Make _eo_obj_pointer_done an inline function

@optimization

evas: Remove useless call to evas_object_async_block

It's already called from the intercept cb.

evas: Inline checks for post-event cb call

This avoid excessive calls to a real function.

@optimization

docs: Fix typos and some wrong expressions in Eina API reference doxygen.

Summary: I had fixed some typos and some wrong expressions, such as capital letters, singular, and orders of groups in Eina API reference doxygen.

Test Plan: Doxygen Revision

Reviewers: stefan, cedric, raster, Jaehyun_Cho, jpeg

Reviewed By: jpeg

Subscribers: conr2d

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

examples: Use printf instead of fprintf(stdout, ...)

Summary:
Also fix warning about lack of a format string:

    warning: format not a string literal and no format arguments [-Wformat-security]
         fprintf(stdout, commands);

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

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

release: Update NEWS and bump version for 1.19.0-beta2 release

eina utils: fix build without getpwent.

If HAVE_GETPWENT isn't defined - the closing brace was missed.
Also prevent situation when strdup() tried to duplicate NULL
pointer, because that could cause segfault.

@fix

interface_scrollable: Improve gravity_set api to support pan changing

Summary:
When you set gravity 1 on scroller, scroller sticks to the bottom
even content is changed.
however, scroller don't work like above, if size of pan is changed.

this commit uses pan_pos_max rather than w/h of content_info
because pan_pos_max is related with both content_size and pan size.

gravity_set will work properly even if both size of content and pan are
changed simultaneously.

Test Plan:
1. Select 'scroll3' in the elementary_test
2. Append enough items so that scroll bar appears (about 30 items)
3. Go to the bottom and Set gravity 1.0
4. Check that scroller sticks to the bottom once you append another item
   (it works)
5. Check that scroller sticks to to bottom once you resize window(pan)
   (it doesn't work without this patch)

Reviewers: eagleeye, jpeg, cedric, woohyun, z-wony, herdsman

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

evas: do not error for smart class objects

Summary: this should fix some spamming in e

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: cedric

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

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

ecore_evas_wayland_common: fix bug that window is not rotated when app request rotate.

In landscape mode, width & height of evas are different with ecore's.
so diff is checked differently according to the  portrait & landscape mode.

elm_test: Add autobounce test for "Scroller 2"

Canvas text: fix clearing existing style

Fixes T5187.
This fixes a leak in evas_textblock_style_user_pop, that made use of the
new key-based efl_canvas_text_style_set api.

@fix

elm_code: Another Coverity fix, better this time.

@fix CID1371322

readme - systemd is default on now... --disable if you dont want it

elm code - fix warning about sign comparisons

elm_code: Fix Coverity issues.

@fix CID1356612 CID1368335 CID1368489 CID1371123 CID1371124 CID1371126 CID1371128

elm config base profiles - fixselect allow to be orrectly iniitted

fixes previous commit...

fix entry selection by having proper cfg update and def fields

this should fix T5183

eo: ensure the generation is correctly clamped

Summary:
when a few recursive event emissions are happening, and in some deep
recursive level a subscription to the same object is happening, the
subscription would just be executed when the complete recursion is done.
that is wrong. The subscription needs to be executed when the event is
called after the subscription is added, undepended from any recursive
level. That fixes that and adds a regression test for it.

This was discovered in e, since e gives a lot of error messages about a eo object
that is already freed. It turned out this object is returned from evas, and exactly
the above happened to the EFL_EVENT_DEL subscription of that object.

Test Plan: make check

Reviewers: tasn, cedric, stefan_schmidt

Subscribers: stefan_schmidt, netstar, zmike, raster, jpeg

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

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

doxygen: fix typos and some wrong expressions in Edje and Eet API reference documentation.

Summary: I had fixed some typos and some wrong expressions, such as capital letters, singular, and orders of groups in Edje and Eet API reference doxygen.

Test Plan: Doxygen Revision

Reviewers: stefan, cedric, raster, Jaehyun_Cho, jpeg

Subscribers: conr2d

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

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

docgen: add -p to print what exactly is being generated currently

docgen: make sure all class retrievals are cached

docgen: cache function instantiations

docgen: cache class retrievals for performance

gengrid: Remove duplicated smart callbacks

WRN... evas_smart.c:219 evas_smart_cb_descriptions_fix() duplicated
  smart callback description with name 'item,focused' and type ''

widget: Reduce WRN to DBG for on_focus_region message

Many widgets don't implement this function, and this case is
perfectly well handled by the caller. No need to WRN here.

evas: Silence useless WRN message on canvas del

ecore_evas: Fix mouse event in inlined windows

This fixes the following ERR message:
ERR<10589>:eina_safety /home/jpeg/e/core/efl/src/lib/ecore_evas/ecore_evas.c:3149
  _ecore_evas_mouse_move_process_internal() safety check failed: cursor == NULL

win: Avoid magic check failed in ecore with inline win

Fixes T4663

elm_test: Fix invalid API use (EO)

win: Avoid malloc in icon_object_set

Also support both Evas.Image and EO Efl.Canvas.Image classes.
Add a test case in elm_test (under "Icon").

I'm not so happy about this patch... it shows that the API
barrier between legacy and EO implemented for images may not
be such a great idea after all :(

evas: Flush post-event callbacks only when needed

This avoids flushing the post-event cb queue when processing
recursive events. See 54e5841b2f6386765dd5e2831aa3481b952e5.

win: Fix window icons with object set

Fixes T4981

evas: Re-fix post-event cb by supporting recursion

The previous patch (b184874fa5a0c3177c) was preventing
post-event callbacks from triggering any form of input event,
including side-effects due to mouse,in. In fact by tracking
which exact events we want to post-process we can support
proper recursion. This fixes crashes in Bryce.

I'm not changing the documentation as this is still a dubious
code design.

Fixes T3144
Fixes T5157

evas: Strengthen post-event callbacks

See T3144 that I marked as Wontfix.

Bryce in E manually feeds events from a post-event callback
resulting in Evas going insane and leading to frequent crashes.
The ideal solution (for E) would be to ensure that everything works
smoothly, the input event data is valid up until the post-event cb
is called, etc... Unfortunately, with recursive events the exact
order of operations may be messed up: the post-event

I don't want to add yet more complexity to Evas events here (it's
already spaghetti all over the place) so I'm simply blocking any
new event feed when running the post-event callback list.

It's not possible to just freeze the events (I tried, it failed).

**********************
Some more explanation:

post-event callbacks are used to implement reverse-order logic
where the on-hold flag of an input event may be set by an event
listener that does not come first.

Here's a situation to illustrate: scroller A inside scroller B.

As events are propagated from children to parents (assuming the
propagate flag is set), we'd assume the events to go first to A
and then to B, which means a mouse wheel event would make the
inner-most scroller (A) scroll, and the outer-most scroller (B)
wouldn't budge.

But as things are designed, A and B are not simple evas objects,
and the actual event-catching object is a top-most transparent
rectangle (top-most in Z stack order). Since A is inside B, B's
rectangle BR is over A's rectangle AR, thus catches the wheel
event first. But in terms of UX we still want A to scroll, not B.

The solution then is to reverse the event processing order and
post-event callbacks are the way to do that. This comes with the
consequence that the event_info needs to remain valid until the
post-event is called, and stay the same (so that the on-hold flag
set by A can be read by B).

Recursive events (by explicit feed or modifying the canvas so
that mouse,in or mouse,out are triggered) mess with this logic,
and trigger the post-events too early (event is not fully
processed) or too late (event_info is not valid anymore... and
crash!).

Thanks @raster for explaining the goal of post-event callbacks!

evas: Ensure post-event cb push is called from an event cb

This rejects calls to evas_post_event_callback_push() that don't
originate from inside an input event callback.

evas/doc: Add warning about post_event_callback_push

Revert "Revert "evas textblock: keep previous size when the calculation is skipped""

This reverts commit b8beb6834b01f9688cb5e7aa177bacbc713318b9.

this now actually works... for some mysterious reason... ? :/ i am
baffled. go back in until we can find the issue then...

doxygen: fix typos and some wrong expressions in API reference.

Summary: I had fixed some typos and some wrong expressions, such as capital letters, singular, and orders of groups in API reference doxygen.

Test Plan: Doxygen revision

Reviewers: stefan, cedric, raster, Jaehyun_Cho, jpeg

Subscribers: conr2d

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

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

efl: fix memory leak in edje and evas.

Summary:
Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>
Signed-off-by: Ayush Srivastava <ayush.sriv@samsung.com>
Test Plan: EFL apps

Reviewers: cedric, raster

Subscribers: rajeshps, ayush.sriv, jpeg, prince.dubey

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

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

ecore_drm2: ERR once when page flip fails with EBUSY

On systems where this happens it'll probably happen a lot, so
we don't want to continuously log this, but since it's definitely
showing a bug somewhere (efl or kernel) it probably should be an ERR.

edje: Fix object leak with Efl.Part

A call to efl_reuse was missing.

evas_loader: add since tag to newly added task_cancelled API

eina_strbuf / eina_ustrbuf: add since tags for newly added APIs

These six have been added in this cycle but missed the since tags.