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

evas render2 work - begin to make rectangles deal with render 2 basic
infra

Edje: Fix indentation after a clean-up commit

@zmike apparently wanted to beat the b0rker in chief,
(see 172a0ccf0cd6) but edje_text.c came out worse than before :)

eo_do() seems to totally mess with editors' auto-formatting in
some cases.

Evas: Fix shadow warning

ecore_imf/wayland: support on-demand show mode

In on-demand show mode, input panel will be shown in case of only an user's explicit Mouse Up event.
It doesn't request to show the input panel even though the Input Method Context has focus.

ecore_x_xi2: remove useless define check XI_TouchXXX.

Summary:
we can use XI_TouchXXX defines after xi2_2. So we don't need check XI_TouchXXX defines.
I think it is simple only using the ECORE_XI2_2 define instead of using XI_TouchXXX defines.

ecore_x_xi2: remove useless define check XI_TouchXXX.

Summary:
we can use XI_TouchXXX defines after xi2_2. So we don't need check XI_TouchXXX defines.
I think it is simple only using the ECORE_XI2_2 define instead of using XI_TouchXXX defines.

Reviewers: raster, cedric, devilhorns

Subscribers: cedric

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

edje: reformat all C files

this has been unreadable for years due to the french tabber.

Revert "edje: set GROUP/SWALLOW/EXTERNAL pass events based on mouse events"

This reverts commit c38f84e64f3ac915d61c0307fc9dce0c66fda7b8.

apparently many existing edje groups were written with the assumption that
this was broken, so fixing it is impossible at this point

edje: set GROUP/SWALLOW/EXTERNAL pass events based on mouse events

previously these parts would fail to consume mouse events as expected,
leading to strange event chains which were inconsistent with other types
of parts

@fix

ecore: move eo docs to new format

eolian/generator: use get/set doc as primary if needed

eina: fix copy and paster error detected by Coverity.

CID 1302701

eolian/generator: support for since tag in doc gen

modules/evas/engines/software_generic: No need to check enums for smaller 0

Evas_Image_Orient is an enum starting at 0 thus we never go below 0. Remove
unneeded checks.

modules/evas/engines/software_generic/evas_engine.c:1558:4: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
modules/evas/engines/software_generic/evas_engine.c:1560:8: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]

emile: Init ptrag as it might get used uninitialized if not

lib/emile/emile_image.c:2099:49: warning: 'ptrag' may be used uninitialized in this function [-Wmaybe-uninitialized]

eavs_3d: Avoid enum check for < 0

Evas_3D_Mesh_Primitive is a enum starting from 0 so it never will be below 0.
Just as the compiler tells us. Remove unneeded check.
lib/evas/canvas/evas_3d_primitive.c:48:4: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]

ecore-wayland: Fix T2466: Update xdg shell protocol code

Summary: This updates the xdg_shell protocol and code to support
XDG_SHELL version 5. This allows EFL/Elm apps to function again in
Weston.

@fix

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

eolian: support for @since tag in new doc comments

This is kind of ugly, but I didn't see a better way to do it without turning
this into a much more complicated parser. So this will have to do unless we
find something else to add, in which case a new doc parser will be required.

@feature

eolian: we need import_types_ref.h included in tarball

If we don't, distcheck fails (because of missing file).

@fix

eolian/generator: fix a false positive in eolian types_gen

While it is true that 'strrchr' may return NULL, in this case it
never will (there is always a newline). Either way, silence Coverity.

Fixes CID 1304558.

eolian_cxx: initialize all members in a struct correctly

Fixes CID 1298133.

@fix

eolian: silence static analysis

Add some unreachable returns to fix CID 1304557.

evas_clickable_interface: adding new event

evas: porting evas smart callbacks to eo

emotion: porting evas smart callbacks to eo

edje: porting evas smart callbacks to eo

It's the first stage of replacing calls of evas_object_smart_callback_add/del/call
with eo_callback_add/del/call.
This will reduce the use of legacy events and callbacks lists in
smart_object which in turn will improve performance and reduce memory
use.

evas-gl-drm: Trap for invalid gbm bo

Summary: If we fail to lock the front buffer, we will not get a valid
bo returned so we should not be calling other functions which require
a bo.

@fix

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

eo: move some eo files to new doc syntax

eolian/generator: generate @ref foo for @foo

eolian/generator: use more correct indent offset in types doc gen

eolian: remove support for short doc comments (//@)

eolian/generator: fix silly shadow warning

eina: Fix warning about unused paramaters

@fix

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

eolian/generator: @param[in,out] is the correct inout syntax

Merge branch 'devs/devilhorns/gl_drm'

This merges a refactored evas gl_drm engine which actually functions now :)

evas-gl-drm: Fix issue with swap mode returning wrong value

Summary: This fixes a crashing issue in evas image cache due to the
engine returning an improper swap mode.

@fix

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

evas-gl-drm: Refactor gl_drm engine to not use dumb buffers

@fix

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

ecore-evas-drm: We don't need to feed mouse events here

@fix

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

ecore-drm: Don't free an output on unplug

Summary: We should not be freeing an output when it gets unplugged (as
it may get replugged). Just disable rendering on that output

@fix

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

evas-gl-drm: Create eglContext before eglWindowSurface

Summary: We should be creating the eglContext before trying to create
the window surface.

@fix

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

ecore-evas-drm: Use XRGB format for gl_drm canvas

@fix

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

evas-gl-drm: Readd support for pre_swap/post_swap callbacks

Summary: The pre/post swap callbacks were forgotten when doing the
engine rewrite, so readd them

@fix

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

ecore-drm: Don't crash when trying to disable an output

Summary: If we call these functions with an already disabled output,
we crash. Check for an output being 'enabled' before we make these
calls.

@fix

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

ecore-evas: Change gl_drm to be the proper engine name

@fix

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

ecore-drm: Remove erroneous error messages from EINA_SAFETY checks

Summary: As the gl_drm engine code will almost always pass NULL
rectangles, remove the EINA_SAFETY checks so that logs don't fill with
meaningles error messages.

@fix

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

ecore-evas-drm: Update ecore_evas_drm for recent evas gl_drm changes

Summary: Previous Ecore_Evas drm code would create the gbm device and
surface inside the gl_drm function. As these are now handled inside
the evas-gl-drm engine itself, this code is not needed here.

@fix

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

evas-gl-drm: Rework gl_drm engine to function again

Summary: Previous gl_drm evas engine code did not work properly (or at
all really). This reworks/refactors the gl_drm engine code to work
again with the changes made to ecore_drm.

@fix

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

evas-gl-drm: Remove old engine code

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

eolian: don't emit an extra empty line in certain cases

eolian/generator: generation for new documentation system

This commit adds the necessary generator logic to emit doc
comments from the new doc syntax. Old doc comments are kept
in for the time being as they're used within the EFL but they
will be removed eventually. This new generator focuses all the
important code in one place, making usage easy.

@feature

eolian-cxx: Removed unused function

Removed unused function and consequent warning

ecore-x: add new grab touch devices functionality.

Summary:
EFL currently supports pointer grabbing. This patch introduces new API
allowing to grab all slave touch devices registered in X server
Grabbing is performed by XIGrabDevice function from XInput 2.0.

By default ecore_x_input_touch_devices_grab grabs all XISlavePointer devices,
having XITouchInfoClass. Function returns EINA_TRUE if at least one touch device
was successfully grabbed. ecore_x_input_touch_devices_ungrab ungrabs all
previously grabbed devices.

To process events correctly change has been done in x_input_handler to emulate
mouse pointer events. If XITouchEmulatingPointer flag is set on touch events and
device is grabbed framework will generate mouse events. This is required
due to X Server design in which mouse events are no longer send to client when device is
detached (grabbed) from virtual core pointer.

@feature

Reviewers: cedric, raster, devilhorns

Subscribers: seoz, cedric

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

eolian: double newline ends summary, not single

This allows you to split the summary on multiple lines
if necessary (for example to satisfy line limit)

@fix

Evas 3d texture: Fix some Eolian warnings.

Evas image: Fix some Eolian warnings.

Evas canvas: Fix some Eolian warnings.

eina: add test cases for eina_matrix3 APIs

Summary:
Added test cases for eina_matrix3_values_get, eina_matrix3_values_set,
eina_matrix3_equal, eina_matrix3_type_get APIs

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

Reviewed By: cedric

Subscribers: cedric

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

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

evas: silence warning.

So I don't like this data being void and stride being in bytes, but changing it
for silencing a warning will be a massive change. I will put that on the major
cleanup that Evas_3D need to receive.

eolian: remove prototype of non-existent API

eolian: add documentation handling API

This adds new APIs that deal with the new documentation syntax
provided by Eolian. The old doc comment APIs are considered
deprecated from now on and will be removed.

@feature

Edje: Add have_selection check to avoid unnecessary function calls

Summary:
Check for have_selection flag when left/arrow keys are pressed to jump
across the selected text, to avoid unnecessary function calls of
comparision, copy of sel_start and sel_end pointers.

Test Plan: Press left and right arrow keys on selected and normal text.

Reviewers: herdsman, tasn

Subscribers: SubodhKumar, navnbeet, cedric

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

evas: for aabb add interpolation frame when base frame is not found

Summary:
Old algorithm searched specified frame of the base frame and when it could not find the specified frame it ignored. So the bounding box was not synchronized with the object.
New algorithm always returns specified base frame or frame interpolation between the nearest base frames. That does synchronizes mesh object and his bounding box.

Reviewers: Hermet, cedric

Subscribers: cedric

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

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

ecore_con: fix error handling case.

Since the move to Eo, in case of error we do not need to cleanup anything when
finalize finish. The kill function was arranged to do so, but it was still emiting
an event exposing the invalid Eo object. Looking closely at the code, that should
have not happened, as there was no symetrical ADD event and the function would do
nothing else. So removing it. This prevent any potential SEGV from a function that
would catch the DEL event.

@fix

eolian: properly fill and free all doc fields

eolian: start exposing Eolian_Documentation

evas_3d: fix coverity scan CID 1302703 and CID 1302702

Reviewers: Hermet

Reviewed By: Hermet

Subscribers: cedric

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

emile: fix region load for jpeg image with orientation different set.

Summary:
If you try to load the jpeg image with an orientation mode defined
using elm_photocam, you can see the broken image(in canse of 90 degree)
or even segmentation fault can happen (in case of 180,270 degree)

@fix

Test Plan: photocam menu on elementary_test

Reviewers: Hermet, cedric

Subscribers: cedric

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

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

eolian: initial test file for docs (no APIs to test yet)

eolian: enums can't be opaque, check properly

Edje object: Fix more Eolian warnings.

eolian: preliminary parsing of doc syntax

evas/gl_common: Reset mvp matrix only for shaders that are being used

Summary:
Evas has quite a number of shaders, and resetting the projection matrix
for all shaders takes some time, which is an overhead when target surface
is continuously being changed (ie. when using proxy).
So here we reset the projection matrix for shaders that are actually used.

Evas GL: support surfaceless make current

Summary:
Evas GL now supports surfaceless make current, where
evas_gl_make_current can be called with sfc parameter NULL.
This closely resembles EGL_KHR_surfaceless_context extension,
where applications that only want to render to client API targets
can make current to NULL surface instead of creating a dummy egl surface.
@feature

Evas GL: Skip surface buffers create/allocate for pbuffer with NO_FBO

Summary:
When pbuffer surface is created with EVAS_GL_NO_FBO config,
FBO should not be created/allocated.

Evas GL: Match FBO config with window surface config

Summary:
When direct rendering is enabled, FBO configuration should match
window surface configuration as FBO will be used in fallback cases.
So create FBO with configuration from window surface.
@fix

Evas GL: Pass correct attachment enum for glDiscardFramebufferEXT

Summary:
When default framebuffer(0) is bound, attachment should contain
COLOR, DEPTH or STENCIL for glDiscardFramebufferEXT.
When a framebuffer object is bound, attachment should contain
COLOR_ATTACHMENT0, DEPTH_ATTACHMENT or STENCIL_ATTACHMENT.
This should be correctly taken into account for indirect rendering,
where internal FBO is used.
@fix

Evas GL: return correct context for context_get with indirect context

Summary:
When evas gl falls back to using indirect context,
evas_gl_current_context_get() should return the indirect context.
@fix

Evas GL: Separate EGL and GL extension lists

Summary:
Separate EGL extensions from GL/GLES extension list, since
we have extension list for each GL version, and we do not want to
check EGL extensions differently when different GL versions are used.
This also simplifies extension string get function as we just need to
concatenate EGL and GL extensions rathan than keeping track of
GL extensions only.

Evas GL: Share texture id in case EGL image is not supported

Summary:
When EGL image is not supported, Evas GL should share texture id
to the gl backend for indirect rendering to work.