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.

ecore-drm2: Fix issue of VT-Switching going blank on returns

As it turns out, when we vt-switch Back to a running Enlightenment,
it ends up processing all outputs and causes Evas to create a giant
canvas to cover all outputs. This new canvas causes a blank screen as
the pageflipping fails (due to fb being too large). As we do not yet
support output on more an one screen anyway, temporarily disable any
output which is not marked as primary.

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

ecore-drm2: Add API for vt-switching

This commit adds a function we can call from inside Enlightenment in
order to fix an issue with VT-Switching. The problem was when a client
application is open, the compositor does not pass along the key-combo
for vt-switch but rather sends it to an application. This patch (along
with upcoming Enlightenment one) fixes the issue.

NB: This late API add approved by Stefan ;)

@feature

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

elementary: Reduce variable usage

As we don't use the framespace x, y here then there is really no need
to have them.

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

elementary: Fix issue of wrong minimum size for some E dialogs

When we are calculating minimum size hints for the window object, we
should be adding framespace to maximum size, not subtracting it...

Honestly this just looks like a brain-dead typo.

Fixes T44225

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

edje_edit: avoid such values like 2.77556e-17

using spinner to change double values means that sometimes
if set such weird values while should go to 0.0

@fix

elementary: Avoid possibly setting opaque region twice

Previously, if the window had no frame object, then this could would
have been setting opaque region twice (for the alpha case). We can
avoid this if we move the alpha check to below the frame check.

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

elm: Fix extra mouse,in events in layout_content_unset

Problem: crash in assert() in terminology.

Scenario:
 Open Terminology,
 Split V by keyboard,
 Move mouse to split 2,
 Create tab by keyboard
 --> abort() in terminology

Cause:
 An extra mouse,in event happens during edje_object_unswallow
 inside elm_layout_content_unset.

Root cause:
 efl_part() in elm_layout had a side effect: edje_recalc on the
 edje object. Causing its geometry to be "properly" defined and
 the mouse event to trigger callbacks.

Solution:
 Avoid calling recalc... somehow.

Conclusion:
 Without adding any new API, edje edit provides internally the
 information that we want: type of an edje part (for box & table).

Fixes T4221
See T4028
See T3509

efl - ecore-x - work around e's badness in makeing an x image for all wins

so e is being stupid and creating an ecore-x image forevery single
window if in x11 mode if it needs it or not. this results in having ti
allocate an actual x image and shm segments. work around this and get
bit order from somewhere else than the x image itself thus avoiding
the allocation until a real get or put is done.

@fix

elput: Use correct filename in unlink

We switched from mkstemp to eina_file_mkstemp() but ended up unlinking
the template filename, which didn't exist on the filesystem.

wayland_shm: Use correct filename in unlink

We switched from mkstemp to eina_file_mkstemp() but ended up unlinking
the template filename, which didn't exist on the filesystem.

ecore-wl2: Add safety check for valid compositor

As we cannot create a new wl_region without a valid compositor, add a
safety check.

@fix

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

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

ecore-wl2: Remove error message during connect/reconnect

Since session recovery can cause apps to constantly try and reconnect
to the display server, we probably should not be filling logs here. On
the other hand, any failures on first connect are now not going to
display any error messages.

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

edje_pick: copy block 'data'

Copy global data from files. Main resoan for it - elementary.
Elementary needs data "version" for load widget styles.

@fix

edje: remove now unused variables

Commit e3ed888aa994d96007da15be89649a92c12bcbae removed the code where these
variables have been used before.

examples: edje: make sure we handle the various corner cases for the po files

Delete the earlier copied files only. Handles out of tree and in tree builds.

edje: Remove invalid call to recalc in efl_part

What a mess... Assuming efl_part() had no side effect on the
object it took me hours to figure out that there was a wrong
call to _edje_recalc_do in the efl_part() function itself.
That was bad, and existed because efl_part() used to be
efl_content_get().

efl_part() should not have any side effect.

Also, fix a return value in content_remove.

Fixes T4214 (invalid redraw and crash in terminology).

elm cnp windows - remove unused vars