Textblock: Fix padding computation (part 1)

Track padding per paragraph, since this is how it is computed.

Problem before this patch:
- If markup text is changed, padding may grow, and the layout is updated (good)
- If the UI itself needs a relayout, the old padding value is NOT reused,
so style paddings will reset the padding to 0.

Test protocol:
- Set some text with style=glow. The whole object should have padding 2,2,2,2
- Relayout the UI, the whole object will have padding 0,0,0,0 (should be 2,2,2,2)

ecore_x: set cursor with xinput2

For some reason XDefineCursor might malfunction when using xinput2. So
when using xinput2, use XIDefineCursor. Also see:

https://bugs.launchpad.net/compiz/+bug/1179155

Evas/cserve2: Add some safety checks when reading socket messages

Fixes CID 1039571 and 1039572.

Evas/cserve2: Remove dead code

Fixes CID 1039925.

ecore xcim module - don't printf every time you start to add stdout noise

ecore ecore_x_e: check 0 and -1 for ecore_x_window_prop_window_get(). According to note for ecore_x_window_prop_window_get() :

Summary:
If the property was successfully fetched the number of items stored in
val is returned, otherwise -1 is returned.
Note: Return value 0 means that the property exists but has no elements.

Reviewers: seoz

Reviewed By: seoz

CC: cedric
Differential Revision: https://phab.enlightenment.org/D457

Evas/cserve2: Stop looking at those dirty pics!

What I mean is erm... images marked as dirty don't have a
cache_key, and that's perfectly fine when the image file
has changed.

Evas/cserve2: Fix potential crash in image_free

Don't try to free an object that was already freed.
Basically, allow image_free(NULL).

Eio monitor: Fix crash on invalid data access

Fix race condition when touching/changing a (theme) file often.
An Eio_Monitor was marked as "delete_me" but the rename callback
was still called, leading to memory access to already freed
objects.

Test protocol was:
ELM_THEME=~/default.edj elementary_test &
watch touch ~/default.edj

Evas/cserve2: Keep image_id when reopening a changed file

If a file had changed, a new OPEN message was sent, and the
client image ID was then changed, but the LOAD message was
sent with the previous image ID. So cserve2 would not be
able to honor that request.

Evas/cserve2: Clear up File_Data when opening a new image

After a file changed, the "changed" flag was not reset in the
File_Data structure, although it was supposed to represent a
new entry altogether.

Evas/cserve2: Rename "invalid" field to "changed"

This flag indicates that a file changed, not that the entry
is invalid. There is another "valid" flag already.

Evas/cserve2: Improve error messages a bit

Evas/cserve2: Add another load error check

This one is very unlikely as it would mean the image is
invalid while being in the hash.

Evas/cserve2: Fix crash on image file change

When an image file is changed, it is discarded from cserve2,
so the references become invalid. In case we were loading a
scaled version of that image, no proper error checking was
done, leading to obvious crashes.

Evas/cserve2: Reduce error logs on invalid glyph load

Change message level from ERR to WRN, when a glyph is not
loadable because FT fails to load it or it contains 0 pixel.

cserve2 used to complain about invalid glyph 3, on a few fonts

Evas/cserve2: Fix crash in server on file change

An inotify callback is triggered when an image file changes,
and it is supposed to cleanup all references to this image.

Unfortunately, it was doing it in a very unsafe way as pointers
could become invalid, because of hash free callbacks in the
referenced image list. Add some list safety and always assume
the pointer might be dead after free operations.

TBH, this _file_changed_cb function looks very confused to me
(as it tries to bypass eina_hash_del).

Evas/cserve2: Fix memleak during shutdown

Don't reset the dead slave callback, as it is used to free the
slave worker's resources.

Evas/cserve2: Use calloc instead of malloc

This will silence some warnings from valgrind (and add some
safety as well...)

ecore-x present fixups

ecore-x now supports XPRESENT

see http://cgit.freedesktop.org/xorg/proto/presentproto/plain/presentproto.txt

ecore_evas_win32: Fix build regression in ecore evas win32 engine

Regression introduced with fb59ac34a1ec65680ca4ff3fc62d1fc42aa72f5a
Simple typo during Eina_Bool convertion.

emotion gstreamer1: Set visualizations if the vis name is not NULL

and not the other way around.

emotion gstreamer1: Also handle DATE_TIME tag

emotion gstreamer1: Fix wrong FIXME comment

emotion gstreamer1: Don't forget to allocate metadata struct

emotion gstreamer1: Implement seekability check via seeking query

emotion gstreamer1: Add support for selecting subtitle channels

emotion gstreamer1: Volumes > 1.0 are valid too

emotion gstreamer1: Add support for buffering and clock-lost handling

emotion gstreamer1: Major cleanup of internal and GStreamer state handling

emotion gstreamer1: Don't set pipeline to PAUSED during seeking

This is not required and is handled transparently inside GStreamer
already.

emotion gstreamer1: Use GST_PLAY_FLAG_VIS instead of the corresponding integer

emotion gstreamer1: Don't use the NATIVE_AUDIO/VIDEO flags

There are too many video formats we don't handle, so we should let GStreamer
handle the conversion if necessary.

emotion gstreamer1: Use refcounting instead of in/out variable counting

emotion gstreamer1: Simplify pipeline creation

Also remove the webcam support, this could've never worked and
needs to be handled differently.

emotion gstreamer1: Some code re-flow in the GStreamer bus handler

emotion gstreamer1: Use GStreamer's path/URI handling functions instead of writing our own

emotion gstreamer1: Rename functions and variables for clarity

Also remove some useless variables.

Unbreak ecore_evas wayland engines:

Val variable is not declared here. Use the variable 'on' which was
passed in to this function.

Signed-off-by: Chris Michael <devilhorns@comcast.net>

ecore_evas - use eina_bool instead of the char for interal data.

evas - cserve2 - avoid segv accessing im->cache

evas gl - fix egl/gles build after alpha4 change for fonts

evas - fonts - move to using 4bit and rel 4 bit compressed font glyphs

this changes the internal encoding of font glyphs in evas to use 4bit
uncompressed if small, or 4bit rle (run length encoded) if larger.
this caves at least 50% of memory on fonts - and more if bigger. with
large fonts (40-80pixel size) we can save in the region of 80% of
memory used for glyphs. this also happesn to allow speedups in
rendering too.

evas gl - prefer alpha4 format for font textures if possible (save memory)

if alpha4 is possible (desktopgl) then use it for fonts as this should
cut memory in half for them and possibly speed things up due to less
memory bandwidth needed

ecore_evas_x: Call _ecore_evas_x_sync_clear() after
_ecore_evas_x_gl_window_new() to ensure eglDestroySurface is called
before XDestroyWindow call.

In case of gl engine, eglDestroySurface should be called before
XDestroyWindow.
Special thanks to Changwoo Kim <ce2010.kim@samsung.com> for the idea.

ecore_evas_x: Fix gl_sync_draw_done initialize routine to handle
exceptional case.

As ECORE_EVAS_GL_SYNC_DRAW_DONE is an environment variable, that could
have unexpected value like 2 or 3. So set gl_sync_draw_done to 1 only
when the env variable is 1.

ecore_evas_x: Refactor/flourish gl_dync_draw_done member from Ecore_Evas
structure.

- moved gl_sync_draw_done for a better structure packing.
- changed gl_sync_draw_done type from int to char which is enough.
- added more description to gl_sync_draw_done

eina: fix memory leak of inserting the wrong value in the hash.

We where inserting the pointer data instead of the pointer, leading to
unaligned access on Sparc (Thanks Lutin to report it and Debian tools/infra
to help us catch it) and also a memory leak.

ecore - clean up code.

fix indentation and use Eina_Bool instead of char.

ecore_evas - fixed indent

[eina] add test case - eina_list_clone, eina_list_reverse_clone

evas gl example - make bg blue to make onefang not freak out. :)

evas cserv2 - remove logical dead code

fixes CID 1113441

formatting - fix

edje - formatting - indenting fix

comment out unreachable code

fixes CID 1132633

evas gl - comment out unreachable code

fixes CID 1132634

evas - remove dead code (comment it out)

removes entirely dead code and keeps it for reference only

edje_cc - remove dead code

fixes CID 1132639

cserve2 - debug bin - fix theoretical leak of socket fd

fixes CID 1039286

eina lock posix - remove dead code in eina_spinlock_take

this solves CID 1106340

emotion: remove some warning in the example.

examples: fix various warnings in ecore and eina examples.

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

examples: fix pointer <-> int conversion (windows 64 bits) in ecore and evas examples.

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

examples: do not do void* arithmetics in evas examples.

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

example: fix uninitialized variable in ecore.

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

examples: remove unused variables and label

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

examples: remove unused param warnings

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

ecore: use WIN32 API to port readdir_r in ecore thread example

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

autotools: really don't build Unix specific example on Windows

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

[ecore_x] Set data of selection event for XDND selection

Summary: Add data set case of selection event for XDND selection if selection atom is XDND

Reviewers: raster

Reviewed By: raster

CC: JackDanielZ, woohyun, Hermet, seoz, cedric
Differential Revision: https://phab.enlightenment.org/D446

evas - unbreak gl font rendering with dst pointer checks

evas - sw render - protect against null pointer surface access

Update PKGBUILD to work with release tarballs as well as Git clones

Reviewers: raster

Reviewed By: raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D439

Evas/cserve2: Fix valgrind warning about uninitialized memory

In cserve2, a shortcut was taken to check if two images were the
same, using memcmp() on the Evas_Image_Load_Opts struct. But it
seems some empty areas in the struct are uninitialized, potentially
making memcmp() fail when the images were actually the same.

This is a minor issue since this function is called only when
bypassing the socket wait.

Also, memset load_opts to 0 and copy all the fields to avoid
the same warning in socket send().

I'm just wondering about the performance impact vs. memcpy/memcmp.

Evas cserve2: Fix crash in cserve2 when slaves die

If a slave dies (eg. killed) when it's idle, then cserve2 will crash
miserably at the next request. Indeed, the dead slave's corpse was
removed from the working slaves' list but not from the lazy idle
slaves list.

Also, set read buffer to NULL after free. Just in case. We never know :)

ethumb: fix build on windows by matching the #if around the if statement.

ecore_con: fix build on windows by matching the #if around the if statement.

efreet: fix build on windows by matching the #if around the if statement.

Ephysics: Silence more warnings from libbullet

The bullet library generates tons of warnings:
1. -Wunused-variable
2. -Wunused-parameter
3. -Wshadow

2 and 3 were properly ignored by pragma GCC, but unused-variable
doesn't seem to work. At least on my version of GCC (4.8.2).
So, let's ignore ALL warnings from libbullet includes.

ecore: don't build Unix specific example on Windows.

configure: correctly detect C99-capabilities for ecore_con_resolver

ac_cv_prog_cc_c99 being empty just means there is no need to add
any flags for the compiler to be C99-capable.
"no" means the compiler isn't C99-capable.

The check is actually unnecessary as EFL demands a C99-capable
compiler these days but if that every changes ...

Reviewers: zmike

CC: cedric
Differential Revision: https://phab.enlightenment.org/D443

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

Revert "fix allocator sizeof operand mismatch reported by clang"

This reverts commit 955cc579d4d0b52cc71e309c35d1850424dc3cca.

why? wrong. no. and it breaks actual execution/functioning. segv land

fix allocator sizeof operand mismatch reported by clang

https://build.enlightenment.org/job/nightly_efl_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-07-1/report-QL6MiM.html#EndPath
https://build.enlightenment.org/job/nightly_efl_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-07-1/report-a2l0Ci.html#EndPath
https://build.enlightenment.org/job/nightly_efl_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build/2014-01-07-1/report-McvqNf.html#EndPath

ecore_evas_x: Refactored by reducing cascaded if statements.

Just return the function. Do not create unnecessary cascaded if
statements it makes indentation depth deeper.

ecore_evas_x: Check 1 for gl_sync_draw_done instead of 0 before sending comp_sync_draw_size_done.

There was a corner case which gl_sync_draw_done is set to -1. So send
comp_sync_draw_size_done when the gl_sync_draw_done is not 1. It can be -1 or 0.

This is a manual cherry-pick from tizen.org.
Thanks to original author Jiyoun Park.

commit 4be6c821df4ccd418dbe44f7e6fecbcd7dd66d74
Author: Jiyoun Park <jy0703.park@samsung.com>
Date:   Thu Jul 4 18:04:11 2013 +0900

    fix lockup issue (N_SE-44214)
    because this sync draw done cannot be send

    Change-Id: Id05978e925d5b44d6ca9af0a1684bbb255b9ec5b

ecore_evas_x: Fixed formatting and added an empty line for a better readability.

windows build - comment out unused function in ecore_pipe

fix warning about always true expression

result is a DWORD and DWRODs seem to be unsigned thus always >=0 thus
the check is useless clutter.

win build - check for pause func and use only if its there

fix missed ifdef lack for getuid for windows

fix mingw build for setuid fix/checks

setuid safeness - ensure if an app that is setuid doesn't do bad things

this makes efl ignore certain env vars for thnigs and entirely removes
user modules (that no one ever used) etc. etc. to ensure that *IF* an
app is setuid, there isn't a priv escalation path that is easy.

eio - protect against posible segv on rename with eio

this MIGHT fix T45 but i can't reproduce to confirm, but reading a
backtrace indicates this could have been the issue. it looks like
there is room for a dangling pointer anyway, so fix.

stable release - cherry-pick me!

evas - gl-x11 engine - put in guessed blacklist of yinvert extn for intel mesa

efl - build - add warning about gnutls to configure and dropping privs

stable release - cherry-pick me!

Evas: Fix some clang warnings in evas_cs2_client.c

- Invalid alloc size (typo)
- Initialized value never read (set twice)
- Potential memleak (call free(msg) in case of send error)
- Null pointer dereference (check nullity)

There are still other warnings, but I believe these are false
positives.

fix typos in EFL documents

fix warning related to argument in generating doxygen