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

elm cnp win32 - do one side at least of unix newlines to windows newln

this PARTLY addresses T3556 ... this handles our own conversion of
makrup to string then to a windows newlined string. this half. i wrote
the conversion code for the other way but it's unused as i am unsure
where exactly to plug in it. following the cnp code makes me not sure
where it goes so for now - not there, but ready to go.

to be clear. out API is unix text. utf8 strings at our api with UNIX
newlines. that is our api. that is the text we accept and produce. if
you deal with another file or interface that does not provide this
then the job of conversion is AT THAT POINT. eg elm_cnp.c has to do
this. as would file loads of text files (and saves) etc. - anything
else like forgivingly handling anything at the api level makes it
totally unclear what our api is and what should go in and come out.

to be portable we have to define what it is and the most portable
thing to do is at the api level within a process we define one and
only one format. UNIX \n format.

@fix - partial

efl ui image - fix view size get to NOT apply scaling

this fixes T3254

edje: fixed issue with applying transformation and stroke width

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje examples - when you cp also just assume success if same file

fix makefiles for examples.... more

@fix

example/edje: updated vg example with interpolation usecase

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: fixed issue with vector part state inheritance.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

example/evas: update evas vg example.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

efl/interface: fixed the shape interpolation implementation.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: Minor fix (return proper bool value)

edje: Use eo_del instead of eo_unref where needed

This fixes the following error message:
Object '0x40000002f0000030' still has a parent at the time of destruction.

Though I actually wonder why ref == 0 and there was a parent.
Something is still a bit fishy here.

edje examples - stop deleting the po files from the src tree!

whatever someones was trying to do.. they just endewd up deleting po
files in the src tree every time you clean! argh! fix!

evas software x11 - be agressive with shm cache and keep it low

shm cache size is now 1.5x the total number of pixels in sw x11
outbutf's and that's it. 32 segment max limit too. this should keep
things down

@opt

evas and ecore_x shm segment management - fix over allocation and perms

so our sysv shm segments were both over-permissive (nothing bad
really, just other users could read and write to/from our pixel data
destined for the screen... they could do this to x11 directly anyway
so no real issue), but be more restrictive and use 0600 as xserver
runs as root so can read/write anyway and we only want our own uid
access. but even more - fix our shm segment flushing to not keep lots
of segments floating about like a bad smell when we don't need them.
we had a cache but it wasnt flushed when it should be since async
rendering turned up. this fixes that and we're back to agressively
flushing them out when idle.

@fix

Revert "access: add an example of elm_object_part_access_object_get"

This reverts commit f8aa67cd18e0f02368ca3aaedc159872feed459d.

access: add an example of elm_object_part_access_object_get

elm_bg: fix typo in doc

Summary: Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: seoz, jpeg

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

elm_conform: set data pointer to obj from NULL.

Summary:
 - In the callback for ELM_PLUG_EVENT_IMAGE_DELETED,
   it tries to get elm_conform's data from NULL.
@fix

Reviewers: Hermet, woohyun, cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

evas: Fix inefficient updates with no_render

is_visible returns 0 if no_render is true, so should
was_visible as well. Yeah, there will be problems if
no_render changes on the fly. Don't do that.

Fixes T4193

wayland_shm: Fix fd leak in dmabuf

Need to close the fd from PrimeHandleToFD.

ecore-wl2: Reduce debug noise on key events

This is annoying to see key events flooding a terminal ... so comment
out this debug statement. If a developer really needs this, then
enable it locally.

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

Revert "access: add API to get access object of given part"

This reverts commit 57d77c9041086ebbf84c2a30d93e95d47fc12705.

We are in freeze right now and this is a new feature. Considering the access
API is still in beta I might have let this slip by but its does not even build

../../../src/tests/elementary/elm_test_popup.c:27:1: error: conflicting types for ‘elm_object_part_access_object_get’
 START_TEST (elm_object_part_access_object_get)
 ^
In file included from ../../../src/lib/elementary/Elementary.h:240:0,
                 from ../../../src/tests/elementary/elm_test_popup.c:6:
../../../src/lib/elementary/elm_object.h:521:54: note: previous declaration of ‘elm_object_part_access_object_get’ was here
 EAPI Evas_Object *elm_object_part_access_object_get(const Evas_Object *obj, const char *part);
                                                      ^
../../../src/tests/elementary/elm_test_popup.c: In function ‘elm_object_part_access_object_get’:
../../../src/tests/elementary/elm_test_popup.c:39:47: warning: passing argument 1 of ‘elm_object_part_access_object_get’ makes integer from pointer without a cast [-Wint-conversion]
    access = elm_object_part_access_object_get(popup, "access.title");
                                               ^
../../../src/tests/elementary/elm_test_popup.c:27:1: note: expected ‘int’ but argument is of type ‘Evas_Object * {aka struct _Eo_Opaque *}’
 START_TEST (elm_object_part_access_object_get)
 ^
../../../src/tests/elementary/elm_test_popup.c:39:13: error: too many arguments to function ‘elm_object_part_access_object_get’
    access = elm_object_part_access_object_get(popup, "access.title");
             ^
../../../src/tests/elementary/elm_test_popup.c:27:1: note: declared here
 START_TEST (elm_object_part_access_object_get)
 ^
../../../src/tests/elementary/elm_test_popup.c:39:11: error: void value not ignored as it ought to be
    access = elm_object_part_access_object_get(popup, "access.title");
           ^
Makefile:43996: recipe for target 'tests/elementary/tests_elementary_elm_suite-elm_test_popup.o' failed
make[5]: *** [tests/elementary/tests_elementary_elm_suite-elm_test_popup.o] Error 1

If you still want this in make sure it actually can compile the tests you added.

build: enable examples build during distcheck

No example should be left behind. Make sure we run the build for them during
distcheck so we can be sure we did not introduce any build problems here.

examples: elementary: make sure the prefs_example* files get into dist

These files have been missing from dist and thus some example builds failed.
We also need to clean them up afterwards now.

examples: edje: fix build of edje files with translation within distcheck

The edje text example covers the translation functionality which need the mo
and po files during edje_cc compilation. Further more the folder need to be
writable to create the new mo/po files.

We need to ensure that this is the case during a distcheck which is read only
on the source tree.

evas: filters: add return value in all cases

Function with non-void return value so we need to add a return in all cases.
Thanks to gcc for complaining here.

elementary: Fix formatting

NB: No functional changes

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

ecore_wl2: also add the del control code

otherwise we are sending the controlcode for delete, which is wrong.

This fixes the backspace on delete problem.

elput: Fix issue of losing mouse input when removing a device

When we are adding new input devices, if we already have an existing
one (keyboard, pointer, touch, etc) then we should just be
incrementing the device count and returning from the init function.
Previous code here would increment the device count, then continue on
and create a new internal representation of the device. This was
causing issues when a device gets removed.

Fixes T4192

@fix

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

access: add API to get access object of given part

elm table and box resize themselves on sizing eval - fix this - dont resize

thsi fixes T3031

@fix

elm_access: fix typo of atspi action

elm button theme - fix anchor style to not pass events when disabled

@fix

elm button default theme - fix disabled events to not pass through

this fixes T2955
@fix

efl - when evas generic loaders merged the gst loader was missed. fix

this fixes the missing gst loader from generic loaders.

elm default config - change default select on focus to be disabled

this also fixes T3198 but only dave's comment.

@fix

ector module - remove sueless chekc for null done already CID 1347411

fixes a pointeless check coverity found

ecore-imf - remove usueless checks for context - CID 1351451

useless checks for ic in code. fix

efl ui image async locking -fix CID 1356199 - locked data cant be messed with

this addresses a coverity issue that in theory looks right

fix CID 1356199

@fix

efl ui video - fix coverity CID 1356997

really minor lack of check if emotion init succeeds. check and
complain if it fails.

@fix

edje entry - fix coverity CID 1357752

fix potential crash if calloc fails and sel is NULL

@fix

evas cache2 - fix coverity CID 1357754

@fix

elm quicklaunch binary - fix coverity CID 1353607

@fix

evas image updates - optimize by over-rendering less with hidden content

so if an img obj had some update regions and was opaque, it didnt del
an update region first before adding the updates, thus possibly
letting update regions pass through the opaque image area

@fix

Gstreamer1: implement subtitles mute/unmute ability

This seems the only way to disable subtitles embedded in the video file.

WIthout this patch calling em_spu_mute_set/get do not work, thus I consider this a @fix

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

wayland_shm: Speed up dmabuf on intel

using map_bo/unmap_bo instead of gem_map_bo_gtt/gem_unmap_bo_gtt
results in a cacheable mapping and a large performance boost.

(dmabuf will still remain turned off by default for the release)

docs: split type serializers and keyword ref into their own module in docgen

elementary: change cursor types for cocoa

Map more cursor types for cocoa.
Fix T3352

ecore_wl2: be compatible with x input

This commit introduces a new state per input, this state is just for the
keymap and no modifieres, the state is used to get the keysym from a
keycode. This is done to stay compatible with x since keyname in efl is
the characters which are describing the key which is pressed, without
the transformation of shift etc.

This also escapes keycodes from beeing a utf8 character, and passes out
a keyname instead of the utf8 symbol.

fix T3976
fix T4184

edje: fix specular color

Summary: We use int from 0 to 255 for color, not float

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: jpeg, artem.popov

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

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

elementary: disable visual hint at odd/even.

This is a work around a huge bug in how genlist handle index. Fixing
it in genlist for this release is way to risky. The default theme is
temporary updated to not use this feature and this patch should be
reverted as soon as we start working on 1.19.

edje: allow inheritance from an aliased group.

elm_win: move window contents back to 0,0 when deleting frame object

fix T4092

elm_win: simplify borderless_set when reapplying the same state

elementary: Remove logically dead code

In all code paths leading to this goto statement being called,
'iter_array' variable will already be NULL, so checking it's validity
is just logically dead code.

Fixes Coverity CID1357392

@fix

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

elput: Use proper variable type

The 'getenv' function returns char * ... not a const char *, so adjust
variable type to match

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

eina: Fix null pointer derefernce

Coverity reports that eina_safepointer_get returns a NULL promise here
(checked 20 out of 21 times). As eina_safepointer_get can return NULL,
we should check the validity of 'promise' here before trying to
derefernce it later.

Fixes Coverity CID1356625

@fix

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

elementary: Remove logically dead code

Coverity reports that this is logically dead code. As we check the
validity of 'model' above this function, then testing 'model' for
validity again in this expression is just logically dead.

Fixes Coverity CID1356617

@fix

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

elementary: Remove logically dead code

Coverity reports that this is logically dead code. As eo_item has been
dereferenced on all paths leading here, and it is also set above, then
it cannot be NULL at this point, thus this check is logically dead.

Fixes Covierty CID1355585

@fix

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

eldbus: Check return value and error out if necessary

Coverity reports that we do not check the return value of
_eldbus_model_proxy_load here. We likely should be checking that in
case the model proxy fails to load so we can return proper values here.

Fixes Coverity CID1355233

@fix

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

edje: Fix unchecked return value

Coverity reports that we call _circular_dependency_find here without
checking the return value

Fixes Coverity CID1349867

@fix

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

evas: Fix missing breaks in switch

Coverity reports that we are missing breaks in the switch blocks here,
so add missing breaks.

Fixes Coverity CID1347413 and CID1347414

@fix

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

edje_pick: reset alias flag for copied group

edje_pick make copy of whole group, even if group is alias. So if
group was alias need to reset this flag.

@fix

eo: Fix override test case to match new policy

The new policy is a lot more restrictive.
Honestly, I don't like it.

eo: Prevent eo_override to be called on an already overridden object

I don't see the point of it but can't be bothered to argue.
For Tom.

wayland_shm: Add missing headers to wayland_shm sources

Some headers weren't being placed in the release tarball.

Revert "elementary: Make wl frame borders above content"

This reverts commit 46904e19a71dbeecbcdb6950c0f372c34f6d0db6.

this breaks stacking entirely and prevents a large number of widgets from
ever becoming visible

docs: split off writer and utils into modules in docgen

docs: begin modularizing the doc generator script

elm flip - make it not crash and interactive mode work again

elm flip has been broken for a while - interactive mode was broken and
it was causing crashes. this fixes that.

@fix

evas image updates - reduce overhead with bordered images not scaled

this fixes over-rendering (where we render the whole image) if it has
a border set. do proper region updates if the image has a border but
is not scaled and the fill is at 0 0 and sams size as the object etc.
etc. so this is simple and doable.

this fixes T4123

evas objkect image - make opaque rect getting code more readable

this just makes it more obvious where its handling the opaque middle
fill case.  see T4123 - i was just reding the code and this seemingly
does the right thing.

ignore generated shaders .x file

edje: fix ambient color

Summary: change specular color to ambient int the function of ambient color

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

elm_win: handle elm,action,menu signal for wl clients

trigger the xdg-shell method for this when the signal is received

fix T3460

theme: copy border icon click -> elm,action,signal program

elm_win: simplify opaque region updating during fullscreening

already handled during _elm_win_frame_del

elm_win: update opaque region when frame is deleted

fix T4092

config: increase base image cache to fit two 4K background in.

This is not ideal, as it would be more accurate to dynamically set
the cache to be a function of the number of screens and their size.

elementary: as screen get bigger, we need more cache to store uncompressed image.

edje_pick: copy dependent group

Parts can use enother groups as source, for example textblock. If source
groups does not exist - group is not valid. This patch extend the group
copy process and copy dependend group too.

@fix

elua: enable module lookups in apps location

This will allow for better organization of the docgen script.

ecore_wl2: dnd: fix leak of event resource when not used

In the else branch the allocated event is never used. Make sure we free the
resource before returning.

CID: 1357753

Revert "elementary: Fix issue of toggling border state causing visual jumps"

This reverts commit fb24955782b9242beb9447ebb6631b2b63921988.

While this patch did fix the issue of toggling window Borderless state
in EFl-Wl client applications, it is "Enlightenment's" stance that it
will not change to match this fix, so this has to be reverted else
internal E dialogs show up with no border (due to them using SSD).

This is going to re-open T4092 sadly :( and now leave that bug with no
possible fix.

ref T4092

Fixes T4126

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

Ui text: remove unused variables

evas: Fix call to NULL object

textblock: Prevent crash in evas_object_textblock_cursor_new(NULL)

This happened in expedite

tests: Add ifdef clang to avoid warning on GCC and elsewhere

Avoids warning about unsupported pragma

eo: Change eo_override to avoid struct passing and GCC construct

This is a (minor) API & ABI break in Eo.h!

I say minor as eo_override shouldn't be used yet (EO is unstable
and this patch includes all the use cases in EFL).

I'm not very happy about the new form of the macro, but it avoids
two things:
- passing in a struct (valid in C, but never used in EFL)
- using a GCC construct to create structs on the fly

It was inspired by the event array define, but I don't think
we need the runtime memcpy here.

See also:
https://gcc.gnu.org/onlinedocs/gcc/Compound-Literals.html

drm2: Fix shadow variable

int ret is not even needed here.

edje lang set - fix coverity harmless complaint

fix CID 1357755