atspi: toolbar - fix cannot call select callback
There is a line to check select_mode in _item_select function.
The item selected(clicked) callback is called when the select_mode is ELM_OBJECT_SELECT_MODE_NONE.
The checking select_mode should be handled at one place.

ecore eina threadqueue test suite - be correct about volatile+padding

fill in the padding of mesages (10 bytes) with something so valgrinds
can be happy and use vlatiole for msgs count as the msgs num should
have been incremented already before the msg sned is done and main
thread/loop gets the msg

eina_threadqueue - fix const ptr warning on spinlocks if atomic disabled

atomics SHOULDNT be disabled anyway, but if they are... fix this
warning. (no real bug)

ecore_imf - fix make check to doa  "load all" check after de-sillifying it

this should make ecore_imf testable with empty env vars also meaning
no env var and the make check test will now ensuree this is set to
exactly test this.

ecore_imf: Fix crash when ECORE_IMF_MODULE is set

evas: Make safety check more lenient

This is to avoid ERR messages such as "object is not an image!"
that can happen when a legacy image function is called on an
eo image (proxy, 3d, ...) object.

Note: This does not fix any known issue, but it's probably a safer
solution.

efl: Cleanup some code (return values)

This removes some useless code in various places, where the
switch from eo_do() to standard function call was not properly
refactored.

This changes:

type ret = 0;
ret = my_eo_function();
return ret;

To:

return my_eo_function();

evas image: Lower ERR message to WRN and fix it

In elm_image, efl_file_get() was called on a legacy Evas Image
object, which generated an ERR message. This was way too much
noise for something that actually works.

It's best not to call efl_file_get/set API on a legacy
object, but it can be convenient for smooth code transitions
from legacy to pure eo.

Also, add safety to those APIs.

Test case: elm_test genlist dnd, start dragging anything.

evas: Mark image fill_spread as deprecated

It was never implemented.

evas: Add safety checks to evas image legacy APIs

This checks for NULL and ensures the type is evas image.

docs: make dark theme default, add switch --graph-theme-light

docs: add default dark theme (via --graph-theme-dark) in docgen

efreet: free fallback cache on shutdown

this fixes errors like:
ERR<4864>:eet lib/eet/eet_lib.c:645 eet_shutdown() File '/home/osauser/.cache/efreet/icons___efreet_fallback_localhost.localdomain.eet' is still open 1 times !

@fix

elm tooltips - fix positioning cornercases

this should fix T4277

mailmap: update mailmap file now that we have all libs merged

Trying to unify the authors a bit here. And yes, Chris wins the price for the
ridiculous amount of variations he uses.

Elm.Entry: use proper theme_set function on edje object

Shouldn't use elm_widget_theme_object_set for handlers/anchors here.

Fixes T4091.
@fix

edje object shutdown - be far more paranoid about ptrs and order

so we didnt set everything to null after being freed and object sae
freed after some data. do the data frees after objects are deleted so
callbacks cant access null data anyway. this makes this edje shutdown
far more robust and safe. massive improvement in stability i hope.

edje - fix access to freed data struct during object shutdown

this fixes T4181

elementary widget: actually this is a right fix than fa4a72d3fcb6a96f6291a0717036b28277c78413

Now rage and enventor both will be fine.

elm key events/disabling/focus - fix after another disabled widget fix

this fixes key events in rage again after
ea2b5e40485a49b5c5aadae98ed379f1c3cf5f71 broke them. this fixes T4285

efl: fix a neon build break.

We found a some build break problem due to a build system.

In efl configuration, our efl configure enables neon if it's available,
but neon gonna partially turned off on evas compilation if the build system
has a non neon CFLAG option (ie, mfpu=vfpv..). In the end, neon is partially enabled
and it turns out a build break.

It's non-sense but we can't allow this breakage.

There is no way to ignore build system option nor override our neon option perfectly,
so, Let efl package ruled by build system overall.

edje: updated the start index of vector resource from 1 to 0

Reviewers: cedric, jpeg

Subscribers: cedric, jpeg

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

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

elementary: keep theme consistent regarding the disabling of odd/even.

This is another patch that should be reverted as soon as we start
7affe8c2047a41a2945cf6c048d59f28c974aa7a and is still a work around
T3086.

elm image - silence warnings of setting smooth if image is edje

emotion generic - vlc - make it off by default as 2 video bends is silly

as per bug. requiring by DEFAULT 2 video back ends is rather
rediculous. pick one and the others are optional. you could argue xine
now is barely worth it, but it's stable and works and has proven to be
not any real pain.

gst1 is a more optimal and far more complete/featured backend that the
vlc generic one anyway. vlc generic has the bonus of out-of-process,
but gst1 is pretty stable (unlike the 0.1 backend), so let's not make
peolpe install 2 video backends (or possibly 3 or more) just to build
efl out of the box.

this fixes T4284

Fix further text checks in Elm_Code T4264

    An incorrect assumption in how unused memory may be initialised
    @fix

emotion: fix autodetection of emotion generic vlc player module.

T4273

elm: scrollable: add guards to include eo and legacy header only when allowed

Make sure the eo and legacy headers are only included when the matching defines
are enabled.

elm: fileselector: add guards to include eo and legacy header only when allowed

Make sure the eo and legacy headers are only included when the matching defines
are enabled.

evas: Fix compiling evas x engine with no gles

This fixes an issue from 19eb7b727fbf35620a13fb65b50d3 where the code
would no longer compile due to not finding noext_glXCreatePixmap in
the structure. This was due to the fact that this field was not
compiled into the structure if gl_gles was defined.

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

elm focus default theme: fix wrong resizing

When focus is moved rapidly, final size of focus highlight is not
matched well with the focused object.
Without running anim in edc, it shows right performance.

@fix

elm image - store smooth scale var in object as intended

this fixesa bug reported by davemds and kuuko in python bindings test
suite where setting smooth flag != getting it right after.

widget: Fix propagation of events on hold

Because of the translation between legacy and eo key
events, the on_hold flag state could sometimes be lost.

Symptom:
 elementary_test -to Focus
 Click on a button inside the scroller
 Up/down would jump to the end of the scroller, rather than
 the next button.

Thanks a lot to Woohyeon for the very good bug report.

glx: Fix black windows in E on nvidia

This fixes calls to glXCreatePixmap that would consistently
fail on nvidia >= 360.

It seems glXCreatePixmapEXT was used instead of glXCreatePixmap,
and that function always returned 0.

One could assume always using the non-EXT version of the
function should be preferred. Unfortunately, doing so for all
the other functions brings back the black windows.

I'm taking a very careful approach by doing this only for drivers
>= 360.

Fixes T3030

@fix

gl_x11: Minor code refactor around eglCreateImage

Trying to debug T3030... hitting a brick wall at the moment.
Add better error handling (for EGL).

elm theme - don't use empty env vars eina_prefix handles anyway

in 14ec825aa9ed9c6ede79d1729fdb291285b3bd6d cedric added trying
ELM_DATA_DIR env var explicitly for looking for themes. first if this
is NULL then we look in a bogus directory (null). secondly eina_prefix
already takes care of env vars like thsi as part of figuring out
prefix with correct namespace etc. so this just is a bug as opposed to
anything useful and the CORRECt implementation is already done in
eina_prefix.

@fix

edje: fix color class broken by b543f4c896d9.

T4270

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

eina: test safepointer in the correct range for 32bits system.

The generation counter on 32bits system is on 8bits, obviously 1024
is more than that.

T4265

Fix text check in Elm_Code T4264

An incorrect assumption in how unused memory may be initialised
@fix

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

tests: eolian_cxx: add define to allow the use of beta APIs like eo

This is needed again now as we put eo back into beta for 1.18.

ecore_audio: use the correct module wrapper

This was code for sndfile. sndfile module should have been
used instead of the pulseaudio one.
It led to a build break when having sndfile but not pulseaudio.

Ref: 879d93377b219dca00be63f8139fee79b7938147

ecore_drm2: Fix a condition where no outputs are enabled

If the first output we test is disconnected but has the crtc of
the primary display assigned then we'll fail to assign the crtc
to the connector it's really connected to later.

This is a quick hack as trying to sort this out properly may be
too invasive for the upcoming 1.18 and isn't really important until
1.19 supports multihead anyway.

docs: put graph before hierarchy

docs: allow rank dir and size to be changed in theme

Eo: Move Eo back into beta.

Eo should now be considered beta again. This commit puts it back in beta
and makes it necessary to define EFL_BETA_API_SUPPORT before including
Eo.h.

docs: support for foldable inheritance graphs

docs: add theme loader for graphs

edje_edit: delete swallowed object before unswallowing (for part type GROUP)

unswallowing set swallowed_object to NULL inside of
_edje_real_part_swallow_clear
so it is impossible to delete object anymore.
And so, changing sources leaves fantoms and undeleted objects.

@fix

elementary widget: fix a wrong disabled behavior.

This is a corner case bug I spontaneously found.

* Scenario.

A. Disable A widget.
B. Add a child B widget to A.
C. Now B Widget theme will be followed to A that is performed by
   elm_widget_theme_apply()
D. This elm_widget_theme_apply() calls elm_widget_disabled_set() (originally.)
E. Now B widget will be logically disabled.
D. Let's enable A widget again.
E. After going through widget disabled sequence, elm_widget_disabled_eval()
   will be called in the last
F. In this function, A widget tries to enable its children. But B widget won't
    be enabled because its logically disabled!

Acutally, nowhere widget change children's disabled states logically,
but it propagates its state to children within volatile way so that
A widget perfectly keeps the disabled/enabled state with its children and
recover the children's enable/disable state once their relationship is cut off.

@fix

elm scroller accel was broken by default confgis setting it to 0. fix

now set default wheel scroll accel to 7 as epr the confgi upgrade code
and also improve the acceleration to "rely on itself" and feedback
loop. this is so much better now i can scroll through things in a
whisk with the wheel.

fixes T4263

@fix

tests: ecore_con: make sure info is initialized

One test does init info before passing it to _url_compl_cb while two others
are not. Also test if inof is NULL in callback here to avoid crashes.

tests/ecore_con/ecore_con_test_ecore_con_url.c:194:4: warning: 'info' may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_event_handler_add(ECORE_CON_EVENT_URL_COMPLETE,

tests: eio: add Eina.h include for eina_init and shutdown functions.

tests/eio/eio_suite.c:33:4: warning: implicit declaration of function 'eina_init' [-Wimplicit-function-declaration]
     eina_init();
     ^
tests/eio/eio_suite.c:38:4: warning: implicit declaration of function 'eina_shutdown' [-Wimplicit-function-declaration]
    eina_shutdown();

build: do not dist now longer available files

Another cleanup for commit dd1d3f0d2d8f7369f7461f54928eac2a4fce99fb. These
makefiles are now longer available better not try to dist them.

build remove now unused eolian subbuild helper makefile

In commit dd1d3f0d2d8f7369f7461f54928eac2a4fce99fb the consumers of the makefile
have been removed. No need to still have it around.

efl runtime dynamic lib loading - make errors consistent and useful

so drop trying to appease the openbsd packages and stick to "upstream
so major versions" and let users fix their systems with symlinks. also
report what we are looking for so they have a chance to symlink to
make efl happy.

at some point we should make a single simple runtime lib linker
subsystem in efl so all these errors are reported in the same way,
input libray names are listed in a simple consistent way etc. etc.

for now we have 3 locations in efl that do this and they are roughly
similar. we can unify it later.

autotools: since it has been broken for some times and nobody noticed, let's remove per directory support.

elementary: remove *.eo.h files from includesub_HEADERS

Otherwise *.eo.h files will be installed twice, so it can break a
parallel install on fast machines, whereby two make jobs may run
concurently, trying to install the same files.

With elementary 1.17.1:
/usr/bin/install -c -m 644 elm_sys_notify.eo.h
/usr/bin/install -c -m 644 elm_sys_notify.eo.h
 usr/bin/install: cannot create regular file 'elm_sys_notify.eo.h': File exists

Fixed in Buildroot:
https://git.busybox.net/buildroot/commit/?id=bba82cbfe4caaa01366fc74fb3b6f4c41587a28b

While applying the fix on latest master (1.18.0-beta1), the only
remaining *.eo.h file is efl_ui_text.eo.h.

Fixes:
http://autobuild.buildroot.net/results/7e9/7e9caffe442dcf40ee41da56fd3f55ffedf3a291

Signed-off-by: Romain Naour <romain.naour@gmail.com>

edje_cc build - link to lua and use ula cflags as edje_cc uses lua

i do not remember it using lua... well that's obvious then - add
cflags/libs.

ecore-imf - fix stupidity of loading ALL modules even if not needed

eocre-imf loaded all input modules even though an env var was set to
tell it to use a specific one. this just wastes memory and slows down
startup time. this uses the env var to choose ro auto-guesses wayland
or xim input if the right env vars are set, and if it doesnt know
falls back to "load all" as before. this saves 28kb of dirty pages.
this helps address T4227

@fix

edje - reduce memory footrpint by dlopening ephysics on demand

this saves about another 80Kb or so in dirty pages by only loading
ephysics when needed. This removed ephysics and bullet library dirty
pages from the process space. this is another patch to address T4227.

@fix

ecore_audio - save 232k of real RAM by dlopening depednecies

so libpuls and libsndfile suck in dependencies. they suck in so much
that by the time linking is done we've written to about 230kb of
PRIVATE MEMORY as dirty pages in symbol tablesm global veriables etc.
etc. - this is just horrible. especially if an app never makes any
sound... it's just wasted memory. this stuff is invisible to normal
memory debug tools. so this begins to address things. please see
T4227. my numbers now put me at:

1780Kb total dirty writable mapped from library file pages. down from
2012Kb.

This fixes some memory bloat reported in the above ticket, but there
is more to fix for sure.

@fix

edje: throw error when vector resource not found.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: Fix handling of polygon and polyline node.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

tests: Place inline window inside a box

This shows off how to properly use an inline window.
I tried to add extra smarts inside elementary to handle those
inline windows like normal objects... but it led to more bugs.

The only way that makes sense to use this inline window
is to separately treat the image object and the window
itself. Any other approach is too tricky.

See T1973

ecore_evas: Register animators for inline image ee

In ecore_evas_object_image_new, register animator tick del/add
catchers, so that animators can work. This fixes mouse scroll
in an inline window.

scrollable: Avoid useless connect to animator

If scrolling Y no need to register X animators. And vice
versa. It's a minor optimization.

Canvas text: fix range geometry calculation for RTL

Fixes T3627.

@fix

data for elm - remove misleading and unused Makefile.am

eo-mode.el: add @nonull and void_ptr.

these were not in the wiki (also fixed), thus I've missed them.

ecore imf - ibus - fix destroy/disconnect handling that crashes

restart e or have ibus input get destroyed and come back... the ibus
module will do bad things to its pants... because the wrong things
were passed to the signal callbacks... this fixers that along with
ensuring a freed string ptr is NULL.

@fix

elua: add table serializer/deserializer

This is important for docgen use (reading theme configs).

tests: ecore: try to keep constness for Efl_Version

It returns as const so better try to keep this be having the avriable const as
well.

tests: eina: comment out unused test to avoid warning

Just to avoid the unused function warning.

eina: promise: fix memory leak in error path

If we would not get a promise back here the allocated array would leak when we
return. Make sure we also free it in this case.

CID: 1360468

ecore: xcb: make sure length_name is always initialized

lib/ecore_x/xcb/ecore_xcb_icccm.c: In function ‘ecore_x_icccm_name_class_set’:
lib/ecore_x/xcb/ecore_xcb_icccm.c:320:11: warning: ‘length_name’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
         s += length_name + 1;

Looking at the code this is indeed possible so better play safe here.

evas: xcb: fix build break of xcb engine

This was broken since commit ed2e78286b351cd8c3e85e1f6b92f25a54fab027
The variable is named differently in the xlib vs. xcb code

elm: object_item: add guards to include eo and legacy header only when allowed

Make sure the eo and legacy headers are only included when the matching defines
are enabled.

elm: sys_notify: add guards to inlcude eo and legacy header only when allowed

Make sure the eo and legacy headers arte onl yincluded when the matching defines
are enabled.

eldbus: make sure eldbus model is hidden behind the BETA API flag

Make sure we keep this in beta for now until our EO based interfaces are ready.

multibuttonentry: Fix wrong definitions.

Summary:
There is no "ELM_MULTIBUTTONENTRY_EVENT_CHANGED"
Because of this, MBE's Internal entry did not get a changed callback.

And MBE does not have changed smart callback.

Those things made by someone during EO interface working i think.

Test Plan: elementary_test

Reviewers: Hermet, cedric, jpeg

Subscribers: cedric, jpeg

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

eo-mode.el for emacs.

This major mode provides colored output, that helps a lot to view and
edit ".eo" and ".eot" files.

It's the first version, I still see some indenting issues with
toplevel blocks such as struct and enums. Nonetheless it's much more
useful than fundamental-mode (pure text).

eina: add back option lost at some point for debugging.

eio: add a test for T3237.

I have been trying to reproduce the reported ephoto issue, but couldn't. This
test case reproduce the reported scenario.

eio: all our tests need eina and we aren't testing eina here, so init once and for all.

efl - elm genlist - clean up sanghyeon's patch to be less code

use if not while, no first, and timing checkis already done at end so
remove from if/while. cleaner now.

fix warnings in sanhyeons's patch/fix

[CONFIG] call elm_rescale() only when scale value is changed

Summary: it doesn't need to call elm_rescale() in _config_flush_get() when the scale value is not changed.

Reviewers: jaehwan, cedric, raster

Subscribers: minkyu

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

genlist: fix memory-leak in filter queue and refactoring queue iteration

Summary:
There are critical memory-leak in filter queue process.
         It looks typo so I fix it and refactoring queue iteration logic to spend
 less time in for and while loop.

Test Plan: Tested by filter elementary test

Reviewers: shashank0990

Reviewed By: shashank0990

Subscribers: raster, cedric, jpeg

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

Fix memory leak issue on genlist_example_02

Summary: The realized items list should be freed by either eina_list_free() or EINA_LIST_FREE when it is no longer needed

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: minkyu

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

evas generic gst loader - handle fwrite return value for coverity

fixes CID 1360467

evas canvas destruction - detect zombie objs and hack at them with axes

ok. so here's the issue at least now. we have eo objects in the canvas
and they have a refcount of 2 user_refcount is 0. the calls stack does
NOT show we are calling callbacks at that time on these objects. they
are not in the backtrace (the canvas is, the objects themselves are
not).

SOMETHING is keeping 2 eo "internal" refs on these objects and i have
no idea what/how/who. it's a royal pain in the butt to find out as the
only way is lots and lots of logging and you get drowned in the
logging...

so what I have now done is a super ugly workaround that detects these
zombie objects that refuse to die and just FORCES them to die when the
evas canvas frees and clears out layers.
ac10a00acc6bacf01bfd208cc78b4eb2a6a925d8 doesn't really cause the
issue, it just brings it out in the open for all to see far more
easily. but something is deeply wrong SOMEWHERE with SOME objects and
our refcounts.

this fixes T4187

docs: support for writing edge styles for individual connections

docs: initial bits for theming support for graphs

tests: Add missing function implementations

In eolian test cases.
Not a problem when EOAPI is defined as weak, but compilation
will fail if the symbols are strong. Eg. on Solaris (it's just
a guess).

Fixes T3889

entry: allow force recalc on "size,eval" "elm" signal

Summary:
In some use cases, font (style) of elm.text in elm entry is changed
on focus and unfocus. user must send "size,eval",elm signal to entry to
force recalc, but this was not happening. This patch fixes that issue.

fixes T4233

Reviewers: tasn, herdsman, raster

Subscribers: cedric, seoz, jpeg

Maniphest Tasks: T4233

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

win: Fix evas framespace under X

Mouse inputs would not work if the window is rotated,
because the framespace was invalid.

Fixes T4132

edje entry - check all minor struct callocs and on fail report + unwind

lots of unchecked malloc/calloc returns, so check these, unwind safely
and complain (highly unlikely to happen unless the system is about to
fall over). this should fix T4230

elementary: fix legacy path set/get for Fileselector specializations

Fix legacy functions elm_fileselector_path_get and
elm_fileselector_path_set.
Make these functions call the correct function when called with a
specialization of Elm.Fileselector (instead of the base class function).

Create internal versions of path set/get functions to avoid warnings
about deprecated function calls.

Fix T4198

@fix

ecore-drm2: Silence compiler warning

We probably can't actually use this variable uninitialized, but
let's init it anyway to keep gcc happy.