Revert "ecore-wayland: Add missing EINA_UNUSED for unused function paramater"

This reverts commit 67766929ca56da08dfe28ff0f69930244d53d10c.

Revert all the xdg_shell changes. Should not have been pushed due to
new API function.

Revert "Added support for xdg-shell protocol in ecore-wayland."

This reverts commit e591dff56dd36c6f45d28dbbaa05b290d2996113.

Revert this. Should not have been pushed yet. Sorry Stefan !! :)

evas-software-x11: Remove EINA_UNUSED for function paramater that is
used

Render_Mode is used here so we don't need EINA_UNUSED for it in the
function.

@fix

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

Added support for xdg-shell protocol in ecore-wayland.

Summary: Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Projects: #efl

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

ecore-wayland: Add missing EINA_UNUSED for unused function paramater

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

ecore-wayland: Fix formatting

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

ecore-wayland: Calloc Can fail, so check that it works (or return if
not).

@fix

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

ecore-wayland: Fix formatting

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

Added support for xdg-shell protocol in ecore-wayland

Summary: Signed-off-by: vivek <vivek.ellur@samsung.com>

Reviewers: devilhorns

Subscribers: cedric

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

evas-gl-common: Fix invalid use of Texture

We cannot use the texture to find a valid format Before the texture is
actually created. This is invalid, leads to "warning: tex is used
uninitialized' message from the compiler, and just plain wrong.
Instead, use the alpha property of the Evas_GL_Image to find the
proper texture format, Then create the texture.

@fix

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

autotools: Renamed --disable-c++11 to --disable-cxx-bindings.

The name was misleading.

Fix T1438

eolian-cxx: Renamed event_*_callback_{add,call} to callback_*_{add,call}.

ecore_drm_dbus: Do not use the same name for global and local variables

We use conn here for a global Dbus connection variable as well as sometimes
in a local scope. Better rename the global one to avoid confusion and
potential failures when the code would fallback to the global one even if
the local scope should be used and the var was simply forgotten.

evas 3d - fix error handling for ftell from coverity

coverity was right again - CID 1222453 - fix.

evas 3d - fix null access for fopen on 3d mesh save

this fixes CID 1222455 yes fopen for write may fail!

evas key bitshifts for 64bit type - fix.

coverity is right. this is another set of thse bitshift problems with
unsigned long long. fix 1224346, 1224345, 1224344 1224343 and 1224342

edje - remove logically dead code

fixes CID 1224348 - coverity is right

edje - fix coverity complaint about invalid mem access

fixes real issue for invalid object - CID 1224349

fix coveirty range complaint on modifer bit mask

this fixes CID 1224356, 1224355, 1224354, 1224353 and 1224352

evas software generic engine - fix tb configuring to be consistent

this addresses CID 1224761 - consistent checks for tb in render engine

eina thread queue - check write return

this addresses CID 1226191. this should never have an error as the fd
should always be valid as long as needed.

evas gl x11 engine do extra check to make coverity happen

this fixes CID 1226192

ecore x vsync - be more conservative about DISPLAY string src

i hope this addresses CID 1229131 - don't trust the DISPLAY var
content much at all - limit it to [a-z][A-Z][0-9][-] only. hopefully
coverity is happier.

fix eina unicode realloc fix of coverity complaints - return right buf.

eina thread queue - fix possible deadlock

coverity pointed out a deadlock - CID 1226193. i hope this fixes it
(need to wait days for a coverity re-run). tests pass and real life
usage works, so i assume its ok. in theory i see no problem either.

embyro-cc - fix coverity overruun issue

overrun in fatal msg access - fix CID 1039315

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