eolian: fix build warning

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

wayland_egl: Remove weirdo buffer age kludge cargo culted from gl_x11

There's no reason why we'd need a full redraw if the buffer age for
this buffer isn't the same as the buffer age for the last buffer, so
let's cut out this thing.

wayland_egl: Use surface damage instead of buffer damage

We should always pass surface damage to eglSwapBuffersWithDamage.

gl_drm: Use surface damage instead of buffer damage

We should always pass surface damage to eglSwapBuffersWithDamage.

evas_engines: Send both surface and buffer damage to outbuf_flush callback

Some engines should using sending surface damage, until now we'd only ever
provided them with buffer damage.

The difference is that surface damage is the damage to the surface the
compositor is displaying, and the buffer damage is the damage to the
buffer the client has rendered.  These are different when the client
is using multiple buffers of different ages to render into.

Anything that calls eglSwapBuffersWithDamage, wl_surface_damage() or
wl_surface_damage_buffer() should be using surface damage, and not
buffer damage.

This patch is intended to make no functional change - any flush cb that
used buffer damage before still should.  Actual fixes to follow.

Apologies if I broke any engines - it's a bit of a copy and wasteland
out here.

Revert "wayland_shm: Add swap mode fallback when buffer age changes"

This reverts commit 62ca4486ea30fd63a5589ea7af373a5cb9982031.

Honestly, I think the gl_x11 code doesn't make sense either. :)

There's no logical reason that we have to do that, though it may be
hiding some other bug?  If that's the case, the bug should be fixed
properly.

ecore: test recursive case on efl_future.

ecore: allow efl_future_cancel on a promise currently being resolved.

This is necessary for allowing proper handling of recursive cancel.

Eo: Add a regression test for ref leak in function resolve

This adds a regression test for the last commit.

Eo: Fix reference leak when failing to resolve function.

When resolving a function for an object the object would get reference,
and then in some failure cases won't be freed.

I suspect this is a regression following the reshuffling that was done
in that function recently.

Thanks to zmike for investigating and reporting this.

Fixes T4740

@fix

Eo: Remove no longer relevant tests.

eolian: fix the nonsensical semantics for variables

Both variables and constants can have extern on them (it means the same thing
as in types, they won't get generated in C). This is different from the previous
semantics which only allowed extern on globals, which makes no sense.

Both globals and constants are allowed to have a value (previously only
constants were); constants are required to have a value. Globals having
a value is just a convenience for initialization during source generation.
This is unlike the previous behavior, where the value was optional for both
globals and constants and only allowed when not marked extern, which makes
no sense either.

Obivously, even when globals have a value, they're not allowed to be used
in expressions, as they cannot be evaluated at compile time (they're mutable).

gengrid: support for non homogenous items

Summary:
Dimensions of gengrid items can be altered for
non homogenity.

Care must be taken however to set the homogenous
size for items using elm_gengrid_item_size_set().

In horizontal mode only the heights will change
and in vertical mode only the widths. Fixed dimension
will be as set with elm_gengrid_item_size_set().

This is forked by https://phab.enlightenment.org/D2422

Test Plan:
elementary_test -to "Gengrid Resized Items"
Current test program provides focus autoscroll
and item looping options.

Reviewers: raster

Subscribers: cedric, jpeg

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

evas: Last fix... now it builds

evas: More compilation fixup with EGLImage

This is what happens when you can't test a patch yourself ^^

evas: Fix compilation with older EGL (maybe <1.5)

EGLImage was not introduced until very recently. EGLImageKHR was
the proper type. Thanks Roy for the report.

ecore: handle recursive trigger of promise by the cancel of a future.

This avoid double free/double callback call.

efl_net_dialer_udp: enable SO_BROADCAST before sending to 255.255.255.255

Like other toolkits, let's enable this automatically for users before
connecting to 255.255.255.255 IPv4 (IPADDR_BROADCAST), otherwise most
systems will just fail to connect and send packets.

efl_net_dialer_udp: "connect" to an UDP server to send and receive data.

Like existing ecore_con code, this does not use SOCKSv5 UDP
proxy. It's kinda cumbersome to add since requires a keep alive TCP
connection to the server, a second UDP channel and framing around the
original UDP frame.

Added UDP_CORK (if present) to match TCP_UDP present in TCP sockets,
this allows one to execute multiple write() calls that will result in
a single datagram, generated when CORK becomes FALSE again.

The efl_io_copier_example.c now accepts this as output. There is no
input UDP as there is no way to notify the server of a connection
(since such thing doesn't exit), usually servers react after a
datagram is received, replying to the source.

efl_net_socket_tcp: only emit error message if cork is being enabled.

by default we'll start with cork=0 and on adoption of a FD we'll apply
cached values, thus we'd try to apply cork=0 (default) and it would
error, which is annoying on platforms without such feature.

since users interested in TCP_CORK will enable it first, they will get
the error at that point.

ecore: replace arbitrary time with 32-bit safe arbitrary time

The end of time is much closer than you think.  it_value.tv_sec is
a signed 32-bit number on 32-bit machines. Using a negative tv_sec
causes timerfd_settime() to fail.

wayland_shm: Add swap mode fallback when buffer age changes

This copies the behaviour in opengl_x11 engine, where the buffer
age needs to be continuously the same to be taken into account.
If the age varies, then we fallback to a full redraw.

Apparently this fixes issues on actual devices. I tested this
patch in weston (I didn't have issues before and buffer age is 1).

edje_edit: avoid generate '(null)' value for image.normal

edje_edit: add correct tweens generation for inherit state

edje_cc: set correct value for attr set for MESH_NODE

On parce the image.normal token set attribute 'set' only for part type
EDJE_PART_TYPE_MESH_NODE.

@fix

elm_atspi_bridge: visible data changed signal not sent issue fix.

Summary:
_visible_data_changed_signal_send function checks for wrong object,
hence always returns with out actually sending signal.

The state is set to pd->object_broadcast_mask and not
pd->object_children_broadcast_mask in _set_broadcast_flag function.

Test Plan:
Send the visible data changed signal from any widget, but signal
is not sent.

Reviewers: cedric, lukasz.stanislawski, kimcinoo

Reviewed By: kimcinoo

Subscribers: rajeshps, govi, jpeg

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

edje_edit: refactor eet file usage

Summary:
Move opening for read/write/read-write and error message to internal method
Reuse eet file from Edje instead of opening it again in read-only mode
Add wrapper for eet_close to skip closing if internal file was used

Reviewers: NikaWhite

Subscribers: cedric, jpeg, #eflete

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

elementary: Fix call to offer_supports_mime function

As this function was misnamed in Ecore_Wl2, update the code here to
use the proper API function.

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

ecore-wl2: Fix typo in API function name

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

map: Fix elm_map usage (typo)

Emile: use stronger ssl cipher

Follows on from 356a1aa87a04a8d1c43e01fa861270d0947069c0 emile
didn't exist when this work was done originally

ecore_ssl: Use stricter cipher suites

Thanks to Victor Pereira from the SUSE Security team for auditing
this and recommending better options.
This has been discussed several times but knowone ever got to
commiting it.

evas/map: do nothing when same map is set again

Reviewers: cedric, Hermet, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

ELM: sync "clicked" usage with other places in the lib

Summary: In all other places "clicked" is defined as a local variable.

Reviewers: tasn, jpeg

Reviewed By: jpeg

Subscribers: cedric, minkyu

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

Emotion: prefer ratio to calculate sizes

Instead of calculate the ratio from the sizes, calc the width from the ratio,
as ratio seems always accurate.
Fallback to the old code if ratio not available (tbh I never see this case)

@Fix EMOTION_ASPECT_KEEP_BOTH with some rare strams, mostly DVD and online stuff

fix edje_cc segv when compiling bling bling theme

accessed parts out of range for the part table. check part count first
before checking. fixes segv. @fix

blinblng: http://enform.haxlab.org/files/blingbling_e21.tgz

evas gl x11 engine - Wrap eglCreateImage

eglCreateImage and eglCreateImageKHR have different parameters - the
attribute list is EGLint for one and EGLAttrib for the other.  Since
EGLAttrib is long, on a 64-bit system it's a different size than EGLint.

This was causing a crash in Enlightenment's wl_x11 on 64-bit machines when
trying to use a gl client.

Add a thin wrapper to use whichever is available and pass the appropriate
sized data.

ecore_evas_wayland: Don't use frame callbacks on windows with no shell surface

If we set a frame callback on a window with no shell surface (ie: and unmapped
window), the frame callback will never fire.  This was preventing some
applications like rage from ever posting a frame.

ecore_wl2: add API for testing if a window has a shell surface

eolian gen: better checks for whether to add space after type

examples: eolian_cxx: adapt code generation to changed eolian_gen commandline

The commandline options have changed for eolian_gen to generate the code. Adapt
the makefile helpers to fix the build break in examples

fix possible eina file shutdown issue

i am not sure as i cannto reproduce this, but i hope this fixes T4677
by ensuring if eina_file_shutdown is called it cannto double-free a
hash.

evas/module: add a new module in vg_saver for eet

Reviewers: cedric, jpeg

Subscribers: cedric, jpeg

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

evas: Fix evas_object_smart_clipped_clipper_get

This was broken for smart objects that are not "clipped smart
objects". This fixes the example evas_smart_object.

NOTE: This EAPI was removed in efl-1.18!
/!\ This was an uncaught API break between 1.17 and 1.18 /!\

@fix

evas: Improve render debug logs

Compiled out by default

ecore_evas - fix setting urgent immediately after show

fixes T4726

ecore-wl2: Fix input and opaque region setting

Bad y co-ordinate resulted in broken shadows after session recovery.

@fix

elementary: Fix issue of using uninitialized values

Coverity reports that Evas_Coord_Rectangles were being used here without being
initialized so declare those variables with some initial values

Fixes CID1362984

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

evas: Fix directly dereferencing pointer which may be NULL

Coverity reports this as a dereference before null check which implies
that 'cur' May be null here, so let's not use it before we check it.

Fixes CID1363765

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

evas: Fix directly dereferencing pointer which may be NULL

Coverity reports this as a dereference before null check which implies
that 'pd' May be null here, so let's not use it before we check it.

Fixes CID1364114

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

evas: Fix missing varargs cleanup

This patch fixes Coverity CID1364123 which reports that (in some
cases) va_end was not being called for var args. As they were
previously initialized with va_start, then va_end should be getting
called.

@fix

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

gengrid: call this on the pam class not on the normal class

this fixes gengrid.

eolian gen: use the new data type API to simplify the code

eolian: add API to get the C class datatype

Fix use of undeclared type in C++ tests compilation

Moved Evas_Object_Intercept_Cb_Type; typedef declaration after enum
_Evas_Object_Intercept_Cb_Type definition

cxx: Fix Makefiles for C++ compilation and installation

ecore: free futures which are still pending on shutdown

otherwise we are leaking a eina list.

This was discovered while running the elm_suite with CK_FORK=no, since
ecore and eina are init´ed and shutdown´ed, after the shutdown this list
points to freeed memory pools. So in the next testcase the list is
invalid and crashes. This fixes it with freeing the list on shutdown.

evas/module: add a new module in vg_loader for eet

Reviewers: jpeg

Subscribers: cedric, jpeg

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

evas: Use proper type in smart objects API (eo)

eolian gen: use the C get function name getter instead of manual concat

eolian gen: utilize the new class C name getter instead of manual concat

eolian: add an API to get the C name used to access the class

edje_edit: new API for generate source code for color classes

Extend edje edit deberate source API. Add two new API.
 - edje_edit_object_color_class_list_get return a list of used color classes for
   given object
 - edje_edit_color_classes_source_generate is generate code for given
   color classes list

edje_edit: add API for get source code of global block data

This API is annex for edje_edit_object_source_generate. Together this
API's provide a mechanism for generate source for custom groups from
a edj file or more.

edje_edit: add API for generate group source code without 'collection'

evas/module: Added a new module vg_loader for svg

Reviewers: cedric, jpeg

Subscribers: cedric, jpeg

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

elm_widget: Remove unnecessary set{} get{} in eo file

edje: fix memory leak

evas: Mark group_add/del as internal

Lacking a proper internal tag, I'm using both protected (it is
in fact a protected access function) and beta (to mark as unstable,
not real API).

New smart objects based on EO only should rely on constructor,
finalize and destructor exclusively. In theory, this should be fine.

Unfortunately it may be impossible to inherit from the Efl.Ui.Win
class as it uses a really bad hack and calls super.constructor
inside the finalize method.

evas/elm: Remove function group_resize

This is an override of efl_gfx_size_set. Same as before, the
order of operations matter so it is possible that a corner
case will break. In particular, legacy code was:
 - intercept
 - smart resize (do stuff), super, super, super
 - evas object resize

The new code is more like:
 - intercept
 - super, super, super, evas object resize
 - do stuff

But unfortunately this broke elm_widget (read: all widgets) as
the internal resize was done before the object resize. So,
inside the resize event cb, the resize_obj size would not match
the smart object size. >_<

evas/elm: Remove function group_move

This is an override of efl_gfx_position_set.
As for the other patches, I hope I didn't break anything.

A problem likely to happen is that the super call was inserted
too early or too late in the call flow. For instance:

  _myclass_position_set(obj, x, y) {
    position_set(super(obj), x, y);
    position_get(obj, &prevx, &prevy);
    do_something_with_delta_xy();
  }

The above code flow is obvisouly wrong, but may have crept in this
patch (such a bug sneaked in inside smart object, breaking
everything at first).

evas: Move move_children_relative to legacy only

While this kind of API seems to make sense with smart objects
(relative coordinates), it is currently not used apart from
the smart object class itself.

So, for now, I'm moving this to legacy to clean up Efl.Canvas.Group
and we can later add the equivalent in a clean "group" API.

evas/elm: Remove group_show and group_hide

These should be just overrides of Efl.Gfx.visible.set. Many
widgets were handling smart show() and hide() manually, which
means this patch is quite large.

Hopefully this doesn't break anything, obviously. But here are
some widgets known to be problematic, as the old code flow was
really strange (sometimes not calling the efl_super function):
 - window
 - notify

evas: Move clip_unset to legacy, remove group_clip

Similarly to group_color_set, group_clip_[un]set should not
exist and should be a result of efl_super and inheritance.

This patch also removes clip_unset from the EO API and keeps
only clip_set(NULL). The reason is that it will avoid bad overrides
of clip_unset() vs. clip_unset(NULL). This also simplifies the code
a bit. Ideally we should be able to reintroduce clip_unset in EO
if we can have a "@final" tag (like java's final keyword), to
prevent overrides.

evas: Remove inexisting private function declarations

evas: Remove method group_color_set

Widgets should simply override efl_gfx_color_set and call
super all the way up to evas object.

Legacy compatibility with call interceptors and early call
abortion (eg. delete_me or obj->layer == NULL) are implemented
with an internal call. See the previous commit introducing the
API.

evas: Add pseudo-internal API to intercept basic calls

This is a poor man's solution to get rid of group functions such
as clip_set, clip_unset, color_set, etc... See the following
commits.

This API needs to be EAPI for elementary but shouldn't be used
outside EFL. This is required purely for legacy compatibility.

Here's the call flow, inside show(obj):
1. if (intercept_show(obj)) return;
2. show(super(obj));
3. do other stuff

evas: Simplify code with intercept macros

evas: Move intercept function types to legacy header

evas_data: Fix safety checks in evas_object_data

This relies on efl_isa rather than the broken magic check
(by broken I mean it only checks for NULL).

evas: Move group color_set after checking for change

I can't see why the group (smart object) color_set function
was called even if the color hadn't change. Let's test and
see if this breaks anything :)

evas: Small simplification of object color_set

group: Replace group_no_render by object no_render

Most of the smart functions "Efl.Canvas.Group.group_xxx" should
not exist and be overrides of the base object function instead.
Cleaning this up is necessary if we want an EO API for custom
smart objects. This patch is the first attempt at removing a
method (the simplest one).

As for no_render, I wonder if propagating to the children
really is necessary. evas_render should skip them already.

ecore-drm2: Free Output modes on destroy

When we destroy outputs, we should be freeing the Output's Modes also
as that was previously allocated memory.

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

ecore-drm2: Use Atomic Modesetting for resolution changes

Since we have atomic properties now, we can use those to set given
Output modes (resolutions).

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

edje_calc: add rounding pixel calculation for transition.

Summary: when transition animation is working, sometimes one pixels loss occurs.

Reviewers: cedric, woohyun, jpeg, raster

Subscribers: jpeg

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

edje_edit: proper work with part id's on part restack and del for map fields

@fix

evas: Fix no update issue if visiblity of smart object has change

Summary: If child has change about visibility, can't check it correctly.

Reviewers: jypark, cedric, jpeg, Hermet

Reviewed By: Hermet

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

windows: remove expicit -levil flag

Summary:
Built from sources version of evil is already added to linker flags and
adding extra -levil makes build fail if evil is not already installed in system.
Looks like this flag was here from old times when all efl libraries were separated.

Reviewers: vtorri, NikaWhite

Reviewed By: NikaWhite

Subscribers: cedric, jpeg

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

evas: Fix not update issue with TBM surface

Summary:
If the pixels of image object has updates
via native_suface_set api with TBM surface,
does not update on screen(no rendering)

Reviewers: jpeg, cedric

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

Merge branch 'devs/iscaro/osx'

Some fixes for OS X.

This series fixes two problems pointed by jayji and some other
compiler warnings.

Patches by iscaro.
Differential Revision: https://phab.enlightenment.org/D4339

Ecore Evas Cocoa: Properly unregister the window events.

Fix T4624

Eet example: Add the correct variable in the hash.

Efl UI Win: Avoid unused variable.

It's only used under X11 and Wayland backends.

Ecore_Thread: Avoid compiler warning

Use PRIuPTR to print the Eina_Thread handle.

Ecore Cocoa: Add missing modifiers field to Ecore Events

Fix T4477

Ecore_Cocoa: Fix repeat modifers keys.

Some values were repeated.

ecore_ipc: fix typos in documentation group names

Summary: fix typos in documentation group names to grouping APIs correctly

Reviewers: cedric, jpeg, Hermet

Reviewed By: Hermet

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

FDO icons: add missing system-file-manager icon

Fix T4638

(cherry picked from commit 3fe1429eee5568ef7af7bc47a9dc43ce65e084ca)

elm_code: Don't allow cursor to be placed mid-tab