elementary_test: Search menu performance optimization.

Summary:
elementary_test search menu is created again and again
at each search query. Here the optimaization is to remember
the menu item and retain them instead of deletion and reuse them.

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Test Plan:
search item in elementary_test and measure the
performace time.

Reviewers: jpeg

Subscribers: cedric

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

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

ecore_input_evas: Enable cancel events by default

After my input event changes, the propagation path has been altered
from:
 ecore_x -> evas_event
to:
 ecore_x -> ecore_input_evas -> evas_event

But ecore_input_evas was ignoring cancel events by default. There
should be no good reason to disable cancel events anymore,
according to @jypark. Also, this fixes an actual bug :)

Fixes T4301

evas filters: Fix minor issue with print

If the call to tostring() fails, replace the string by (invalid)
rather than ignore the error.

Fixes CID 1308612

edje_cc: Remove unused variable 'got_hash'

got_hash will always be zero at the point where it's read.

Fixes CID 1261439

examples: elm: enable +evas3d_scene_on_button_example again

This was disabled at some point but works fien in my testing so enable it
again. If we no longer want it we should remove it completely.

build: examples: make sure we build all examples in list

At some point we lost the \ after track_example_01 and in combination with the
commented out line right afterwards we lost all examples coming after this line.

This just surfaced with JPs latest commit when I did a distcheck build but was
there for a longer time. We now make sure all disabled examples are moved out of
the multi-line list.

eina: add includes for getpid() to work on MacOSX

Without this I get errors from the clang compiler used on Travis for some OSX
builds:
../src/lib/eina/eina_inline_lock_posix.x:845:27: error: use of undeclared identifier 'getpid'

Elm.Entry: force cursor change on size_eval signal

This is a follow-up to ba060fe93236da37e4c8bbd84c1bcf58e27979d6.
Fixes T4274.

elua: load modules from local dirs first

This fixes cases when running scripts locally - local modules
are preferred over systemwide, avoiding possibly outdated system
scripts from being run.

eina: safepointer: add since tags to new eina safepointer EAPI

Canvas.text: fix segfault with ellipsis cases

The specific handling for (0.0 <= ellip < 1) doesn't support multi-line cases.
One of the main reasons is that we haven't had the chance to define the wanted
behavior for multi-line.
This is a temporary hack to fix a segfault. The behavior is still undefined,
though.

Fixes T3885.

examples: Fix button theme

A program was not handling the various states properly.

elm filesel - handle api that might provide path too big for buffer

this addresses a path that could be too small (unlikely due to max
path generally being 4k), for a stack buffer. this should make
coverity happy. fix CID 1356634

eina_module - check result of module load from an array of modules

we don't give any debug or info if a module in a list fails to load.
we should at least offer debug info. do that. this fixes coverity
issue CID 1039687

examples: Fix path to edj theme files (theme examples)

A bit complex but should work in most cases (installed & not)

examples: Compile edj files in elementary examples

ecore gl module - dont check value that is already valid

make coverity happy with CID 1347410 - it's technically right that the
code assumes the im ptr is valid until then as it dereferences it
until then and then stores it under pd->image but then checks for
NULL.. it would have crashed already if it were, so rmemove the
pointless check.

elm init - handle return value of ecore_file_init make coverity happy

this isn't an issue as init will never fail, but makes coverity happy
with CID 1353589

elm dbus menu - handle return value of eldbus_message_iter_arguments_get

handle possible error return of eldbus_message_iter_arguments_get()
according to coverity CID 1353591. this should fix that.

elm app client - check return of eldbus_message_iter_arguments_get

as accoding to coverity CID 1353593 chekc returns of the above
function and return/continue as apporpriate if these fail (which
generallly they shouldnt unless some end of the dbus client is broken).

efl vpath - make sure we check returns of mkdir just in case for errors

so vpath has a fallback if all things bad go wrong to mkdir a homedir
in /tmp and if that fails use /tmp or use / - if $HOME isnt set ... but
$HOME alwasy should be, so this is only for "a broken system".

this should fix CID 1354286

edje entry - handle insanely low lemory if list_append fails

this fixes coverity CID 1357752

elm_need_efret() - improve error handling according to coverity

fix CID 1360466

we don't actually chekc if efreet inits right with elm_need_efreet()
as we just assume it inits (And there pretty much is no case where it
will not), but make coverity happy and check.

@fix

emotion modules -0 check return value of eina_module_load and print err

this should inform the user via eina log errors that a module load
failed. this fixes CID 1360954

evas: Mark native_surface as const

The void* argument should be considered read-only.
This doesn't change anything, really.

elm_map: fix wrong check of source search

Summary:
When map source is set by elm_map_source_set(),
it foreaches enlisted sources, and should return
when the source cannot be found.

@fix

Test Plan:
Call elm_map_source_set(map, [any type], [Wrong source name]);
         Observe  ERR("source name (%s) is not found", source_name) message.

Reviewers: Hermet, cedric, jpeg

Reviewed By: jpeg

Subscribers: conr2d

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

efl - edje entry - dont emit changed on markup set unless text changed

if the text didnt actually change we generat a lot of noise in changed
signals for no change at all in text. shortcut this and check the new
and old text and compare ptrs, 0 length and strings etc.

this fixes T4045

@fix

evas cserve2 client - treat an invalid size header as fatal

this should make coverity CID 1361430 happy

ecore_cocoa: remove pointless debug message

This message was also very annoying because systematically
printed out in the syslog when a cocoa window was closed.

Edje_Edit.h: undef EAPI at the end of the header file

Fixes T4303

Revert "elementary: keep theme consistent regarding the disabling of odd/even."
Revert "elementary: disable visual hint at odd/even."

This reverts commit c64071ae132ddc5894d5daad8a874377a176aac5.
This reverts commit 7affe8c2047a41a2945cf6c048d59f28c974aa7a.

This reverds the odd/even theme changes because this BREAKS
ENLIGHTENMENT COMPLETELY. Enlightement also uses odd/even list looks
and REMOVING this REMOVES every 2nd lost item member (it makes them
0x0 sized). This is a major break in actual FUNCTIONALITY in the name
of looks. NO. Revert.

We ship with this look. no more breaking stuff like this in the name
of cosemtics and not ACTUALLY fixing any bug at all. it creates bugs.

Evas font: fix size query again

Some wrong variables were used in the rewrite at
8c6effae8ee027a928bcee79968a0b21d8250487, basically reverting the original fix.

edje_cc - fix inheritance of programs and store channel too in copy

this fixes programs that inherit and play audio on specific channels.
i found that sound worked on single line entries but not multipline
beause single line inherit from multi but didnt copy this program
param - the channel number. this fixes this

@fix

elm config - dont rescale ui if no config that needs rescale changed

also finger size, and other thigns that could affect ui look that may
need elm rescale to work.

this fixes T4296

@fix

evas cserver client tool - handle possible < 0 sized return

this is cserver so its trusted but handle possible < 0 size msg.
address coverity CID1361243

pdf loader - check return for rmdir

fixes CID 1361218

eina log - handle possible overflow from high mem (last few bytes) or env

fixes CID 1361219
if you had an env var a few gb in size or we had stuff in the last bit
of memory address space this might be an issue, but that won't happen.

@fix

eina log - keep coverity happy with potential ptr comparison overflow

fixes CID 1361220

in theory yes end minus start could be insanely huge or end be very
high in memory thus causing an overflow. this would have to be in the
last few bytes of memory space, so it never going to happen. and the
input from the env var has to be sane anyway as its user controlled.

@fix

embryo_cc - fix coverity complaint about assert

fix assert using a strance assert syntax and go back to basic asserts.
fix coverity CID 1361224

@fix

Eio: Update my email address.

eina threadqueue - for paranoia lock+ulock+free in order locks taken

so mimic order of locks taken to be paranoid ... this wouldnt be a
deadlock issue but just to be safe instead of sorry.

edje: mark new edje_mmap_3d_has() API with since tag

ecore_wl2: mark new events with since tag

evas: add missing since tag for new evas_object_type_get() API

ecore: properly track events in Efl.Loop.Fd object.

Thanks Gustavo.

Revert "elementary/layout: attach edje object API with eo compositing"

This reverts commit 8a988717e18c4e9e6d4a0752441471a124056878.

It seems we can't actually inherit from class more than once and neither eo
nor eolian will complain about it. You will just get random weird behavior.
This patch should come back once we have made an interface of edje.

Simple test case to spot out a bad bug in elm.Entry

The bug is explained in T4296, basically the "changed" signal in elm.Entry is called randomly.

eina thread queue - ensure blocks cannot be freed until lock+unlock

just in case... check blocks cannot be freed until locked and unlocked
just in case someone still is holding a lock.

README: clarify poppler dependency

We actually need poppler-cpp which is packaged in various ways in different
distros so better give both names here.

README: update build dependencies and build instructions

Remove part where we recommend the now merged loaders and players and update
build dependencies as well as recommended options.

ecore imf test - remove unused variable

ecore eina thread queue test - be anal about thread exiting cleanly

ecore-con - fix complaint about null slave list due to shutdown

so ecore-con already cleared the slaves list on shutdown but an
ecore_thread may stillbe active and thus removing from a NULL slaves
list wont work, so skip if slaves is NULL.

@fix

eina lock semaphores - do not use shared semaphores and fix osx names

the way eina does sempahores, they can NEVER be sensibly shared
cross-process portably. so enabling sharing is a pointless idea. in
fact some os's like openbsd check if the sempahore addres is in a
sharable mem region and deny init if it is not. on osx you dont use
shared memory but a sempahore name you share instead... and this is
not exposed thus it can't be shared either. if we did process sharable
semaphores we'd make shm segments and/or name them in a sharable way
were you can share the idenitifer of the shm segment and/or the offset
address or name from osx. but we don't, so making them
process-private is the right thing. sharable sempahores will need a
whole new api.

this also fixes osx naming to make the name pretty unguessable/private
and opened exclusive (or it fails) by using pid, sem counter, and 4
random numbers. it's not a security mechanism as the create will fail
if there is a clash. chances are low. we unlink before anyway. good
enough for osx for now.

@fix

eina thread queue/semaphores - check even more unlikely errors + complain

check some really unlikely alloc errors and semaphore deadlock errors
and report if that has happened. just in case.

eina thread queue - check returns fo semaphore new/lock/release to debug

so openbsd may have issues with semaphores - maybe. this would catch
them if it it. why - if sem_init/sem_wait are different... i dont know.

ecore con eet test - dont use localhost port 8080

so thats a common local port to use for other things... 8080 - don't
use it, so use 12346 much like 12345 is for the other tests. far less
likely to clash. fix up tests to work better.

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.