edje: Fix Coverity CID1299026 and CID1299027: Explicit null dereference

Summary: Coverity reports that this potentially is passing a NULL
pointer to strcmp. As the GETSTR macro Can possible return NULL we
should be checking that return before passing it to strcmp.

@fix

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

evas-render2: Fix Coverity CID1306141: Dereference null return value

Summary: Coverity reports that eina_thread_queue_wait was returning
NULL and thus msg variable being used here causes dereferencing a null
pointer.

@fix

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

ecore-input-evas: Fix Coverity CID1306803 (logically dead code)

Summary: This cleans up the _ecore_event_evas_mouse_button function.
Coverity was reporting that the state tests in the 'if' function could
not be be reached due to eel variable being null. This moves the if
(!eel) check to run before the state tests.

@fix

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

evas-gl-drm: Cleanup compiler warnings when building gl_drm engine

Summary: This cleans up nasty compiler warnings when building the
gl_drm engine. If we do not define EGL_EGLEXT_PROTOTYPES then we end
up with implicit declaration warnings

@fix

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

Eo: Fix windows support.

This is heavily based on a patch by Vincent Torri. I just refactored it
a bit so it doesn't break ABI on Linux, only on Windows (where it was
broken anyway).

This patch changes things so on Windows, functions are looked up only
based on their name. Because of the indirection (and export/import
tables) windows does, this is the only reasonable way to make it work.

Eo: Fix a potentially dangerous lack of {}.

You should always use curly brackets. Especially when the inside statement
has its own curlys. This can be confusing and has already lead to bugs in
many projects.

eina/simple_xml_parser: don't parse the <, > in the attribute string.

The tag identifier should not be inside of the string "~".
Previous logic doesn't care that scenario,
the parser context can be corrupted and it will be no more useful.

@fix

eina: add eina_simple_xml example.

This example just prints tag and it's attributes in xml file.
But this example intended for the scenario,
if the attributes have the "<" or" >" characters inside of the string.

efl_control,efl_text: convert docs

efl_model_base: convert docs

Examples: Fix emotion basic example

Examples: fix compilation due to inline

Undefined references were occurring on inline (not static) functions.
Adding static to these functions solves the issue.

ecore-x: move grabbed_is check to multi_handler

Summary:
Previous implementation used mouse_handler to create mouse events from touch
events if the device was grabbed and no emulated mouse events were emitted by X server.
However the same functionality could be achieved by using multi_handler with
additional condition on grabbed devices.

Test Plan: elementary_test -> Multitouch

Reviewers: cedric, raster, devilhorns

Subscribers: seoz, cedric

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

efl_gfx_base: convert docs

evas_test_mesh: 3d -> canvas3d (fixes broken test compile)

evas: change interface evas_3d to evas_canvas3d

Summary:
Regard to https://phab.enlightenment.org/T2479 it is need for right generation
Eolian files

Reviewers: cedric

Subscribers: cedric, tasn

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

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

eina: add eina_quaternion_set

Reviewers: cedric, raster, Hermet

Subscribers: cedric, artem.popov

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

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

evas_canvas3d: cube chaotic was not initialized and relied on luck to be zeroed by default.

Summary: Cube chaotic was rotated, because angle in structure of "SceneData" have random number.

Reviewers: cedric, Hermet

Subscribers: cedric

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

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

ecore-evas-drm: Default to GBM_FORMAT_ARGB8888

Summary: Seems some platforms (my setup here at least) don't have EGL
configs which suppose GBM_FORMAT_XRGB8888, so default to ARGB

@fix

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

evas-gl-drm: Add support for EGL_MESA_platform_gbm

Summary: This change adds support for using EGL_MESA_platform_gbm to
get the display directly from gbm and to create a window surface
(usable by egl) directly from the gbm surface.

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

ecore_input_evas: remove tab and unusable code

remove tab and code which is not used

eolian: remove the outdated and ugly database printer (needs rework)

eolian: check for C type keyword before trying to append as C type keyword

This fixes a segv when non-type Eolian keyword is used.

@fix

eina_matrix: Added test cases for fixed point matrix functions

Summary:
Added test cases for all the functions related to fixed point matrix

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

Subscribers: cedric

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

evas: use the right function with freetype backend.

evas: properly handle cleanup case.

efl_image: convert docs

efl_player: convert docs

efl_gfx_gradient_radial: convert docs

efl_text_properties: convert docs

efl_gfx_shape: convert docs

efl_player: Add playable property

Summary:
Add "playable" property to Efl.Player.
It should be used to check if the currently loaded data can be
played.

Reviewers: tasn, felipealmeida, raster

Subscribers: cedric

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

ecore_imf/wayland: Add NULL check before accessing

 Ecore_input: Add "ECORE_EVENT_MOUSE_BUTTON_CANCEL" event

If ecore get the mouse cancel event(ex: display off, or window stack change) from below, we need to deal with.
Currently, evas also generates up event using evas_event_feed_mouse_cancel function when window hide is called.
if cancel event occurs, ecore also call evas_event_mouse_feed_cancel like window hide situation.
cancel event means all button or touch event should be canceled, but in the future, if we need to deal each cancel event according to touch point,
we need to add api like evas_event_feed_multi_cancel

@feature

Edje: Add doc in edcref for description.clip_to

Thanks Dave for pointing this out.

evas/gl: Add null checks

Summary: Add null check for glGetString version check and evgl_engine

Test Plan: Local tests

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: wonsik, cedric

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

Evas: Remove shader_3d .x generated file from BUILT_SOURCES

This should fix the build after make clean.
Even though those .x files can be re-generated at compile time, they
are checked in in Git and shipped in tarballs. They must not be removed
during make clean.

Thanks Ross Vandegrift for the report.

@fix

Evas GL: Fix error message (ERR --> INF)

evas/gl_x11, gl_cocoa: remove static variable in eng_font_draw

Summary:
When evas is recreated, this static variable value remains,
so it references already freed memory.

Test Plan: Local tests

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: wonsik, mer.kim, cedric

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

Edje: Add test case for "no_render" and "clip_to" state flag

Evas: Add "no-render" flag for proxy sources & clippers

Those objects should never be rendered on the canvas, even if they
are visible. On the other hand, they need to be rendered in mask or
proxy surfaces.

note: this patch includes some extra whitespaces changes :(

@feature

Edje: Add clip as a property of a part description

Since masking, for performance and themeing reasons, it becomes
interesting to be able to switch clippers on the fly. In particular,
switching from an IMAGE mask to a standard RECT clipper can save a
lot of power when masking is not required.

This new flag "description.clip_to" will behave a bit like a mix of
rel.to and visible:

- It points to a part by name, just like part.clip_to. This will
  override the clipper set by the part, or override the default clipper.

- Like "visible", it can not be interpolated between two values, so
  it will switch only at the end of a transition.

- By default there is no clip override, which means Edje will fallback
  to the part's clipper, if any, or the base (group's) clipper.

NOTE:
- Since a clipper that does not clip anything becomes a standard object,
  it is visible and rendered. This will in 99.999% cases not be the
  intended behaviour. Currently we can simply use a transparent RECT
  in order to always have something clipped by the clipper, but this is
  a hack and this will trigger rendering of masks in their surfaces even
  when they are not actually used.
  Ideally, there should be a flag indicating to Edje & Evas that an object
  should be considered a clipper in all situations, and never be rendered
  on screen.

TODO:
- Support Edje Edit
- Add Embryo & Lua functions
- Add support in edje_convert
- Add Edje/Evas flag to mark objects as "no_render"

@feature

eina: add test cases for various eina_matrix operations

Summary:
Added test cases for various operations of 3*3 matrix

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

Subscribers: cedric

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

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

evas-gl-drm: Cleanup gl_drm engine code a bit more

Summary: This cleans up the gl-drm engine code a bit and avoids the
need to constantly destroy and recreate the gbm device itself. This
also adds checks during Outbuf create to handle the swap_mode as per
previously added enum.

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

evas-gl-drm: Move gbm_surface field to Outbuf structure

Summary: This moves the Engine_Info's gbm_surface field to the Outbuf
structure

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

evas-gl-drm: Add swap_mode enum

Summary: This adds an enum we can use for setting/determining the
swap_mode to use for the gl_drm engine

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

evas_event: evas_event_feed_mouse_cancel set EVAS_EVENT_FLAG_ON_HOLD

Evas_event_feed_mouse_cancel generates mouse up event if it meets situation witch mouse should be canceled.
but app also have to know this information. I add cancel information into evas up event generated by cancel function.
@fix

evas/vg : Fixed issue regarding geometry change of vg object.

evas/engines : renamed ector native engine to "default" from "freetype"

evas_smart: Add a check for zero size before malloc

Summary:
Size may be zero while allocating interfaces private data,
when NULL is passed in EVAS_SMART_SUBCLASS_IFACE_NEW for ifaces argument.

If size is 0, then malloc() returns either NULL, or a unique pointer
value that can later be successfully passed to free(). It is implementation
dependent. This case would be likely to occur in case of smart classes with
no interfaces.

@fix

Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>
Reviewers: raster, Hermet, tasn, cedric

Subscribers: SanghyeonLee, sachin.dev, singh.amitesh, cedric

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

edje entry - also filter 0x7f (DEL) char on input as it's invalid entry

@fix

Evas filters: Remove unnecessary copy of buffers

When doing blur from alpha to rgba, there was an extra copy
step added, that was not required.

This should improve the performance a little bit in this situation.

Evas filters: Fix blend with color with rgba buffers

- blend { color = red }
- blur { color = red }
were not working with RGBA input. This patch fixes that.

@fix

edje: Remove unused variable

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

eolian: remove support for old event doc syntax

eo: move all event doc comments to new syntax

eolian: remove support for old doc style on types and vars

evas,ecore_con: more struct/enum doc updates

eolian: fix a bug in expr serializer with wrong sign insertion

edje,evas: convert type docs to new format

eolian/generator: fix enum field doc generator bug

This resulted in a field using the next field's docs rather
than its own, because it incremented the member before
even trying to retrieve the docs.

@fix

Evas: Remove unused variables.

These were introduced in ed01a32c136aaba8fb23a12afaf334b4c47fc62e.

Eo: Add Null check

Summary: Add Null checking when allocate memory.

Reviewers: cedric, tasn

Reviewed By: tasn

Subscribers: cedric

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

edje: add po file support in edc translation.

Summary:
   Implementation to support .po files in edc for translation

Test Plan:
   Test Code to test this implementation is done as part of efl/src/examples/edje/edje-text.c and efl/src/examples/edje/text.edc
   edje_cc -md . text.edc && gcc -o edje-text edje-text.c `pkg-config --libs --cflags ecore-evas edje evas ecore eo`
   ./edje-text

   1) Click On the text "Click here"
      The language gets changed.

Reviewers: shilpasingh, cedric

Reviewed By: shilpasingh, cedric

Subscribers: cedric, rajeshps, govi, poornima.srinivasan

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

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

evas: move evas_object_inject to super's ctor

Summary:
As we always call evas_object_inject in every Evas Object's ctcor,
it seems sensible to move this repeated bit of code to the super
(Evas.Object).

Test Plan: Expedite, Elementary_Test and pretty much everything

Reviewers: cedric, raster

Subscribers: JackDanielZ, cedric

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

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

evas - eo canvas events - move viewport to last to match enums in legacy

Evas GL: Make INF message more explicit

Direct rendering is actually quite hard to enable, so make the
message very explicit for elm_glview users. Thanks zmike for
reporting this msg.

Evas GL: Fix coverity CID 1304559, 1304560

Summary:
<CID 1304559: Logically dead code>
The dead code is only valid for GLES backend, so move if statement
to be used for GLES backend only.
<CID 1304560: Bad bit shift operation>
When calculating depth bit, bit shifting could be done with negative values.
@fix

Test Plan: Local tests

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: wonsik, cedric

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

elua: unexpose "old" documentation api from eolian bindings

elua: remove unused code from lualian

elua: bind recent eolian APIs to eolian lua api

Edje object: Fix remaining Eolian warnings.

Edje object: Fix some Eolian warnings regarding callbacks.

Edje object: Fix Eolian warnings (and migrate types).

Also add an eot file for general edje types.

eolian: add new __undefined_type builtin to silence warnings

eolian: fix wrong type keyword check

Efl model: Fix Eolian warnings (and migrate types).

Evas: Fix abi break in Evas_Callback_Type

You are not allowed to insert elements into an enum, except for at the
end!!!

Spank spank spank.

Was introduced in: 86751fdbaf28796736876e23b86f26480e22de52

Evas canvas: Fix Eolian warnings (migrate types).

ector,ecore,eio,emotion: convert some docs

Eina debug: Fix shadow warning.

This is especially obviously wrong, beacuse the original parameter is not
even used.

Edje_entry: Fix control + (x,a,y,z,m,c) not working issue.

Summary:
When caps lock is "On" and we press control + (x,a,y,z,m,c) none of
these operations work issue fix.

@fix

Test Plan:
1. Caps lock On
2. Input some text
3. Press ctrl+a, ctrl+c etc:-
4. Select, copy, cut etc:- operations does not work

Reviewers: tasn

Subscribers: raster, rajeshps, govi, poornima.srinivasan, navnbeet, subodh6129, cedric

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

ecore: null cb function is unacceptable.

@fix

evas/render2: Put evas_render2_th_main.c in EXTRA_DIST to ship it in the tarball

We are not able to put it in SOURCES as it is not possible to compile this as
standalone object out of the context it is included in.

ector: always initialize the array contents even without fetchfunc

Also use a static array instead of a VLA as we know the size at compile time.

Fixes CID 1294210.

@fix

evas: Remove unused variables

Summary: These variables are actually unused here, so remove them.

@fix

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

ecore-x: Fix compiler warning of unused variable

Summary: XIDeviceEvent variable is only used if we have support for XI2_2.

@fix

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

ecore-x: Fix warning of defined but unused function

Summary: The _ecore_x_input_grabbed_is function is only used if we
have support for XI2_2, so move the defines around a little to fix a
compiler warning.

@fix

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

ecore-x: Fix warning of unused variable

Summary: We only need the 'devid' variable if we have support for XI2_2.

@fix

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

elua lib: sanitize all file paths before writing them

This will prevent random nonsense from being written in.

It changes semantics slightly (documented) and also fixes
CID 1267463.

@fix

eolian: pass rbuf to doc_error

This allows us to correctly free the buffer in error scenarios.

Fixes CID 1304728.

@fix

eina semaphore lock - don't wake up because of signals

@fix - this fixes eina sephamore lock/wait to not return just because
of a signal sent to the process - try again if the semaphore wait is
interrupted by a signal as opposed to a semaphore wakeup.

eina btlog - util - shut up coverity

eina_btlog is a backtrace "cleaner" that takes a raw bt from eina log
and gives a full readable bt for humans. it's run-and-quit binary like
ls or rm - it does its job and exits, so leaks are really of no
concern here...

CID 1297407

evas/evas_op_add_main: Remove superfluous if branch.

We do the same in both branches: return func.

CID: 1267478, 1267480, 1267482, 1267486

evas/evas_op_mul_main: Remove superfluous if branch.

We do the same in both branches: return func.

CID: 1267468, 1267471

evas/evas_op_mask_main: Remove superfluous if branch.

We do the same in both branches: return func.

CID: 1267481, 1267485

evas/evas_op_copy_main: Remove superfluous if branch.

We do the same in both branches: return func.

CID: 1267483, 1267484, 1267487, 1267488

evas/evas_op_blend_main: Remove superfluous if branch.

We do the same in both branches: return func.

CID: 1267469, 1267470, 1267473, 1267474

evas/evas_op_sub_main: Remove superfluous if branch.

We do the same in both branches: return func.

CID: 1267476, 1267477, 1267479, 1267489

ecore x - remove warning on unused variable