edje: Fix shadow variable warning

mapbuf: Make point_color a property

container: Remove method part_name_get

Remove this for now, as it may not apply to all contents.
Note that this was a new API in EO.

efreet: free the allocated buffer before returning

This fixes the commit 169a08c03a6a7270e185bda85d4ab9afd9063c8e (efreetd:
BSD optimizations). Coverity rightly pointed out six different leaks of
various buffers on error paths.

CID: 1374949 1374950 1374951 1374952 1374953 1374954

gl_drm: Fix rendering problems with dmabuf

Somehow this long standing bug wasn't obvious until wayland 1.13.0 made
some additions to an opaque structure.

This changed the frequency that new buffers came to us with the exact
same pointer value of a buffer that had just been freed.

This shortcut in eng_image_native_set has always been wrong - we need to
proceed to the end to make sure we pick up new dmabuf attributes.

efreetd: BSD optimizations.

Summary: Related to ticket T5475.

Reviewers: raster, cedric

Subscribers: jpeg, cedric, raster

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

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

examples/evas: Improve linespacing consistency

Summary:
Also fixes a handful of obvious indentation irregularities,
including some reformatting of some printf() multi-line indents that
commit a71b770b did not properly adjust.

No functional code changes.

Reviewers: cedric

Subscribers: jpeg

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

ecore_evas: Improve linespacing consistency

Summary:
Adds a line between each function so it's more obvious what doc goes
with what API routine.  Reorganize the doxygen elements so they're
consistently ordered and spaced.

No code or documentation changes; mostly just whitespace.

Reviewers: cedric

Subscribers: jpeg

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

evas: adjust pointer mode set to work in headless scenario.

With the wayland backend, it is posible to have no seat connected
until later. This would lead to calling evas_object_pointer_mode_set
and fail without returning a boolean it was impossible to detect it
did fail. This patch correct it.

evas: make sure that even when nothing is rendered there is a pair of RENDER_PRE and RENDER_POST events generated.

ecore_evas: fix link issue on macOS

696ed3e2e8f97de482e98db34997ad1848da985d introduced a build failure on
macOS. _ecore_evas_subregister is being used in a foreign code module,
but it was not exported. Enforcing EAPI gives this symbol enough
visibility to be used outside of ecore_evas.

ecore_input: suppress warning (unused parameter)

eina: fix for escapable charachters not getting escaped if it comes after '\t' or '\n'

Summary:
Escaping is not happening whenever any escapable characters is coming after
'\t' or '\n'. It will also fix invalid read of 1 byte which happens for string where
last charachter is '\t' or '\n' like "eina\t".

Test Plan:
Take a string like "eina\t ". Observe space which is followed by tab is not getting
escaped.

Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
Reviewers: shilpasingh, rajeshps, govi, cedric

Reviewed By: shilpasingh

Subscribers: cedric, jpeg

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

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

ecore_x: add function to show and hide X cursor.

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

ecore_x: handle DPMS state for a more accurate idling evaluation.

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

ecore-evas-wayland: Remove unused variable

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

ecore-drm2: Add start of refcounting for FB objects

Small patch which starts to implement refcounting on framebuffer
objects. This will be needed so that we do not free FB objects while
they are on the screen.

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

ecore_drm2: Move test for atomic_req into atomic flip path

We ended up doing an occasional atomic ioctl when we shouldn't.
This should be a harmless failure, but it's also a pointless one.

ecore_drm2: Fix dmabuf fb destroy

I missed the fact that it'd run through the existing non-gbm path on
destroy, resulting in a (probably harmless) bad ioctl

examples/evas: streamline the intro description statement for examples

Summary:
For people browing through the examples, having the opening statement be
concise and consistent will help them more quickly find what they're
looking for.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Test Plan:
Some of the examples had identical opening statements (e.g. the image
object examples).  I've tried to give each a unique description defining
what they are demonstrating, but you may want to doublecheck I got these
correct.  Of particular note, to me evas-images5.c looks like just a
fixup to evas-images4.c, so I'm not sure what makes these two distinct.

Subscribers: cedric, jpeg

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

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

evas: fix typos in Evas Text docco

Subscribers: cedric, jpeg

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

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

ecore_evas: doc experimental comp_sync API

Summary: (And indicate it's likely to be made private in EFL 2)

Reviewers: stefan_schmidt

Subscribers: stefan_schmidt, cedric, jpeg

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

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

elm_theme: check whether object is valid before calling edje_object_mmap_set

Summary:
In Issue scenario when _elm_theme_set was called,object was already deleted by app causing
unnecessary addition of the style in style_not_found list.
Check for object validity, if object is NULL return theme apply failed.

Test Plan: Make object pointer NULL before _elm_theme_set is called.

Reviewers: cedric, shilpasingh

Subscribers: rajeshps, govi, cedric, jpeg

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

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

examples: use printf instead of fprintf(stdout, ...)

Summary:
Applies same change as e8355c93 for evas, to the remaining examples.
This uses the shell command-line:

    src/examples/evas$ grep -sr 'fprintf(stdout' . | cut -d: -f1 \
        | uniq | xargs sed -i "s/fprintf(stdout/printf(/"

Note that use of the "fprintf(stdout" construct can generate warnings
when -Wformat-security is enabled, if the fprintf statement has no
format arguments, so in addition to the stylistic simplification this
also helps quell those spurious warnings.

Subscribers: cedric, jpeg

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

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

Ecore_Evas (Win32): add geometry_get() method and fix dpi_get(), using the new API in Ecore_Win32.

This fixes fullscreen feature in Elm on Windows as the geometry of the desktop
was not known.

In case of multiple displays, the desktop, where the window is displayed, is used for fullscreen.

@fix

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

Ecore_Win32: add API to retrieve the geometry and dpi of plugged displays

@feature

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

ecore_evas: refactor win32 backend to use generic render infrastructure.

ecore_evas: refactor wayland backend to use generic render infrastructure.

ecore_evas: refactor sdl backend to use generic render infrastructure.

ecore_evas: refactor psl1ght backend to use generic render infrastructure.

ecore_evas: refactor fb backend to reuse generic render infrastructure.

ecore_evas: refactor extn backend to use generic rendering infrastructure.

ecore_evas: refactor drm backend to use generic render infrastructure.

cocoa: refactor cocoa backend to use generic render infrastructure.

ecore_evas: refactor ews backend to use generic infrastructure for rendering.

ecore_evas: first step into refactoring ews backend.

elm_cnp: Fix typo

Summary:
Removes extraneous 'or' in debug message due to change from commit
27375735.

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: bu5hm4n, cedric, jpeg

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

edje: null checking the return value of a function eina_mempool_malloc

Summary:
The return value of the function eina_mempool_malloc was dereferenced without checking. I added the checking code similar to the other codes.
@fix

Reviewers: raster, cedric, jpeg, herdsman, woohyun, stefan_schmidt

Subscribers: stefan_schmidt

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

evas: Fix can't open tiff file on loader

Summary:
Evas can't open tiff file because of no implement in client read api.
I wrote codes simply for open.

Test Plan: self

Reviewers: jpeg, cedric, jypark

Subscribers: stefan_schmidt

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

docs: Fix broken edje tutorial (swallow 2)

elm_code: Move the 'default' font name and size into the theme.

This can still be overridden by the standard method calls

eolian: help and version shall not cause an error exit status

eolian_gen called with --help or --version is a valid action. It shall
terminates with the 0 exit status.

ecore_evas: first step into refactoring x backend.

ecore_evas: first step into refactoring wayland backend.

ecore_evas: first step into refactoring win32 backend.

ecore_evas: first step into refactoring sdl backend.

ecore_evas: first step into refactoring psl1ght backend.

ecore_evas: first step into refactoring fb backend.

ecore_evas: first step into refactoring extn backend.

ecore_evas: first step into refactoring drm backend.

ecore_evas: first step into refactoring cocoa backend.

ecore_evas: use generic function for rendering ecore_evas buffer backend.

ecore_evas: add infrastructure to properly track sub ecore_evas.

ecore_evas: add logic to allow partial refactoring of backends.

ecore_evas: introduce a generic rendering function.

ecore_evas: adjust buffer engine with nicer behavior from evas.

evas: always pair RENDER_PRE and RENDER_POST.

evas: don't double check if the canvas has changed.

elm_cnp: fixup behaviour

On touch devices there is the normal gesture to touch on the screen and
hold until the drag operation started.

For users of a mouse there is the gesture of just click and drag the
mouse away.

This commit changes the behaviour of the start based on the device that
sent the event

ecore_drm2: Add checks for using a plane after death

We keep planes on the plane list to ensure a released plane is removed
from display - however this means that if a caller starts messing with
a plane after release, that it could potentially reposition a plane it
doesn't own anymore.

Use EINA_SAFETY macros to prevent this.

ecore_drm2: Replace plane state release flag with plane dead flag

The release flag is actually less useful than the existing in_use flag
for determining if a plane is unused.  If a new plane is assigned before
the next flip cleans up released planes, then it can point to a released
plane state, and both it and the previous user will be freed on the next
commit, leaking a plane.

Putting the flag in the plane structure fixes this while still allowing us
to keep released planes around to ensure a recently released plane is
cleared from atomic state.

ecore_drm2: Call flip test in plane release

If we don't do a flip test, the atomic state isn't updated.  This fixes
a potential problem where the last operation in state preparation is
a release - the following commit wouldn't include state from the release.

ecore_drm2: Fix flip test error handling

We need to free the atomic req if commit fails, so let's merge these
failure paths and simplify code a bit.

ecore_drm2: Require x and y value in plane assign

Saves us a flip test, and gaurantees that we're always testing with a
reasonable x, y and not something leftover from a previous assignment.

ecore-drm2: Release any marked planes during atomic commit

This patch fixes plane_state values during atomic flip test for any
planes marked for release. When the fb_flip actually completes, we
will remove the marked plane(s) from the output.

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

ecore-drm2: Add 'release' flag for planes

As we cannot immediately remove a plane from an output, due to needing
an atomic commit to actually remove the plane from screen, we can use
a 'release' flag to indicate that a given plane needs removal from the
screen during our next atomic commit.

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

ecore-drm2: Don't remove plane from output list on release

As we need to be able to commit a new plane state for any released
planes, we should not be removing them from the output list just yet.

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

ecore-drm2: Fix formatting

NB: No functional changes

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

ecore-drm2: Don't assign plane to output list if test fails

There is little point in assigning a plane to the output list if the
atomic flip test fails.

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

ecore_drm2: Use stricter cursor plane size check

Unfortunately the plane sized returned from the cursor plane query isn't
a limit, it's an exact size.  Sometimes you can use a different size,
but that's completely hardware dependent - so stick to the advertised
size.

drm-ee: Use a plane for the evas

Assign a plane at startup and update it in flip.

gl_drm: Support atomic updates

Grab a plane and lock it down for displaying the canvas, update it on
page flip.

ecore_drm2: Add API to update the fb for a plane

ecore_drm2: Rebuild atomic state from plane_destination_set

Call a test commit to rebuild the atomic_req

ecore_drm2: Fix atomic flip with no new buffer

We don't have an atomic state, so we need to create one before the flip

ecore_drm2: Pass user data to atomic flip

We need the user_data to come back to us in the flip handler like it does
in the non atomic flip.

ecore_drm2: Add some atomic state tracking

I think we're now at the point where the two paths are merged.

Still no atomic functionality because nothing assigned the primary plane,
so we have no atomic state to commit.  The machinery should be in place
though.

ecore_drm2: Handle atomic state in _release_buffer

Further merging of atomic and non-atomic paths

ecore_drm2: Make _release_buffer take a state struct instead of fb

Lets us push the NULL set into _release_buffer for some clean up

ecore_drm2: Refactor common code from non-atomic path

We'll need to perform all this for atomic operations as well.

ecore_drm2: Fix typo in comment

ecore_drm2: treat atomic flip without a req as an error

We should use the safety macro here instead of silently continuing so we
can get something in the log if this happens.

ecore_drm2: remove fb parameter from _fb_flip()

Time to start smashing the atomic and non-atomic paths together.

ecore_drm2: Replace output fbs with state structs

next, pending, and current are going to have to deal with atomic state
instead of just fbs soon

ecore_drm2: Move atomic state into an output state struct

This should make it easier to share code paths between atomic and non
atomic operations.

ecore_drm2: Perform test flip during plane assignment

Make sure we can commit that plane at assign time so when we hook up to
the scene graph it knows when it can safely use a plane.

ecore_drm2: Drop static from _fb_atomic_flip_test()

We're going to need this one in plane assign to test commits

ecore_drm2: remove flip test from commit path

We'll be doing tests as we build up plane state assignment.  it's too late
to do anything about it if we fail here - failed tests will block plane
assignment in the first place so the scene graph knows it still has to
render those visual elements.

ecore_drm2: Add ecore_drm2_plane_release to release planes

Opposite of plane assign.

ecore_drm2: Store output in plane structure

This will simplify a bunch of API that would otherwise have to pass in
both output and plane - and in some cases we might not have the output
handy anyway.

ecore_drm2: Remove next, current, pending from plane state

These will be output state eventually

ecore_drm2: Assign planes in ecore_drm2_plane_assign

Don't just test, allocate the resource, and then prevent further
assignments from trying to use it as well.

ecore_drm2: Don't use drmModeAtomicMerge

We're creating the entire state from scratch already - trying to merge
with the old state will likely bring in state we just tried to replace.

ecore_drm2: Remove extra #ifdef

It's the same as the previous one, so combine them.

ecore_drm2: Fix typo in error message

Stop printing "commit test failed" from non-test commits.

ecore_drm2: Add an env var to disable atomic pageflips

Allows testing non-atomic paths without a reboot or a rebuild.

ecore-drm2: Do atomic commits per-output

In cases where output monitors have different frequencies, we need to
be doing atomic commits on a per-output basis. This patch modifies the
ecore_drm2_fb_flip function to support doing atomic commits per output.

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

ecore-drm2: Free output atomic request on destruction

When we free an output, make sure we cleanup any existing atomic
commit requests

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

ecore-drm2: Remove newly added atomic API functions

As we need to do atomic commits on a per-output basis, these 2 newly
added API functions can go because these functions did one atomic
commit for all outputs

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

ecore-drm2: Remove #ifdef for HAVE_ATOMIC in plane_assign function

As there is nothing inside this function which requires any Atomic API
calls, this #ifdef can be removed and the function can then still be
used to assign Primary planes for non-atomic use cases.

@fix

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