Ector: remove use of #warning directive as TODO.

There's already a fixme in code, and an error output at runtime.
As discussed on the ML, using #warning this way is bad.

eio - fix crashes due to stale eio files in tracking list

so eio_eet didnt free fiels the same as the rest of eio. it thought it
was special. it thought it could just go free() it's objects (even
though they inherit the core Eio_File object type in their structs).
everyone else ended up calling eio_file_free() EXCEPT eio_eet. so the
eio_(long)_file_set() funcs ewnded up adding this eio file to the
tracking list via eio_file_register() but never unregistered because
they really liked to just do their own thing anyway...

BAD CEDRIC! SPANKING TIME

http://33.media.tumblr.com/3422c76c33c3b9b045f623ff73e0bf8d/tumblr_mhvu61N9br1rbavngo1_500.gi

so this unifies all allocation to now use a single allocator (that
also ensures all eio async io objects are zero'd out), and a SINGLE
free path, and then it all works. no more valgrind complaints on e
shutdown/restart whilst eio "things" were used before or ... maybe
still around.

this should fix T2129

so try again "git master" after this commit.

@fix

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

emotion gst1 module - remove printf

this debugging printf seems to have been left around - remove it so we
don't get unexpected noise in gst playback

emotion - gst1 module - handle long standing bad map/unmap of gst buf

this fixes a pending XXX where we didnt keep the buffer mapped as long
as the evas image object pointed to the video pixel data. this fixes
this along with less over-zealous refinf to make things more obvious
(now last_buffer actualyl really matters and if it was mapped and
refed).

@fix

Evas: Fix memory leak

Fixes CID 1293000

Evas tests: Add callback_del order test

Evas: Remove callbacks in reverse order with callback_del

This function is deprecated, because del_full should be used instead.
Still, the documentation specifies in which order the callbacks should
be deleted.

Fixes T2077 (not really fixes according to the comment but fixes
according to the doc).

Evas tests: Add debug logs telling people to ignore ERR

Some of the ERR messages are expected and it's part of the
test suite to test for invalid cases (parsing stuff).

Eina tests: fix warning about pointer signedness

Eina tests: suppress warnings over deprecated functions

Evas GL_X11: Fix memory leak

bindings/eo: Only install Eo.hh once to avoid failing double install

Eo.hh is listed in both dist_installed_eocxxmainheaders_DATA and
dist_installed_eocxxheaders_DATA this leads to a situation where they file
should be installed but already exists and thus the installation fails.

00:32:27.919 /usr/bin/install: cannot create regular file '/srv/build/workspace/changely_efl_configure_xcb/destdir/include/eo-cxx-1/Eo.hh': File exists
00:32:27.923 Makefile:32620: recipe for target 'install-dist_installed_eocxxmainheadersDATA' failed
00:32:27.923 make[4]: *** [install-dist_installed_eocxxmainheadersDATA] Error

Given that it is the only item in dist_installed_eocxxmainheaders_DATA I suspect
it was but into the other rule by accident.

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>