Revert "Eo: Fix deref after free."

Coverity found the issue, but it's an issue we've already fixed in the
past. I don't know how it got lost, but it seems like someone did a bad
merge. Probably when migrating to Eo2.

This reverts commit 831c20464d212fde8d6ee125342e90e5e64efc1e.

Eo: Fix deref after free.

In some rare cases it was possible for a pointer to be referenced after
it was already freed. This is now fixed thanks to coverity.

@fix

CID 1039898

edje_calc: Document some intended fall through cases in switch-case

We share code here for different types. Make a note and also marked them as
false positive in coverity.

CID 1039377, CID 1039376

ethumb: Memset whole contex in md5 implementation

I don't know what they tried here but if we consider the content
of ctx sensitive we should memset the whole thing.

CID 222382

ecore_imf: Account for terminating null byte on copy

strlen retruns the count with the terminating null byte but strncpy expects
length with it. Account for that.

ecore_imf: Avoid potential buffer overrun

We can't just blindly copy the full string into a 128 byte buffer. Adding
some check here to actually stay below the buffer limit or return.

CID 1193233

fix ecore_thread_global_data_wait to work when no data exists yet

@fix

this is based on feedback from wonil choi, where this didn't work
right. this also has an if check for the hash itself.

fix behavior of edje entry to match other toolkit entires

this fixes T1229 in the part about the cursor behavior when you press
updown/left/right when a selection is there. @fix

edje_calc: Fix invalid parameters during first calc

In particular, ellipsis is -1 by default in Evas, but at this
point (first layout calc), the parameters used for recalc are
incomplete and ellipsis would then be 0 by default (calloc).

As a consequence, Edje will call ellipsis_set(0) enabling
ellipsis even on objects that force "ellipsis: -1".

Solution: set all the parameters before entering text/tb calc.

I believe the other changes are only color and image padding
and should not affect recalc_single.

fix ecore_con_url using 100% cpu and just let fd handler do work

this fixes T1251 - @fix

evas/wayland_shm: unlink the REAL temporary file

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>

Eolian/Generator: fix the class description generation.

The class description was not used during the generation of the legacy
and Eo headers.

Evas filters: Fix NULL dereference (legacy_strdup)

CID 1210809
CID 1210813

+ remove comment

Evas filters: Fix NULL dereferences

CID 1210807
CID 1210808

Evas filters: Remove logically dead code

CID 1181889

edje/Edje_Edit: _edje_edit_real_part_add init field of Edje_Real_part

Summary:
To correcly display the added pats using the edje edit api, update
init the filds of Edje_Real_Part. Initialize the next filds: type and
typedata. It filds initialized accordingly to Edje_Part_Type value.

@fix

Reviewers: raster, cedric, seoz, Hermet

CC: cedric
Differential Revision: https://phab.enlightenment.org/D854

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

edje_calc: Don't add padding twice to text objects

Text objects declared in Edje will see their padding added twice,
as the Evas_Object_Text itself contains the padding already.

This WILL break some EDC files. It's a bug nonetheless.

Should this be backported?

edje_text: Remove no-op pad_get

This call to pad_get is useless.

edje_calc: Update some disabled code so it can compile

Yup, that's basically a no-op but allows other devs to test
the old behaviour of that function as it's still there:
_edje_part_recalc_single_text

add soem class description to eo timers to test if it gets generated

de-elipsis edje

the word is "ellipsis". it's bad enough that we have released misspelled api, but we can at least spell things correctly internally.

edje_cc now prints helpful warning messages when it compiles broken text parts

edje_cc should throw errors when inheriting a part from the same part

release: v1.10.0 beta2

Revert "eolian-cxx: re-enable Makefile_Ecore_Audio_Cxx installation and tests."

This reverts commit 1e592d22b77f1e818e0bc4d1350abf59f146dcc2.

ecore-drm: Send proper keycode to ecore key event, and remove
erroneous variable.

@bugfix: Since later functions which process the ecore key event will
end up subtracting X's broken keycode values from This original
keycode value, we had better send the proper code with the offset
already added.

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

evas/evas3d: reset image properies if the image is used for scene.

eo ecore timer - improve documentation in the eo file

evas/examples - revised evas-3d-proxy

evas/evas3d: discard proxy textures list properly.

this fixes evas-3d-proxy example to be exited correctly.

evas/evas3d: fix proxy texture source set.

the appeneded texture should be object itself but not private data.

Compilation: align EOLIAN, EDJ with CC and others

Hate seeing those unaligned things:

  EOLIAN   blah
  EDJ   blah
  CC      blah
  CC      blah

OCD mode :)

Evas: Fix abort() when using ETC2 with alpha

Ecore evas: Fix crash after image_data_set

In some cases, the image has a format different from RGBA,
so image_data_set will be invalid as the new data is expected to
be RGBA. This happened with ETC2 images.

evas/evas3d: applied eo intensively and fix indentation.

ecore-x - fix setting protocol atoms for icccm for 64bit

@fix

evas: remove logically dead code.

CID 1211984.

Note that in error case, the calloc is either not passed or
returned NULL so free will never be called.

evas: fix uninitialized value usage in Evas_3D.

This code look suspiciously similar to src/lib/evas/canvas/evas_3d_mesh.c
line 682.

evas: move the type and enum of Evas_3D into the unstable part of the API.

I am concerned that we do duplicate the color format of a texture insteed
of reusing the colorspace enum from Evas (that already exist). I believe
this is a side effect of not using Evas_GL_Image for texture.

This let me think that I wont allow this API out of BETA until we reduce
duplication by using Evas_GL_Image as it seems it will also impact the
public API of Evas_3D.

evas: fix uninitialized value in Evas_3D.

CID 1211986.

evas: remove left over Evas_3D public/released API.

evas: fix use of uninitialized value in Evas_3D_Mesh.

evas: fix uninitialized value being used when the engine doesn't support Evas_3D.

CID 1211988

autotools: define @datarootdir@ explicitly to avoid config.status warnings.

Summary:
This patch removes warnings like this:

config.status: WARNING:  '/home/savio/src/efl/pc/eo.pc.in' seems to ignore the --datarootdir setting

Reviewers: cedric, JackDanielZ, smohanty

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D843

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian-cxx: remove annoying compilation warnings and messages

Summary:
- remove an automake warning caused by a variable redefinition
- remove unnecessary verbosity during debug compilation

Reviewers: cedric, smohanty

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D844

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

redo lazEDC parsing a bit to fix desc.state omission

lazEDC missing keywords for scale/noscale

fix be9b27ac99dc4fcb49c340b8bdf4bfcbe4240e5f for edje_cc

do_params is set when expecting parameters, not req_params

lazEDC syntax update: imprecise -> noprecise

ecore_x_image_get() should not abort when shm creation fails

...also when shmat() fails

ecore-x should print an error when shmget fails

examples: fix eolian_cxx Makefile.am to work with --with-eolian-cxx.

Summary:
Includes Makefile_Eolian*_Helper.am and -- since
_EOLIAN_GEN_DEP and _EOLIAN_CXX_DEP can't be resolved from
src/examples -- redefine the generation rules locally.

Reviewers: cedric, stefan, stefan_schmidt

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D836

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eina-cxx: add EFL_PTHREAD_CFLAGS to EINA_CXX_CFLAGS and EFL_PTHREAD_LIBS and -lm to examples

Summary:
EFL_PTHREAD_CFLAGS are needed for tests and users that use
efl::eina::thread's and EFL_PTHREAD_LIBS to eina_cxx, eo and evas examples.

Reviewers: cedric, stefan, stefan_schmidt

CC: cedric, savio
Differential Revision: https://phab.enlightenment.org/D832

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian_cxx: pass eo_class by reference instead of by value.

Summary: This patch gets rid of a "defect" pointed out by Coverty Scan -- as per jackdanielz request.
CID 1211985.

Reviewers: cedric, smohanty, JackDanielZ

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D840

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian-cxx: re-enable Makefile_Ecore_Audio_Cxx installation and tests.

Reviewers: cedric, stefan, stefan_schmidt, smohanty

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D839

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian: fix pkg-config eoincludedir variable to point to @datadir@ instead of @datarootdir@.

Summary: .eo files are installed at $(datadir)/eolian/include/ actually. The set-up fails when configure --datadir is different from --datarootdir.

Reviewers: cedric, JackDanielZ

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D838

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eina-cxx: remove unused parameter names and fix function without return in examples.

Summary:
Fix function with return type and not returning any value and
remove warnings about unused identifiers in examples for eina C++.

Reviewers: cedric, stefan, stefan_schmidt

CC: savio, cedric
Differential Revision: https://phab.enlightenment.org/D837

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian_cxx: removes C++ examples when C++11 isn't supported

Summary:
Added HAVE_CXX11 guards to Makefile*.am files for C++ binding to avoid
compilation errors for examples when C++11 isn't supported. This also
disable installation of all EFL CXX pkgconfig files.

Reviewers: cedric, stefan, stefan_schmidt

CC: cedric, savio
Differential Revision: https://phab.enlightenment.org/D831

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian-cxx: fix #include directives generation

Summary: Fix eolian_cxx #include directives generation for relative paths

Reviewers: cedric, stefan, stefan_schmidt

CC: cedric, savio
Differential Revision: https://phab.enlightenment.org/D833

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

Evas test: Update filters tests (Lua) - part 2

Rendering & padding tests

Evas test: Update filters test (Lua) - part 1

Evas filters: Fix some parsing errors

Evas doc: Add example for padding_set (filters)

With preview and all.

Evas doc: Move filters reference doc to Lua

Part 2: improve doc design, translate examples to Lua syntax and
        add a few words about Lua itself.

Evas doc: Move filters examples to Lua

Part 1: auto-generated examples with previews

gitignore: Ignore PNG files for the filters' doc

evas-wayland-shm: Fix 'oopsie/typo' in temporary file name template

@bugfix: When creating a temporary file for the buffer's mmaped data,
we need to make sure there is an appending '/' in the name else
mkstemp will fail due to improperly formatted filename.

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

Eo: Rename descs2 to descs (remnant of the Eo2 change).

Eo: Improved docs a bit.

evas-wayland-shm: Use XDG_RUNTIME_DIR if available for creating mmap'd
buffer file.

@bugfix: When we create the mmap'd file for shm buffer access, try
using the XDG_RUNTIME_DIR first as the place to create the file. If
that does not exist in the environment, Then fallback to using /tmp
directory.

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

ecore-x: Set keycode in ecore_event structure during key press/release events

bugfix@: As we have support in the Ecore_Event structure for storing
keycode on key press/release, let's utilize it. This will simplify
wayland compositor code as we won't have to parse key names to get the
keycode.

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

ecore/example: Modify the explanation of ecore evas extn plug example.

call edje_cc link combining when popping the group so the last group doesn't not get linked

autotools: make it possible to forcibly disable neon.

eo: improve error reporting, update tests

eo: improve error reporting for _eo_api_op_id_get()

silence a few warnings

eolian_cxx: fix install-examples and (post-install) compilation.

Summary:
This patch fixes T1226 by adding a Makefile.examples to
examples/eolian_cxx. It also fixes a bug in bin/eolian_cxx: the
include paths were not being correctly generated for directories
outside EFL tree.

Reviewers: cedric, smohanty, stefan_schmidt, stefan

CC: uartie, wayland-efl, felipealmeida, raster, woohyun, cedric
Maniphest Tasks: T1226

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

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian_cxx: fix compilation of examples out-of-tree

Summary: Changed erroneous top_builddir for include paths for eolian_gen tool.

Reviewers: cedric, stefan_schmidt, stefan

CC: cedric
Differential Revision: https://phab.enlightenment.org/D830

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

fix emotion_test signal emit object

@fix

do not use EINA_FALSE instead of NULL

evas/evas3d: removed export of evas_3d_node_add() and apply eo to examples.

but still i don't like eo apis used in app side (if it's only way)
we should export legacy apis for convenience.

lets decide this policy later.

rg_etc: Add README to EXTRA_DIST

rg_etc: Add a README for the Zlib license

evas/evas3d: call eo_unref() after eo_add().

since eo default ref count is 1 and increased by canvas parent again,
they should be unref() called if they wanna deleted automatically.

I wonder if evas unref the objects while it removes the objects from it's list,
then user don't need to call eo_unref() manually which may cause a mistake in usage.

evas-3d: Fix typo in evas-3d-pick.c

Summary: Fix typo with eo_do in evas-3d-pick.c

Reviewers: Hermet

Reviewed By: Hermet

CC: cedric
Differential Revision: https://phab.enlightenment.org/D828

eolian_cxx: Fix out of tree build

Passing a relative path prepending "./" is generally the wrong thing to
do in autotools because the auto vars may be absolute. Fix build:

...
ERR<17525>:eo_toknz lib/eolian/eo_lexer.rl:1325 eo_tokenizer_database_fill() unable to read in .//home/lucas/p/e//efl/src/lib/ecore/ecore_poll.eo
Couldn't load input file: .//home/lucas/p/e//efl/src/lib/ecore/ecore_poll.eo

evas/evas3d - adjust examples window size.

evas - removed stdlib.h math.h header include in each source files.

It's included in the evas_common_private.h.

evas/evas3d: don't be crash when it's running on sw backened.

It doesn't have a plan that evas3d supports sw backened in the near future,
let it prenvent engine function access in case of sw backened.

evas/evas3d: set gl backened forcely in the examples

Right now, evas 3d doesn't support software renderer,
we set the gl backened forcely.

Eolian_CXX: Fix generation warnings.

Since there was a bug in Eolian (Spank on you Savio that you didn't tell
me :-)), the cxx generator needed some workaround that is no more
mandatory now.

Eolian: fix type resolving for implement functions.

The type of the virtual functions implemented in a class was let unresolved
and led to inconsistency when working with them.
In fact, the type can be resolved easily after all the classes that the
specific class inherits from are parsed and stored in the database.

Fix T1228

eolian_cxx: remove warnings from bin and lib.

Reviewers: cedric, smohanty, stefan_schmidt, stefan

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D826

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

examples: fix the include paths of eina example.

Summary:
"make examples" was failing for parallel build set-ups (i.e.,
top_builddir != top_srcdir).

Reviewers: cedric, smohanty, stefan_schmidt, stefan

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D825

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

autotools: make it possible to disable support for C++11.

edje entry no longer selects none with unimplemented ctrl+w key press

edje entry no longer prints control codes

this has been an annoyance for way too long which has required various hacks to suppress

Eo gdb: Removed unneeded eo_backtrace and fixed eo_step for eo2.