embryo_cc - coverity mem access issue - fix

this fixes CID 1039316 in case error num is beyond errmsg bounds

embryo_cc - fix coverity buffer overrun

fixes CID 1039317

fix coverity leak detection in ecore_evas_x hints...

this fixes CID 1188012

evas - cserve - dummy slave - shut coverity up and unmap mmaped mem

while right, coverity doest know that this is just a dummy example,
so... make it happy by unmapping. fixes CID 1193218

fix possible resource leak in edje entry - coverity spotted

this should fix CID 1193228

eina - and follow-on calloc fixup - 0 terminate first calloced string

this follows on from 6dc52db881cbe29dfe8b9a2a0a6745a8b3fa0673

eina unicode - fix calloc stupidity

in looking at CID 1230994 i noticed that eina_unicode_unicode_to_utf8
and eina_unicode_utf8_to_unicode are really dumb. they calloc an array
of bytes then proceed to fill them all in anyway. why? also the
realloc handing in eina_unicode_unicode_to_utf8 wasn't strictly corect
and could leak memory. so this just fixes silly slow code and a leak.

address non nul terminated string due to strncpy

this addresses CID 1230994. as such  eina_unicode_unicode_to_utf8()
always returns a nul terminated string. so it's guaranteed. but yes -
if string is 7 bytes or longer it will not put a nul byte on the
destination. as such for a single unicode char this can never happen
as in utf8 it's 6 bytes. but since eina_unicode_unicode_to_utf8()
safely returns a nul terminated string at all times - we can just use
strcpy safely. no need for strncpy. also handle null return from
eina_unicode_unicode_to_utf8()

fix resource leak on format lookup fail

so in fixing one CID for a format lookup failing and thus invalid mem
access, a possible leak was made in this case. fix that by doing
lookup before any allocation and if lookup fails, return there

this fixes CID 1230999 1230998 1230997

ecore-con - fix possible leak due to other fix

this fixes potential leak due to
ff652084e028b3c8e8f69cf16baecbad32bb40e7 fixing a valgrind access
issue. make sure the server is in the list before handling error
conditions in case a callback sets delete_me (should never though).

fix non 0 terminated buffer in ethmubd slave

this fixes a potential non 0 terminated string buffer in ethumb slave
process -  harmless for apps, but might happen. this fixes CID 1193232

autotools: try to simplify our Makefile.am to reduce warnings.

autotools: use non deprecated macro.

I still don't know why there is still a warning.

autotools: do not build upower module on Windows system.

Evas textblock: strcpy->strncpy (although the buffer is always big enough).

Many tools will complain about the use of strcpy, and it's not that hard to use
strncpy anyway.

CID1039581.

Evas bidi: Make code more explicit (silence coverity).

CID1039922.

Evas textblock: remove checks for impossible null derefs.

This improves code clarity based on input from coverity.

CID1190401
CID1190402

Edje dbg_info: Mark root as not null (will never be).

evas gl common - fix possible negative array accesses if search format fails

this should fix CID 1223506, 1223505, 1223504

evas gl common - fix potential invalid access during texture free

this should address CID 1223507

evas-wayland-egl: Add missing free

Summary:
Add free to eng_gl_context_free
@fix

Test Plan: N/A

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

evas-wayland-egl: Fix typos about egl context attribute list

Summary:
Fix typos 3 to 2.
@fix

Test Plan: N/A

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wayland-egl: Fix Outbuf structure not having evas & engine info

gl_generic functions expect the Outbuf to contain the canvas, so we
fixed the eng_window_new function to accept that, And the engine info.
This simplifies the calls to eng_window_new.

@fix

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

wayland-egl: Fix calls to eng_window_new

Since we simplified the eng_window_new function to accept the canvas
and engine info, we need to update calls to that function

@fix

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

wayland-egl: Fix eng_window_new function prototype

We need to pass in the canvas and engine info structure as the
gl_generic engine needs the outbuf->canvas to be set.

@fix

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

evas-wayland-egl: Fix evas wayland_egl engine crash on mesa driver

Summary:
Evas wayland_egl engine crashes on startup when it is running with mesa.
Evas is trying to get EGL / GLES extension function addresses where evas_gl_symbols.
It is called before eglGetDisplay. Then mesa considers that current egl platform is
X11 if it has no EGL_PLATFORM env var. This behavior of mesa is bad. But we should
provide workaround until it is fixed. Thus EGL_PLATFORM env var should be defined to
wayland before calling eglGetProcAddress.
@fix

Test Plan: run elementary_test under wayland with wayland_egl engine.

Reviewers: raster, stefan_schmidt, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

eet: fix possible ABI break.

eina: rename to eina_module_symbol_global_set

As discussed on the ML, eina_module_global_set should become eina_module_symbol_global_set.

Evas text: fix text object size with empty texts.

@fix.

fix valgrind report on ecore con server connect failure

this should fix T1508 - my testing seems to indicate so. test suite
succeeds, some manual tests of mine work. so is ay this is a good fix
that could fix critical segvs in apps using ecore-con - eg like
weather module in e

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

autotools: proper fix for generated Makevars and distcheck

Reverts 21da4a54545cd6871d1d888566ce9fe488afef67.

It is needed to generate Makevars in-tree even when building out-of-tree because of
how Autotools work. However, distcheck doesn't properly remove the Makevars file in
the generated distdir and makes po/ read only, preventing the build system from
generating an up-to-date version of Makevars. This commit adds the required hooks
needed to fix this behavior.

more versions of vbox mods panic - blacklist all vboxvideo users from vsync

imf - fix x11 vs wl fix to leave x11 alone for those without scim

scim has a nasty habit of sitting and hanging if tryng to set up and
no scim daemon is there. since in x11 you may have multiple input
methods, leave it to be manually set up via env var, and if not then
dont work at all.

ecore-imf - fix broken state on non-x11 environments

this fixes ecore-imf to auto-pick scim or wayland vased in DISPLAY and
WALAND_DISPLAY env vars. it's not a perfect/good/correct fix. it
should be picking based on canvas engine type, but that is a far more
invasive effort, so for release, do this simple one that brings back
wayland input panel and keeps x11 input methods going as long as app
is either x11 (DISPLAY) or wayland (WAYLAND_DISPLAY). if both are set,
it'll get it wrong. as i said - that's a more invasive fix.

eldbus: do not ignore numeric "0" in property_set

"0" value was considered to be error even if signature was numeric

@fix

evas-wayland-egl: Fix evas wayland_egl engine not loading due to missing symbol

Seems the newer wayland-egl code was missing the log domain variable
and thus would not load at time of dlsym due to missing symbol. This
fixes that problem by defining the variable.

@fix

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

ethumb client - fix access to pending struct that has been freed

th pending cancel may call callbacks... which can mess with the list
and thus the pending_add we have, sso do our work before calling this
(like set pending_call to NULL).

eolian-cxx: Scan all files only when explicitly requested.

Fix T1463

edje_cc_handler: Fix build break from recent Edcref changes

Having correct syntax is needed here. ;)

Edcref: final doc improvements.

* Be consistent with the way we specify parameters
* A last css refinement

Edcref: fixed (I hope) the doc for parts aliases

Edcref: try to give sections titles more context

Also fixed some incorrect @context
Changed the @edcsubsection tags so they live in first line of the doccomment, this way you can read them when the file is folded-closed, improving the code readability.

Edcref: tab-- , they tend to break the html render

edcref: move the Links functions so they dont break descriptions docs

Plus some other small doc adjustment

evas/gl_common: Use the right list when removing aptt

Another classical copy and paste error. All references but one replaced.

CID: 1223502

edje_edit: Actually check for z if we want to append z

Classical copy and paste error.

CID: 1224760

update po files

ecore_file: Check return value of chmod()

Handle the error case here if chmod() fails.

CID: 1039696

modules/emotion_generic: Remove unnecessary NULL checks for event data

We are already using the pointer before we check it here. We also don't
check it in the other callbacks so align the handling in this file.

CID: 1039635, 1039636, 1039637, 1039638, 1039639

edcref: move 3 params to be in the correct section

edcref: be consistent wrt params

Edcref: and finally apply a cool style

Now the edc ref is decently readable

edcref: fix the base_scale property docs

base_scale is a property, so document it propertly
Also added missing @since

autotools: install .pc files to the correct directory on FreeBSD

edje_edit: fixing return value of edje_edit_program_name_set

Summary: If name of program is setted to its current value EINA_TRUE should be returned

Reviewers: cedric, seoz, raster, Hermet

Reviewed By: Hermet

Subscribers: cedric, reutskiy.v.v

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

ecore_x_vsync: hack to workaround bug in vboxvideo

there is a kernel oops when using vboxvideo 4.3.14 and one calls
drmWaitVBlank(), then do not init drm when using such driver.

    https://www.virtualbox.org/ticket/13265

Eolian/Generator: fix generation of class functions.

The generation for legacy was missing. The patch removes too the EINA_UNUSED
flag for static functions data.

edcref: review all the @since usage

* put them inside the @property block they belongs
* do not show micro version (1.7 instead of 1.7.0)

edcref: keep properties name aligned with their parameters.

Also make them bold

doc style: better @since html render

Now @since have a left colored border, like @deprecaded, @note, etc

Also keep the number on the same line:
instead of:
since
        1.10
it now render as:
since 1.10

edcref: quick access list: fix a link and make it more compact

evas gl - glx - sety current to NULL on free of window or context

hunting a mem leak i found we dont nuke the current context when
freeing it or the window bound to the current context. fix this.

edcref: unbreak formatting of the page

Every @block need a @context, or the table will broken up and formatting will be bad.

Also be a bit less redundant

eina_tile: Add missing since version tags for union, substract, etc

Merged directly after 1.10 release so mark it as since 1.11

ecore_x: Add missing since to various new ecore_x_screensaver* functions

ecore_evas: Add missing since for new ecore_evas_cursor_unset()

Added this cycle. Also with a typo and some trailing whitespaces while
looking at it.

ecore: Add missing since version for new ecore_loop_time_set()

Introuduced during this cycle. Note it down in doxy.

eina-cxx: Added test for inheriting constructors in the C++11 compiler

This adds the inheriting constructor feature test to the m4 macros to
avoid compilation of the C++11 binding in constructors without this
features, where they would unavoidably fail compilation. This
eliminates GCC 4.7.x since inheriting constructors were available only
from GCC 4.8 and forward.

evas gl - did not free all atlas formats - fix

we have expanded atlas formats long since - so use #define to free
correct amount instead of 6

evas gl - fixz minor leak on exit of rectangles

these were static rect cutouts, so they stayed around on exit and thus
we "lost" them. this nukes them on context free and each new frame.
fixes the "leak"

edcref: small typo in doc

evas: Fix not to clean and re-render if position of object enabled map is changed

Summary: Fix not to clean and re-render if the position of an object enabled map is changed.

Reviewers: raster, cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

ecore vsync - add getenv env var to disable vsync attempt

this is a quick try to allow testing and fixing of the vbox vsync
kernel panic with ability to turn it off so debugging can be done

evas-wayland-egl: Implement eng_gl_context functions

The eng_gl_context functions are used for Evas 3D, so let's support
those in the wayland_egl engine.

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

Merge branch 'devs/devilhorns/wayland_egl'

Fix wayland_egl engine to work again (see Phab Ticket T1478).

Previous (french) changes had broken the wayland_egl engine. This
rewrites the wayland_egl engine to function again, this time using
gl_generic and glsym_evas_gl calls.

evas-wayland-egl: Fix wayland egl engine to work again

Previous (french) changes to evas_gl code broke the wayland egl engine. This
batch of changes fixes that by rewriting to engine to work with new
evas_gl functions.

Fixes Phab ticket T1478

@fix

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

evas-wayland-egl: Add support for EGL_BUFFER_AGE_EXT and add external
function pointers for glsym_evas_gl functions

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

evas-wayland-egl: Add support for SWAP_MODE_QUADRUPLE and use
Eina_Bool for boolean values

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

evas-wayland-egl: Rename Evas_GL_Wl_Window to Outbuf

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

evas-wayland-egl: Fix includes for using GL Generic

Recent changes to evas engines require using evas_gl_generic now, so
let's sort out the headers and include the gl_generic one we need

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

ecore_evas_buffer: fix dereference null return value

Summary: This fixes coverity CID 1210810

@fix

Test Plan: N/A

Reviewers: raster, cedric, stefan_schmidt, devilhorns

Subscribers: cedric

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

edje: fix coverity defects

Summary:
CID 1214113: logically dead code
CID 1230295: dereference null return value
CID 1230296: use after free
@fix

Test Plan: N/A

Reviewers: raster, cedric, jpeg, q66

Reviewed By: q66

Subscribers: cedric

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

textblock: Update visual_pos before calling _size_native_calc_line_finalize.

Summary:
In items loop of _size_native_calc_line_finalize,
last_it should be replaced with new item according to position.
But, visual_pos is not prepared and it is always zero in the function.
So, we need to update visual_pos.
And when textblock only has LTR text,
we can replace last_it according to item list sequence.
@fix

Test Plan:
It includes test cases using the following test case.
 1. "i<b>。</b>"
 2. "。<b>i</b>"

Reviewers: seoz, woohyun, sohyun, tasn

Subscribers: raster, herdsman, cedric

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

evas: fix evas events order and count.

Before this patch, if you were hooking to any event above
EVAS_OBJECT_EVENT_IMAGE_PRELOADED you will end up receiving another
event instead and be fooled into receiving the right one.

In the mean time I did discover that our table is missing one item, the
equivalent to EVAS_CALLBACK_DEVICE_CHANGED. I don't know how to properly
fix that one for now, so put NULL instead to at least avoid a random
memory access.

build_eolian_cxx_test: Set dependencies correctly and remove BUILT_SOURCES

This took quite a while to figure out. The two object names for the cxx callback
test and callback.c have been to close. callback.c needs the eolian headers and
the CXX test needs the eolian_cxx headers.

Split it up and set the deps correctly. If you always run check with something
like -j 9 you might have never noticed as it was very likely that the headers
have been generated due to the wrong dep before.

build/drm: Fix broken build from scratch and add missing library dependency for evas_drm engine

Summary:
This patch fixes following two problems:

1. libevas.so library has a dependency with ecore_drm if '--enable-drm' configure
option is given. This problem is due to 'EFL_INTERNAL_DEPEND_PKG([EVAS], [ecore-drm])'
in m4/evas_check_engine.m4 file. A dependency with ecore_drm should be moved to evas
drm engine not libevas.so. And also this macro makes an error while installation of evas.
 $ make uninstall; ./configure --enable-drm; make && make install

2. missing ecore_drm dependency for evas drm engine.
USE_ECORE_DRM_LIBS macro should be used for building evas drm engine with ecore_drm
library. ECORE_DRM_LIBS macro doesn't have 'libecore_drm.la'. It is used for building
ecore_drm library.
@fix

Fixes T1473

Test Plan:
1. Remove EFL libraries in installation path: $ make uninstall
2. Configure with --enable-drm: $ ./autogen.sh --enable-drm
3. $ make && make install
4. Check module.so of evas drm engine whether it has a library dependency with ecore_drm
  $ readelf -a $EFL_GIT/src/modules/evas/engines/drm/.libs/module.so | grep NEEDED
  $ readelf -a $INSTALL_PATH/lib/evas/modules/engines/drm/v-1.11/module.so | grep NEEDED

Reviewers: stefan_schmidt, devilhorns, raster

Subscribers: cedric, torori

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

evas/textblock: fix to keep original format when a markup tag is matched to a style tag.

Summary:
We can define a style tag as opener, closer and own closer.
If there is a markup tag that is matched to style tag,
it is reprocessed to format node inside of textblock.
But, when the format node will be converted to markup text,
'/' character can be appended to text at closer and own closer style tag.
Even if original markup tag does not has '/' character,
it will be appended according to format node information.

It makes some issue when compare input text with output text.
@fix

Test Plan: This commit includes test case.

Reviewers: woohyun, raster, sohyun, tasn

Subscribers: herdsman, cedric

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

evas/map: correct the interpolation in the span calculation.

As far as i tested, this quality will be better in map transformation,
especially in case of the object rotation on z-axis.

ecore-x - use last modifier mask found, not first to match xserver logic

this makes altgr not conflict as per
https://phab.enlightenment.org/T1273 by using the last, not first
occurance of the key in the mask modifier bits

release: Update NEWS and bump version for 1.11.0-beta1 release

evas tests: remove TODO comment already done

build: Remove files from EXTRA_DIST that got removed.

3bf0425af06a5f4baac2f940e020a8c95fc498e4 removed these files from the
repo but not from EXTRA_DIST.

Revert "Eolian CXX: Get rid of built sources for the callbacks test sources."

This reverts commit 638e72fda4bd312247c133038b26d8105315176f.

After this change the files did not end up in the tarball and failing make distcheck

../../src/tests/eolian_cxx/callback.c:11:25: fatal error: callback.eo.h: No such file or directory

Edje vim: Migrate vim files to the editors repo.

ecore-drm: implement function to disable ecore drm inputs on vt
switching

Summary:
When vt is released by SIGUSR1, drm_inputs have to be disabled.
For that this revision includes,
    1. Unrefer udev monitor
    2. Remove fd handler for udev monitor
    3. Release dbus device

Reviewers: stefan_schmidt, devilhorns, gwanglim

Subscribers: cedric

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

Eolian CXX: Get rid of built sources for the callbacks test sources.

This is not needed now that dependencies are well in place.

Evas textblock: Correct word start/end moving at new line or line begins with spaces

Summary:
Word start/end works incorrectly when it goes to new line or line begins with spaces.
Ex: In elementary_test/Entry, place cursor at the end of line, press ctrl + right arrow keys: cursor moves to begin of next line. In this case, cursor should move to end of 1st word in next line.
Ex2: In elementary_test/Entry, add some spaces to begin of 2nd line ("   uses markup"), place cursor at the first word ("uses"), press ctrl + left arrow keys twice, cursor moves to begin of 2nd line. In this case, cursor should move to begin of last word in 1st line.

This patch provides a fix by considerring next/previous text node to move cursor to correct place.

@fix

Reviewers: woohyun, raster, tasn

Subscribers: cedric

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

eolian: fread fix

Instead of 1 element of size N, we should read N elements of size 1 in
order to sensibly compare the result with the result of calling ftell.

ecore_x: don't go below 0.

It seems that when things go wrong it does happen that we start with i == 0. It
will then have been walking into info[-1] 'happily'. Changing the test, to first
decrement and then compare should stop that issue.

This should @fix T1467.