evil: remove unused files (evil_p*) and uneeded evil_uuid.c file (managed by mingw-w64)

Just keeping up with latest mingw-w64.

eio: forgot to test when xattr tests are disable.

eio: this should not be visible outside of this file.

eio: add test cases for eio xattr module.

Summary:
Added test cases for eio_file_xattr, eio_file_xattr_set and eio_file_xattr_get functions

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

eio: reorder include to fix build on Windows.

eio: headers are part of the source code and should be handle like they are.

ecore_win32: don't fail if OleInitialize() had been called before.

ecore_file: less confusing documentation of ecore_file_download API

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric

Subscribers: cedric

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

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

ecore_file: fix crashing test of ecore_file

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric

Subscribers: cedric

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

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

evas: more fix for include order in an attempt to fix Windows build.

evas: update Evas_Object_VG examples to use eina_matrix api correctly after change.

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

evas: update matrix calulation as the behaviour of eina_matrix api changed.

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

eina: updated matrix_translate, matrix_scale and matrix_rotate implementation to do compose of the matrix in not identity matrix.

This is a late change that has been discussing on the ML as we don't want to release an API, we can't make faster.

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

Revert "edje: refactor styles_del() logic like styles_add()"

We are in a feature freeze. No room for refactoring/new features, only
surgical (or if you must, non-surgical) fixes!

This reverts commit fa0384ed2c0f784d26f0bfbd4ff40940921768c4.

edje: refactor styles_del() logic like styles_add()

evas: restore the value of "do_region" in jpeg loader as EINA_TRUE

this was an unwanted change from 1ab8cc75c4b85d3421ebfd34d04f2f4c01e99468

@fix

Evas textblock: remove freed item from the line list

@fix

Evas masking: Skip GL fast path for border images

Right now the engines don't support mask tiles so we can't
just scale up an image on-the-fly when doing a masking operation.

Skip fast path and force render of border images into their own surface.

Evas render: Fix debug logs

Those are only printed out if REND_DBG is set to 1 in evas_private.h
--> Non functional change.

eio: add new tests for eio.

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric

Subscribers: cedric

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

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

evas: fix bug in Evas_3D examples.

Summary: Fix evas-3d-shooter and evas-3d-parallax-occlusion.

Reviewers: cedric, raster, Hermet

Subscribers: cedric

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

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

epp: fix annotate.

edje: fix misspelling of annotate.

Summary: Anotate to Annotate

Test Plan: test it to make sure I didn't break it...

Reviewers: raster, cedric

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

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

evas: fix possible segfault in evas_3d_node_member_add/del.

Summary: @fix

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

ecore-drm: Don't fetch output geometry on every mouse movement

Summary: This is a better way to allow mouse movement across multiple
outputs as this does not require us to fetch output geometry on Every
mouse movement (as before).

@fix

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

ecore-drm: Fix mouse movement across multiple outputs

Summary: When running E-Wl with multiple outputs, we could not
previously move the mouse pointer across to the second output. This
commit fixes that issue.

@fix

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

ecore-drm: Fix hotplugging of outputs

Summary: In order to process hotplugging of outputs, we need to send
the 'dev' structure as data to the callback so that we can update
outputs. This fixes hotplugging of outputs when running E

@fix

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

evas-drm: Mark the Ecore_Drm_Fb as complete when we get the page flip event.

Summary: This fixes Ticket T2289 where the mouse pointer was leaving a
trail while moving. This was caused due to not waiting for the page
flip to complete

@fix

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

ecore-drm: Add pending_flip field to Ecore_Drm_Fb

Summary: This adds a pending_flip field to Ecore_Drm_Fb structure so
that when we schedule a pageflip event we can wait until the flip is
complete before trying to reuse the buffer again.

@fix

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

evas-drm: Handle render mode for 4 buffers

Summary: If we are using 4 buffers, we should be returning
MODE_QUADRUPLE for render_mode_get function

@fix

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

mailmap: update q66

eio: reorder tests linkage to unbreak debian-based systems

@fix

eina: remove warning from eina_str tests.

ecore_con: add test cases for ecore_con_eet.

Summary:
Added test case for ecore_con_eet.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

ecore_con: add test cases related to cookies.

Summary:
Added test cases related to ecore_con_url cookies.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

ecore_file: improved Ecore_File_Download test.

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric

Subscribers: cedric

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

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

evas: fix compilation in the Evas 3D examples.

Summary: Fix compilation line.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

Revert "eo: add eo_error_get"

As discussed on IRC and ML. We are in a feature freeze phase, and this
patch is not essential. Furthermore, this patch was never discussed.

This reverts commit 537c7fe9e3a72242c386b8851dd450fdc6312171.

eo: add eo_error_get

This is pair of eo_error_set.

Emile: Reduce ERR logs on invalid JPEG images

When reading the head of a file, we may get the error that it is
not a JPEG image (which is normal), so we should not print any ERR.

The JPEG header read function can indeed be called to test whether a
file can be opened by the JPEG loader or not (any file).

Note that JPEG files don't have reliable magic numbers, so we
don't check them, but rely on libjpeg instead.

Fixes T2290

evas sw: fix compile warning

modules/evas/engines/software_generic/ector_surface.c: In function '_ector_cairo_software_surface_surface_set':
modules/evas/engines/software_generic/ector_surface.c:89:4: warning: implicit declaration of function 'evas_common_cpu_end_opt' [-Wimplicit-function-declaration]
    evas_common_cpu_end_opt();
    ^

eo_gdb.py: fix syntax error

I guess this has never ever worked. I hate everyone for making me look at python.

@fix

eio: correct header inclusion for Windows.

evas: split ector surface from gl generic backend in an attempt to fix windows build.

evas: split software backend use of ector surface in an attempt to fix windows build.

tests/eeze: Disable libcheck timeout like we do for the rest of our suites

Recently I have seen that eeze_suite times out more often on jenkins when
being under load. Looking in this revealed that we have been using the default
4s timeout for eeze while almost all others suites had the timeout disabled
already. In eeze there should be no test that would really block and if I find
one I can simply add a specific timeout like we did for eina_barrier.

Evas GL: Add some safety code

Set some pointers to NULL to avoid invalid accesses after free.
Move the similar condition for sfc after the make_current check.

Evas GL: Disable direct rendering if there is a map

The engine itself (gl_generic) will detect whether the target
surface (canvas where to draw the gl stuff) is the window
backbuffer or not.

Evas GL: Fallback to indirect rendering if mul_col != 0xFFFFFFFF

@fix

eio-cxx: Added Eio.hh to .gitignore file

Since Eio.hh is a generated file, added it to .gitignore file so it
doesn't pollute git statuses.

eolian-cxx: Fix dependency problem in makefile

Fixed lacking explicit dependency for C Eolian headers, causing random compilation errors in test for eolian-cxx.

Evas GL generic: Fix log domain

The log domain EvasGL is used by evas_gl_common and initialized
only when an Evas GL surface is created.

Evas render: Fix double-clipping issue with maps

Situation:
 - Evas Object A has a clip C and a map M.

Problem:
 - Clip C will be applied once inside the map surface S and
   again when the surface S is drawn to the canvas.

Solution:
 - Track whether the current object is the mapped object
   or a child of the mapped object. In the first case,
   discard the clipper when rendering to the map surface.
   In the second case, the child's clipper is PROBABLY[*]
   inside the map, so it should be applied when rendering the
   map surface itself.

Note: This also applies to masks.

[*] This is clearly not the ultimate clipping fix.

Evas GL common: Fix RGB+A textures

Since the shaders system rewrite, RGB+A (used for ETC1+Alpha encoding)
did not work anymore.

Evas GL common: Fix texture preload with RGB+Alpha

Evas GL masking: Skip fast path if colorspace is not supported

Masking in GL assumes only one texture to sample from. This means
RGB+Alpha and YUV types are not supported. While it would
make sense for RGB+Alpha, it doesn't make any sense for YUV (because
masks are alpha planes and YUV is opaque...)

Emile image: Fix decoding of ETC1+Alpha images

This bug is probably a copy&paste issue when writing emile.

Evas GL common: Fix masking of maps based on RGB+A textures

That's like a case that will never happen.
Apply a map on an image loaded from an ETC1_ALPHA image. Oh, and
also add a mask on top of that.

efreetd - cache - mark cd as const as it isnt needing a free on return

eina-cxx: Fix compilation error on clang

Clang implicitly deletes the default-constructor on range_ptrlist, which must be explicitly defined.

cxx: Made automake files depend on their C counterparts for compilation

Avoids redundancy and maintenance for non-C++ developers.

eolain_cxx: Fix C++ support for new Eolian features

Added optional constructor methods for C++ Eolian wrappers.
Changed the interface of wrappers' main constructors.
If there are optional constructor methods they should be passed as variadic
template argument at the end of the constructor.
To support variadic template arguments, the optional "parent" parameter is
now the first parameter and there is another constructor without the
"parent" parameter.

Checking for @optinal and @nullable attributes instead of @nonull.
Now @nonull is the default, and eina::optional is only used when @optional
or @nullable attribute is specified.

The names of constructor methods no longer have the class name prefixed.

Added unit tests for checking the binding of optional constructors.
Added new .eo file to be used in the test.

Changed the generated documentation of constructors.

Changed the efl::eo::inherit accordingly, to address these new features.
Now the constructor methods should be explicit called in the
efl::eo::inherit constructor, which will receive them via variadic
template arguments.

Added another constructor to efl::eo::inherit for passing the parent
object.

Updated some tests and examples to follow the new interface.

Removed some code that is no longer necessary.

Also, fix Eolian C++ support for constructing properties. fix
assertion when parsing constructing properties.

Now if a property is a constructing property eolian_cxx will generate a
constructor method that have the property name (without the "_set" suffix).

eina_cxx: Add methods to get an eina::accessor from an eina::ptr_array

eina::ptr_array was missing the methods to get an eina::accessor from it.
Added these methods.

eina_cxx: Fix compatibility between eina::optionals of convertible types

Now an eina::optional that wraps a specific type can be constructed or
have content assigned to it using another eina::optional that wraps a
different type, provided that the two wrapped types are convertible
between each other.

Added "disengage" member function to be able to disengage a R-value
eina::optional of different contained type.
It also adds for increased convenience.

Fix constructing an engaged eina::optional from an disengaged one.

Fix small assertion problem of trying to construct an eina::optional
already flagged as engaged.

Fix incorrect use of std::move. Changed it to std::forward.

Added constexpr property for trivial constructors.

Added auxiliary function "make_optional".

Added unit test to check compatibility between eina::optionals of
convertible types.

eo_cxx: Fix signal_connection disconnect crash

Fixed crash when disconnecting event inside of its own event callback.
Instead of deleting the callback object immediately during disconnection
(which causes the callback to be freed), the deletion is now scheduled
for later (using ecore_main_loop_thread_safe_call_async).

Updated some Makefiles to proper include ecore now that it is used in
all event wrappers.

Added a unit test to verify crashes under these circumstances.

ecore_cxx: Fix perfect forward of callback argument

eolian_cxx: Add protected methods and events to C++ wrappers and fixes

Using eina::string_view in eolian generated interfaces (instead of
std::string) to allow lightweight passing of both C strings and C++
std::string.

Also, No longer using eina::optional in generated headers for types
that already implements the concept of null state (like Eo wrappers
and eina_accessor).

Also fix allocating callback objects require by class methods
(i.e. static) in static vectors so the memory will be freed when the
programs exit.

Added a new test case for testing callbacks on class methods.

Moved method definitions and supplementary code from generated C++
wrappers to auxiliary header file (.eo.impl.hh) generated together
with the main ".eo.hh" file. Updated Makefiles to list such files in
the compilation and cleanup processes. Updated .gitignore to include
these new generated files.

Made general adjustments on the documentation of generated C++ wrappers

Added "PREDEFINED" preprocessor macro definition in the Doxyfile.in in
order to make some adjustments for better documentation in the C++
generated headers.  Excluding generation of documentation for classes
in the "eo_cxx" namespace (the namespace for "abstract" eolian C++
wrappers).  Now generating the documentation for the events too.
Hiding some auxiliary code from being documented. Some aesthetic
adjustments for generated white space. Generate documentation for the
main constructor of C++ wrappers and added auxiliary grammars to list
parameters names.

eina_cxx: Add eina::string_view class (non owning reference to string)

Add an implementation of string_view to Eina C++. It is a non owning
reference to a string that allows lightweight argument passing of both
C++ std::string and C strings (const char*) on interfaces.

eolian_cxx: Declaring all inherited events on the concrete C++ wrapper

Added declaration of all inherited events on the concrete type to
avoid unnecessary type conversion on event callbacks.

Note that when two events happens to have the same name, only the
event of the derived class is declared in the concrete class. Access to
such events should be made through the base class type specification.

Fix on forwarding callback object (changed std::move to std::forward).

Additional line break at the end of event handling methods in the
generated headers to improve readability.

efreetd: If we fail to allocate space for subdir_cache, then get out

Summary: Fix coverity CID1294212 (potential Null pointer dereference) as
the alloc for subdir_cache Could fail, however if it does we Were
still trying to use it. In the case that it fails, ERR msg and return.

@fix

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

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

Ector: Fix distcheck.

The generated Eolian headers should *not* be included in the dist
tarball. This makes autofoo go crazy (why would it? ffs) and is
wrong anyway.

Evas object: verify the parent is valid and fail if not.

It seems like before the switch to Eo, all the object_*_add functions
had magic checks that only allowed non NULL Evas pointers to be passed.
Since we switched to Eo we apparently missed the early failure we used
to have, and we instead fail later. This patch should fix that.

Thanks to jaehwan for reporting.

@fix

eina: fix typos in doxygen

build: Make sure the ecore_drm Makefile gets included after its dependencies

This problem have been observed by Scimmia22 on the Arch builds as well as on
the jenkins wayland build job. While compiling works fine the relinking during
make install fails with ecore-drm linking does not find eeze or eldbus as its
deps. This only shows on systems with no efl installed, a build from scratch.

As far as I can see we have all dependencies set correctly in configure as
well as in the Makefile which are working fine even in highly parallel builds.
It was a bit surprising here to me that the include order is still important
with our correct dependencies. Autotools wisdom is welcome here to either
explain to me why this is needed or what the correct fix would be.

The includes all moved before Ecore_Evas because that would use ecore_drm if
enabled.

Fixes T2281

Evas GL common: Properly disable texuv3 when only texuv2 is used

Yet another vertex array fix (GL crash).

Evas GL common: Enable vertex arrays only when requested

This should fix Dave's crash with the GL engine.

Indeed the pointer to the color array is passed directly
to the driver, without memcpy. Then, during glDrawArrays, the
driver will try to address it... but it could as well be NULL or
invalid. So, crashes would occur.

Also mark use_vertex as always true. We keep this field only for the
alloc() function.

Emile: Fix memory leak

The binbuf wrappers were never freed.

Evas GL common: Fix size of vertices data buffer

This actually has more chances of fixing Dave's GL crash.

Evas GL common: Only push color VBO for map when required

Since the new shader selection system is able to check the
value of nomul, we can avoid pushing color when they are not
required (color is opaque white).

Hopefully this will fix Dave's crash (mayyyyybe...)

evas-software-x11: Fix dereference before null check(s)

Summary: This fixes Coverity CID1270028, CID1270029, CID1270030 where
variables RGBA_Image and Native where dereferencing function input
paramaters Before null checking them.

@fix

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

evas-gl-x11: Fix dereference before null check

Summary: This fixes Coverity CID1293519 where einfo was being used
Before it was being null checked (which Could have caused a crash if
in fact einfo Was NULL).

@fix

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

edje: Fix dereference null return value

Summary: Fix Coverity CID1287154. _alloc function can return null
however 'color' was never checked after call to _alloc and thus Could
be NULL.

@fix

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

evas-3d: Fix Resource leak from CID1271635

Summary: This fixes a reported resource leak (by coverity).
_pack_meshes_vertex_data allocates memory stored in 'vertices' however
vertices is never free'd unless count > 0

@fix

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

ector: Fix Coverity CID1293003

Summary: This fixes an issue of coverity reporting Explicit null
dereference as fetchfunc is defaulted to NULL, yet no check is made
later before calling it.

@fix

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

evas: Fix CID1293000 (resource leak)

Summary: This fixes a resource leak of eina_array in
eng_ector_renderer_draw which would return without freeing an
allocated array.

@fix

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

efl: Remove dead configure options for drm-hw-accel

Summary: This option Was added originally so that software drm and
hardware drm could be done in the same 'engine'. Since we have drm and
gl_drm now as separate engines, this option is no longer needed.

@fix

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

efreed cache - fix coverity warning on rm cleanup on sanity check

sanity check didnt check return value of ecore_file_recursive_rm(). do
that. coverity should be happy now.

efreetd - fix loading of extra_icons.dirs and icon.exts to not lose char

efreetd was losing the last char on every line of extra_icon.dirs and
icon.exts thus resulting in gradual degredation of these files as more
and more changes happen to have things like:

.jpeg
.jpe
.jp
.j
.

etc. for extension or dirs like

/home/raster/.e/e/icons
/home/raster/.e/e/icon
/home/raster/.e/e/ico
...
/home/ras
/home/ra
/home/r
...
/ho
/h
/

.. you get the idea. before long the list of extra icon dirs (and
extensions) was massigve and caued all sorts of filesystem rummaging.
this fixes that to no longer degrade these files. this also changes
their names to force new files to be used instead of the broken old
ones.

efreetd - cache generators - dont unlink then rename - fix

if you unlink then rename, there is a gap between the unlink and
rename - no file is there. the point of the rename is to be atomic.
either old file OR new file. nothing in between. this fixes that race.

efreetd - startup slowness fix for recursing directories

this fixes major slowness in things like enlightenment login. when
enlightenment starts, efreetd is started. efreetd takes a long time to
start because it is scanning lots of directories recursively. it is
early in code freeze, so i think this can go in as a fix, even though
it is more of an optimization, but as such it has a fairly major speed
impact, so i consider this fixing a core performance problem for
things like logging in. The solution is a cache file containing all
the sub directories in a directory tree. we still have to stat every
directory, but this avoids a lot of stating of all files as well and
avoids any readirs etc. so it is much faster. on an ssd this comes out
to 4 times faster for efreetd to start up. on an hdd it's about 2x
faster to start uncached.

this should bea good fix for startup times - on my systems thats a 1
second speedup on ssd (out of about 8 seconds boot time) and 3 seconds
on hdd (out of about 39 seconds boot). so w save 1 and 3 seconds
respecively in boot + login.

now can we get this to 0 or close? that's a matter of designing
something like a deferred scan + monitor add so we wait until "startup
is done" then set up in the bg for a bit. that might shave another 3
seconds off boot time for hdd's but for ssd's wil barely blip (maybe
save 0.1 sec). so leave that till normal dev mode.

@fix

efreetd - improve sanity checking for recursive icon monitoring

i found my efreetd was not just monitoring a small set of dirs but
literally $HOME - recursively. this explains a lot of performance
issues i have seen on spinning hdd's vs ssd's. i never knew if was in
efreetd though... until now. for whatever reason my efret cache caused
efreet to try monitro a list like:

/
/h
/ho
/hom
/home
/home/
/home/r
... etc.

when trying to monitor something like:

/home/raster/.local/share/icons

that means it ended up trying $HOME as wellas a vast range of dirs it
just shouldn't sensibly ever try. i have no idea how my cache ended up
this way, but deleting it and re-populating it fixed it. so this ads
code to detect such insanity (as wellas give actual complaints on deep
recursion so we can see things better later). not a bug fix as such,
but a major improvement to detecting bugs which is what we want for
stability.

I also noticed that one legacy pixmaps dir is not recursively
monitored, but another is - so be consistent and recurs both.

Also apply checks to desktop file monitoring too.

evas - gl common engine - dont complain on every surace opt fetch

this causes a complint every single frame - not a good idea.

ecore x vsync - ensure drm fb is set to close on exec - intended

@fix - minor - fixes T2138 as muc as efl can (dri card0 file)

eio-model: Fixed concurrent access to filter_cb and monitoring empty directories

Added a spinlock, since contetion should be non-existant. And removed wrong if.

eio_monitor: Removed unnecessary eio_file_direct_stat

Removed unnecessary asynchronous stat, which could make eio_monitor to
lose events between the monitor creation and stat completion.

eio-model: Fixed crash when files are created in the temporary dir

Fixed a crash when temporary files were created before the tested file, causing a dereference of a NULL pointer.

Edje: Edje_Edit - new part addition should initialise threshold as -1

After adding new part all draggable fields should contain default values.

@fix

ecore_file tests: Added new tests in ecore_file

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric

Subscribers: cedric

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

ecore_evas: remove unwanted binary.

This binary should have been removed before merging the branch. It did
sleep in for no good reason, removing now.