elm_code: Fix some syntax highlight after strings

elm_code: Syntax highlighting for multiline comments

luncher: Until we fix expansion zoom a little less.

This allows an area in the middle where you are only over 1 icon...
clicktastic

elementary conformant: clear internal pointer if holded object is deleted

Summary:
Set sd->scroller as NULL when the scroller object is deleted.
It prevents error logs when elm_conform tries to delete resize callback
function from deleted object.
@fix

Test Plan: N/A

Reviewers: raster, cedric, jpeg

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

eolian: rmeove dependency on basename/libgen.h/evil

elm_code: simpler handling of token painting

evas font: revise evas_font_init/reinit functions

Summary:
The static flag in evas_font_init() was not useful.
It could be replaced by checking address of "fc_config".
FcInitReinitialize() function was not necessary to reload
configure and font files in Evas. It would be meaningful
when only Evas use Fontconfig's function without own "fc_config".
To reload "fc_config", calling FcInitLoadConfigAndFonts() is enough.
And there is no need to load "fc_config" from evas_font_reinit()
when "fc_config" is not prepared.

Test Plan: N/A

Reviewers: herdsman, raster, tasn, cedric, woohyun

Subscribers: jpeg

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

po files - update line's + comments

elm_code: Improve string syntax parsing

elm_code: Add initial simple syntax highlighting for C code

For c source and headers lookup a simple syntax definition.
Use a very simple text processing line by line to tokenise.
Simple but it gets us one step closer to cross-language syntax

docs: WIP overridden method/property documentation support in docgen

docs: use new implement based doc API

docs: remove some duplicate doctree methods

docs: add implement to doctree

eolian: merge database_fill into parser

eolian: prevent memory leaks in db fill

eolian: simplify database_fill by reducing the number of passes

evas: Fix very unlikely crash in evas render

During a stress test of EFL a seemingly impossible crash happened, where
one of object's cow fields was NULL inside evas_object_was_visible.
Nothing in the code flow can possibly lead to this situation but
it still happened. For information, the object's delete_me flag was 2 in
evas_object_is_active() called from _evas_render_phase1_object_process().
So let's add a small safety check for crash prevention.

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

Rename Efl.hh to Efl.eo.hh in CLEANFILES

This avoid make clean removing Efl.hh in the tree

wayland: Try not to draw into surfaces without roles

xdg v6 will barf if we attach a buffer to the surface before
we receive a configure - even if we attach before trying to
make a shell surface.

So we need to prevent any drawing into surfaces until we've
decided what kind of surface they'll ultimately be.

Fix T5090

eolian: add implement validation (currently for docs)

eolian: move base function doc API into implements

This allows us to unify retrieval of docs for both regular and
overridden funcs without having two separate APIs. It's currently
missing validation and docgen is still not adjusted properly for
it either, but at least there's this. Enables retrieval of docs
for overridden funcs by default as well.

gstreamer buffer unref emotion: too many urefs! Seems I read the documention incorrectly!!!

Reviewers: raster, jpeg, cedric

Reviewed By: raster

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

edje_cc: delete unused image sets from edj file.

Delete Edje_Image_Directory_Set instance from edj file in case if image
set isn't used inside any part. This commit apply behaviour from unused
images to the unused image sets.

@fix T5109

eolian-cxx: Add all cases in switch to remove warning

eolian-cxx: Do not call eolian_function_scope_get with EOLIAN_PROPERTY

cxx: Modify how to generate C++ headers and allow cyclic dependencies

Allow cyclic dependencies in generated C++ headers by changing order
of includes and creating forward declarations.

eolian-cxx: Added cyclic compilation test

build: Fix wayland scanner related breakage

Oops, autoconf noob at work.

ecore_imf/wayland: Auto generate wayland text-input protocol

Needed to do some fairly invasive changes because naming conventions
changed since we checked in the header.

Also, since I'm done converting protocol, change the gitignore to
use wildcards for wayland protocol.

This fixes a long standing bug in text-input binding and restores
previous OSK behaviour.

ecore_wl2: Auto generate www protocol

Needed to copy the protocol file from Enlightenment.

ecore_wl2: Auto generate teamwork protocol

The name was non-standard and we didn't have the xml in tree,
xml has been copied from Enlightenment.

ecore_wl2: Auto generate xdg v6 protocol

Mostly trivial, needed to update private header as we'd used a
non-standard name for our generated copy.

ecore_wl2: Auto generate xdg v5 protocol

Remove our generated copies, also rename the XDG_VERSION macro
so it's not quite so confusing now that we support v5 and v6.

ecore_wl2: Remove private copy of subsurface protocol

This became core wayland functionality a long time ago, and we
now depend on wayland new enough to have it, so we should never
need the stale copy we had in tree.

ecore_wl2: Auto generate dmabuf protocol

Remove the server protocol file as we don't need it here.

Generally a trivial change as we have a recent version in tree with
no weird changes.

ecore_wl2: Auto generate session-recovery protocol

Check in the latest session-recovery xml and remove the checked
in generated files.

build: Add rules to auto-generate wayland headers from protocol xml

We've been checking in protocol headers and implementations that
wayland-scanner generates.  We should just generate them from the upstream
xml files.

build: Disable automake compatibility warnings

We require GNU make, so stop complaining when autoconf does things
that require GNU make.

ecore-drm2: Add API function to set output rotation

This patch adds a new API function that can be called from
Enlightenment wl_drm module to enable output rotation.

NB: Only works if Atomic support is enabled as it rotates the hardware
plane directly...and we don't support planes without Atomic enabled.

@feature

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

ecore-drm2: Add API function to return supported rotations of an output

Small patch to add an API function which can be used to return the
supported rotations of a given output. This is used inside the
Enlightenment wl_drm module to determine if rotations is supported on
an output.

@feature

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

ecore-drm2: Expose Ecore_Drm2_Rotation enum

As we will need these values when doing rotation checks inside wl_drm
module (for randr rotation support), let's move them out of the
private header and expose them in Ecore_Drm2.h

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

ecore-drm2: Minor formatting fix

NB: No functional changes, just formatting

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

ecore-drm2: Add API to determine if a device prefers shadow buffers

Small patch to add a new API function that can be called to determine
if a given drm device prefers the use of shadow buffers. This API
will be used later to provide some optimizations on various platforms.

NB: Requested by Derek

@feature

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

eolian: add syntax for overridding docs in implements

ecore-wl2: Add function to get touch device

When we are sending input events, if we have no pointer device then we
should be setting ev->dev to a touch device (as touch events are
treated as pointer events inside EFL).

NB: This allows EFL clients to get touch events in Enlightenment.
There are still some small hiccups here (can't close terminology by
pressing the 'x' in the corner, cannot scroll elm_test srollbar, etc).

Likely EFL needs to change wrt all this...perhaps adding events for
touch that are separate from pointer ?...

ref T5094

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

elput: Return touch coordinates when we have no pointer

If a user calls elput_input_pointer_xy_get (as is done via
ecore_evas_drm) and a pointer does not exist, we never return any
coordinates for this function.

Enlightenment is using ecore_evas_pointer_xy_get (which when using the
drm ee, ends up calling elput_input_pointer_xy_get). If we have no
pointer device, then no coordinates are ever returned and touch
clicking does not function properly.

To fix that we will check if a touch device exists and supply the
coordinates from that (in the case where there is no pointer device).

@fix

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

Edje_Edit: add forgotten API for map.zoom.x/y

evil: include "sys/types.h" where is define off_t

edje_edit: add Efl.File.mmap.set implementation for edje_edit

This fixes edje_edit_program_source_get for mmaped edje_edit object
@fix

elm_test: Make the animated filter a bit prettier

This is a basic example of fire text.

The gif was generated with a GIMP script from
http://fence-post.deviantart.com/art/GIMP-Animated-Fire-Script-84921148

Ui text: fix size evaluation for default setup

Now the size evaluation will query for the native size of the
canvas.text object, and continue with calculations to set the min size
of itself.
This fixes a bug in containers where the widget's size wasn't picked up.

Also, the canvas.text object wasn't reporting 'changed' on text changes.

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

evas: Fix classic text styles with filters in textblock

Textblock filters support RGBA input which means legacy styles
can be used in conjunction with filtering. Not recommended, but
it works. Note: We may decide to drop this behaviour and use
alpha-only inputs for simplicity.

Still missing: support for filtering strikethrough, underline, or
embedded items

elm_test: Add a filter to the code entry

Just because I can. It's the filter code editor after all, deserves
a filter of its own. Plus, it tests that we can embed a filter
in the default style, and edit text with a filter and everything
works as expected. Yay!

evas: Fix crash with invalid input in textblock

evas: Redraw tb filters if source changed

Filters can have sources like image proxy, and should trigger
a redraw in case the source has changed. Since we cache the
filter's output, we need to first check whether the sources
have changed before reusing a previous output buffer.

elm_test: Search bar enter key activates test on exact match

Just type "entry" and hit the enter key.

evas: Fix textblock filters with large items

If the line height is different from the text item height (eg.
because there are large embedded items) then we need to add
an extra offset to the draw commands.

Note: items themselves are not filtered (yet, at least).

evas: Drop unused buffer from textblock filters

This is a first step before implementing some form of caching of
those output buffers. At the moment, it very aggressively deletes
any buffer that falls outside the clip of the textblock object.
Note that this is better in terms of memory usage but way worse
in terms of render performance (eg. scrolling). If a textblock
is a proxy source then we keep all the buffers (the entire object
is to be rendered).

+ fix a crash

evas: Fix unsafe code in previous patches (tb filters)

This also reduces the overhead required per text item when there
is no filter.

evas: Introduce event style_insets,changed for tb

This will be triggered in the rare case when a textblock's
insets are changed (ie. the padding due to filters or style).

This fixes invalid sizing in the test case in elm (due to a
lack of event after program_set).

@feature

evas: Implement buffer caching for textblock filters

This is the most basic optimization that needs to be done for
filters to be useful: cache the output rgba buffers for each
filtered element. Hopefully this doesn't leak. I'm not making
any promises about that though :)

evas: Implement filter sources support for textblock

evas: Implement filter_data_set for textblock

This is a function that allows passing variables from C or EDC
to the filter's Lua code. Useful in particular for color classes
from EDC.

This data would be the global data but we could eventually add
a markup tag to specify a data value per filter instance. For now
a single data value per tb object should be more than enough though.

elm_test: Improve gfx filter test case for tb & text

Now we can test both of those widgets with filters. We
could also add an image. This test case still needs some
improvement anyway.

evas: Adjust tb padding from gfx filters padding

This makes gfx filters padding work just like the standard style
padding, which means the tb user must offset the object position
by -l, -t and increase the object size by l+r,t+b.

evas: Remove unused 'inset' field from tb items

evas: Fix tb filters spanning over multiple items

If a gfx filter was applied to a block of text spanning over
multiple text items, then it would improperly render as the
filter context was stored in the format, rather than the text
item. This is fixed by using a list of contexts in the format
node rather than a single context.

evas: Set gfx filters by name on textblock

This allows for much simpler markup, and easier tracking of
which filters are in use, etc... Next: data and sources.

evas: Experimental textblock support for gfx filters

This is only for testing purposes for now. Eventually we need
to fix the following things:
- terrible performance (cache buffers)
- force redraws based on filter padding
- expand textblock padding based on max filter padding
- add sources, data and a filter name/code hash
- test! :)

evas: Remove most extra data from filter mixin

This should save a bit of memory for all image & text
objects. This exploits the previous patch for the post-render
job queue added to evas, and simplifies this bit of code.

evas: Add a very simple post-render job list

This will be used by the filters

evas: Refactor evas filters a bit

This is a preparation step for (experimental) textblock support.
Textblock objects won't have a single filter, and the buffer's
geometry wouldn't be that of of the object itself. Thus a few
internal APIs need to be reworked first.

eina: Remove EAPI eina_freeq_main_set()

I believe this function is not required and should not be
used by applications. If there is a very good use case to
use your own main freeq, then the API could be added again.

For now, removing the set() is probably the safer option.

Note: the API was introduced in the upcoming 1.19

evas: Use eina_slstr to simplify render debug

Only enabled if REND_DBG is manually enabled inside evas_render.c

eina: Introduce Eina_Slstr for short-lived strings

Built on top of the new 'postponed' free queue, the short-lived
strings API allows users to return new strings without caring
about freeing them. EFL main loop will do this automatically for
them you at a later point in time (at the end of an iteration).

The APIs provided will either duplicate (copy) or more generally
steal an existing string (char *, stringshare, tmpstr, strbuf),
taking ownership of it and controling its lifetime. Those strings
can then be safely returned by an API. From a user point of view,
those strings must be considered like simple const char *, ie.
no need to free() them and their validity is limited to the
local scope.

There is no function to remove such a string from the freeq.

The short lived strings API is not thread-safe: do not send a
short-lived object from one thread to another.

@feature

eina_freeq: Add mode for postponed deletion

While this reuses the existing (but new) infrastructure of
eina_freeq, the mode of operation and objective is very different
from the default freeq.

By default, any object added to the freeq is basically already
freed from the user point of view, and the freeq itself only adds
a tiny layer of memory safety by deferring the actual call to free
and optionally filling the memory blob with a pattern ('wwwww...').
This is mostly thread-safe (requires thread-safe free functions).

This new type I called postponed is intended to store objects that
will be short lived. This is not thread safe as the life of the
objects added to this queue depends on the thread that adds to
the queue. The main intent is to introduce a new API for short-lived
strings.

@feature

eolian: merge internal funcs in database fill

eolian: remove/add APIs and clean up implements system

As there is no need to have separate is_auto, is_empty and
is_pure_virtual for functions and implements (each function has
its own base implement by default) I removed the function ones.
Instead, I added a way to retrieve a function's base implement
so that you can instead do the checks on the implement even when
you only have the function.

I also moved base implement build directly into the parser instead
of the database filler. That allows for significant cleanup. I
also removed distinction of implement pointers in Eolian_Function
for get and set as implements now always contain an entire thing
so the pointer was always the same anyway.

Things should still behave more or less the same, but ordering
of generated functions has changed because ordering of implements
has changed.

interface_scrollable: Unify basis of calculation of page_get logic

Summary:
If rtl mode is set, current_page_get api should return reversed page number.
To do that, make x position x-axis reversed before page calculating.

Also bring_in and page_show should show the reversed page in rtl mode.
This patch modify the functions to support that.

Lastly, scroller should be scrolling based on the right edge of the page.

This patch is a combination of the patches(D4559,D4560)

Test Plan:
1. Run scroller test on elementary_test
2. Turn ui mirrored mode on
3. Manipulate scroller in various ways
    - It should scroll proper position when you click next or prev btn.

Reviewers: woohyun, taxi2se, z-wony, cedric

Subscribers: cedric, jpeg

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

elm_multibuttonentry: Fix MBE got a wrong state.

Summary:
Even User set a MBE state as "shrink" when MBE created.
MBE has been changed the state as "none" during added items.

This patch will be fixed that bug state.

@fix

Test Plan:
Add below line after create mbe.

'elm_multibuttonentry_expanded_set(mbe, EINA_FALSE)'

Then Add items using item_append API.

See the result. mbe is not on shrink mode.

Reviewers: Hermet, jpeg, cedric

Subscribers: cedric, jpeg

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

elm_code: Add some safety

elm_code: Fix selection when moving around with cursor keys

Also fix the handling of newlines in our selections
@fix

net gadget: try to improve the theme

* make the 2 monitors fill based on tx/rx percentage vals
* try a more bluish version (still need a bit of love)

Note that this is not working atm (okra need to fix in E)

Luncher: Fix engage theme to scale more smoothly.

There is still a layering issue and the algorithm could be improved

ecore-wl2: make session recovery listener global

global proxies have a single instance and do not work with
multiple listeners

@fix

elm_box: allow aspects 1 <= x < 0.0

this was a typo in my initial implementation

@fix

ecore-wl2: Use input serial for move and resize functions

This is a partial revert of 2bd880c6c7 which changed to use
wl_display_get_serial function. When can do that for most things,
however things like window move/resize need to use the input serial
which started the move/resize.

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

docs: generate correct eolian function signatures

Now signatures don't contain full class name, just func name.

eolian: change eo file syntax @virtual_pure -> @pure_virtual

eolian: use consistent pure_virtual naming in API

Previously one API used just virtual (incorrect) and one used
virtual_pure (which just sounds weird). So unify with a single
name, pure_virtual, similar as in other lanugages.

This does not change eo file syntax yet.

eolian: consistent and cleaner error values from APIs

Now all error/unknown/etc values returned from APIs are zero,
previously it was a mix of zeroes and minus ones. Also, some
enums that had no error/invalid value before have one now, which
allows for better distinction between what is an error and what
is an intended result.

elm_bg: rotate image according to the orientation info

rotate image into the standard orientation according to the EXIF data
(think about taking a picture from camera and setting it as bg)

efl_ui_win - switch to the element being used for border type

elm_gesture_layer: update doc.

Summary:
Add description about gesture APIs.

Some of the developers told me that they really have no idea what is this for when first see the API doc...

Reviewers: Hermet, raster, jpeg, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

elm.hoversel: revert unnecessary item expansion & get proper size

Summary:
The changes of previous patch (4ea7effe70423b5f88f98aea45a96582669aa280)
are reverted, and item calculation is fixed correctly.
The main reason why hoversel item has wrong size in screen rotation is
that hover doesn't update geometry when the size of target object is changed.

Test Plan: elementary_test -to hoversel

Reviewers: cedric, jpeg

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

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

elm_hoversel: reset alignment before calculation

Summary:
When screen is rotated, _resizing_eval() will be called twice by hover_parent
resize and hoversel movement.
If the alignment is changed to the right in the first call because of the geometry
of edje part is not updated yet (hoversel uses edje part geometry in calculation),
hoversel keeps being right-aligned in the second call,
even though there is enough space to show with default alignment.

Test Plan: elementary_test -to hoversel

Reviewers: cedric, jpeg

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

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

ecore_file: add case to properly rename a file in Windows

Summary:
If the file with a new path was created and 'rename' wants to replace the old path to the new path. 'rename' will return:
   Windows 7: -1 (errno=EEXIST) (EEXIST == 17)
   Ubuntu: 0

**EEXIST**
**Ubuntu**: The link named by new is a directory that is not an empty directory. (https://linux.die.net/man/3/rename)
**Windows 7**: Files exist. An attempt has been made to create a file that already exists. For example, the _O_CREAT and _O_EXCL flags are specified in an _open call, but the named file already exists.(https://msdn.microsoft.com/en-us/library/5814770t.aspx)

Test Plan:
**Sample code to rename in Linux and Windows if the file with the new name already exists:**

int main()
{
const char *_old = "old";
const char *_new = "new";

int fd1 = open(_old, O_CREAT);
close(fd1);
int fd2 = open(_new, O_CREAT);
close(fd2);
printf("rename:\t%s -> %s\n", _old, _new);
int r = rename(_old, _new);
if (r == 0)
{
printf("GOOD\n");
}
else
{
printf("CODE ERROR:\n"                  );
printf(" -rename...: %d\n", r    );
printf(" -errno....: %d\n", errno);
}
return 0;
}

Reviewers: raster, vtorri, jpeg, NikaWhite, reutskiy.v.v, an.kroitor, cedric

Reviewed By: cedric

Subscribers: artem.popov, cedric, jpeg

Tags: #efl, #windows

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

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

emotion: add support for obtaining embedded artwork from media files with gstreamer1 backend.

Reviewers: raster, cedric

Subscribers: cedric, jpeg

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

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

Ui text: fix deletion of the swallowed text object

The Efl.Canvas.Text object loses its parent due to a specific behavior
of Edje when swallowing objects.
The mentioned object then loses sync with the Ui.Text object, thus not
getting deleted upon destruction - that's actually because Edje
reparents to the canvas upon its own deletion.
Adding this fix until we figure out if there is a better alternative.