elm_table: remove old api!

elm_grid: remove old api!

efl_ui_grid: remove old api!

elm_box: remove old api!

efl_ui_box: remove old api!

elm_toolbar: we need to register the items in the submanager

focus: Avoid infinite loop in window

I kept the safety error message for easier debugging.
Test scenario:
  elementary_test -to "Window Inline"
  Click on an entry. Press Shift+Tab.

Ping @bu5hm4n

widget: Make focus_mouse_up_handle internal

I don't think this belongs to the public EO API.

Ref T5363

combobox: Mark as legacy only if legacy API is used

See bc2fe6bb778b559e6c88f836d8cbdfb631b193b4

evas: Override del() for evas objects

This makes EAPI evas_object_del() and EO API efl_del() work the same on
evas objects, i.e. a del() implies an immediate call to hide() and mark
the object as "delete_me".

If the refcount remains > 0 the object won't be actually deleted, thus
EFL_EVENT_DEL won't be triggered. I think it would probably be a good
idea to have a new event "del,request", to signal reference owners that
this object "wants" to die.

Ping @raster @zmike

ector: remove unused value

The stored value of v_last is overwritten in another place,
its stored valued is not used.

eio: fix multiple open/close chain example.

eio: update example for openning and closing file asynchronously using a chain.

evas: there is no effect of variable assignment while clean up.

Summary:
There is no effect of assingning a variable while clean up.
@fix

Reviewers: raster, cedric

Subscribers: cedric, jpeg

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

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

ephysic: comparing with same statement.

Summary:
There is no effect of comparing with same expression.
@fix

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

emile: comparing less than zero with unsigned interger.

Summary:
Unsigned integer should not be compared to less than zero.
@fix

Reviewers: cedric

Subscribers: cedric, jpeg

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

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

eina: comparing less than zero with unsigned long int

Summary:
Unsigned integer should not be compared to less than zero
@fix

Test Plan: NA

Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

eina: same expression on both statement.

Summary:
Comparing with same expression is avoided.
@fix

Reviewers: cedric

Subscribers: jpeg

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

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

efl_ui_focus_manager_calc: we dont need the explicit root handling,

that is done later in line 1205

efl_ui_focus_manager_calc: focus a child if a logical is focused

If you call focus_set(m, o) where o is a logical child, then the focus
will go to any none logical child of o, or if there is nothing in the
children of o, then the focus will remain on the now focused element.

Revert "edje: moved from edje_multisense.c to Ecore_Audio.h for select the type of render."

This reverts commit c1c424a4075a85375c83f4c137b74c9919c5fc57.

revert edje multisense changes to match ecore audio revert

Revert "ecore_audio: a minimal template for playing sound on WINDOWS is added"

This reverts commit 7a98f617e94ef42787e06e685a8d5e7be34fff21.

this commit breaks compiling things against efl badly. i'm on holiday
and haver only a mini screen and keyboard wwith me so any serious wok
in hunting these issues isn't going to be fun or easy. as this causes
bad build brreaks this is worth a revert IMHO. please re-submit wwhen
you've tested against efl by building things against it. also our
public headers just should not have such ifdefs/if's that change api
presented based on how efl is cnfigured. we expose the same api and
macros and types regardless of internal config.

emotion gst1 - handle null imagwe data returrn and dont crash

fix crash i spotted thanks to getting img obbj data returning null.
@fix

efreet: unsigned int cannot be compared to less than zero.

Summary:
Wrong camparision with less than zero for unsigned integer.
@fix

Test Plan: NA

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

evas: there is no effect of variable assignment outside the function.

Summary:
Variable assigned is not used anywhere else, making it unused.
@fix

Test Plan: Na

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

eet: wrong comparision to less than zero with unsigned integer.

Summary:
Unsigned interger should not be compared to less than zero.
@fix

Test Plan: Na

Reviewers: cedric

Subscribers: jpeg

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

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

elementary: passing NULL to variadic funtion can be undefined.

Summary:
Passing NULL to variadic funtion may lead to undefined behaviour.
@fix

Test Plan: NA

Reviewers: lukasz.stanislawski

Subscribers: cedric, jpeg

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

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

evas: fix for sscanf use with save flags input.

Summary:
As 'flags' can be supplied by application, in case application is supplying
a big string(error case), it can cause crash in some version of glibc.

Setting maximum possible input (0x64) length as width specifier

Fix for static code analyzer warnings
- scanf without field width limits can crash with huge input data on some versions of libc

Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>
Test Plan: Run SonarQube

Reviewers: cedric, raster, Princekrdubey, rajeshps

Reviewed By: cedric

Subscribers: jpeg

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

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

elementary: fix unintentional visibility of elm_hover in combobox constructor

Summary:
since fd9cf1d3b628f70d66ec85a4f2433c675d9ad243, All Eo canvas objects remain
visible by default.
but elm_hover doesn't want to be shown in combobox constructor.
showing elm_hover in constructor occurs unexpected behavior.
this fixes that bug.

Test Plan:
1. elementary_test - Combobox
2. Click the combobox to Expand the list.

Reviewers: jpeg, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

efl: properly initialize Eina_File.

CID 1381605.

build: fix make examples on Windows

Summary: We should define the windows version before including evil.

Test Plan: Run make examples on windows newer than XP.

Reviewers: vtorri

Subscribers: jenkins, cedric, jpeg

Tags: #efl, #windows

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

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

evas: make sure we always open the file and look at its header before trying to load data.

eio: migrate example to use Eina_Future.

evas: remove evas_canvas3d_texture_file_set and efl_file_set instead.

evas: remove unused function evas_cache_image_request.

evas: remove unused function evas_cache_engine_image_request.

evas: remove unused function evas_common_load_image_from_file.

evas: remove unused fonction evas_gl_common_image_load.

eina: continue to check for magic.

evas-gl-image: Fix dereference after null check

Coverity reports passing a null pointer 'im->gc' to
evas_gl_common_context_flush which directly dereferences it, so lets
be sure that 'im->gc' is valid before passing it to context_flush

Fixes CID1374273

@fix

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

evas-gl-common: Fix dereference after null check

Coverity reports that there may be a null pointer dereference here so
check that 'error' exists before trying to set it.

Fixes CID1374272

@fix

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

eolian-bin: Free previously allocated eina_strbuf

Coverity reports that the eina_strbuf 'param_call' leaks when it goes
out of scope here, so fix the leak by freeing the strbuf

Fixes CID1381502

@fix

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

evas: remove image_load from the engine.

evas: remove use of engine image_load from Evas.Canvas3D.Texture.

evas: remove duplicated code and rely on Efl.File { get; set; }

We can almost remove image_load from the engine backend after this patch.
One little bit left in Evas_3D.

edje: remove duplicated code and rely on Efl.File { set; get; } mixin.

efl: make Efl.File a mixin that implement file_set for everyone.

eina: Eina_Prefix should never contain a NULL, no code expect it.

eina: allow eina_file_close(NULL) to simplify code using Eina_File.

elementary: update code highlighting to follow eolian syntax change.

ecore-evas-drm: Implement support for hardware output rotations

This patch adds support for doing output rotations via hardware. This
is implemented inside the Ecore_Evas engine so that it is transparent
to the caller of ecore_evas_rotation_set.

ref T5999

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

ector: minor grammar fixups

Reviewers: cedric

Subscribers: jpeg

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

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

edje: moved from edje_multisense.c to Ecore_Audio.h for select the type of render.

Reviewers: NikaWhite, raster, cedric, vtorri, rimmed

Reviewed By: NikaWhite, vtorri

Subscribers: artem.popov, cedric, jpeg

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

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

ecore_audio: a minimal template for playing sound on WINDOWS is added

Summary:
Realization of audio data playback through WASAPI (Windows Audio Session API).(minimal)
WASAPI model:
1. Find  a playback device (in the default system)
2. Register the client on the playback device.
3. Create a playback format for audio data.
4. Initialize the client with the created format, and access mode,  ....
5. Take the object-rendering from the client for play the data stream.
6. Play data stream

ecore_audio model:
1. create a object for play the data stream(out object)
2. create a object to receive the data stream(in object)
3. register in the out-object the in-object
4. play data stream

Necessary:
Realize the ecore_audio object to play the data stream using the WASAPI model.

How implemented:
1. The object ecore_audio_out_wasapi is implemented
2. object ecore_audio_out_wasapi - the object constructor is find a playback device (in the default system)(WASAPI)
3. _ecore_audio_out_wasapi_ecore_audio_out_input_attach -  register in the out-object the in-object
   3.1 Register the client on the playback device.(WASAPI)
   3.2 Create a playback format for audio data.(WASAPI)
   3.3 Initialize the client with the created format, and access mode,  ....(WASAPI)
4. _write_cb - play data
   4.1  Take the object-rendering from the client for play the data stream.(WASAPI)
   4.2  Play data stream(WASAPI)

Reviewers: cedric, vtorri, raster, an.kroitor, NikaWhite, FurryMyad, rimmed, t.naumenko, Jaehyun, bowonryu

Reviewed By: vtorri, NikaWhite

Subscribers: artem.popov, cedric, jpeg

Tags: #windows

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

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

elput: Fix issue of pointer rotation not being set on startup

elput_input_pointer_rotation_set may be getting called before input
devices are actually added, so we need to be able to set pointer
rotation when a device actually gets added.

To fix this, we will store the requested input rotation value, and
apply it at the time of input device creation.

NB: The current case is that the wl_drm module will apply existing
output rotation before Input has completely initialized, thus any
calls to set the pointer rotation there will not succeed as input
devices have not been created yet.

@fix

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

ecore-file: do not emit events when an inotify watch is removed

???????????????????????????????????????????????????????????????????????????????

@fix

elementary: Mark unused function parameter as unused

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

ecore-con: Mark unused function paramater as unused

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

Canvas layout: fix Efl.text (non-legacy) TEXT parts

Current implementation should also support TEXT parts (like button)
with usage of efl_text_set.

Follow-up to 0061a4bb349e6c8e122913791bf83f7f40b0cede.

Windows: remove XP support and forbid compilation and run on this platform

Test Plan: run on XP

Reviewers: jpeg, cedric, ajwillia.ms

Reviewed By: ajwillia.ms

Subscribers: ajwillia.ms

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

Ui layout: handle Efl.Text legacy code better

This should cover the cases without need to duplicate a lot of code.
Follow-up to deca3c947ff1a2bed3f4baadce576105055af266.

Canvas layout: improve legacy code paths for Efl.Text

Better handling of specific cases where you want to efl_text_set on
a TEXTBLOCK part.
Follow-up to af1595fd0160f76de82cddeb2576143f65492c0b.

elm_code_widget: set appropriate background colour.

Canvas layout: fix legacy text set

Keep the legacy code path when using edje_object_part_text_set.
Fixes e's notification that got broken after
3642b3ae679e3bedba4155fb9e54b44483b3b9e1.

Also, limit new efl_text_markup set to TEXTBLOCK parts.

eina debug - add missing unlock in erite err case

fix CID 1381503

efl_ui_focus_object: also implement focus.get

evas_focus: skip focus set if the focus state is the same

ecore_con: simplify Efl.Net.Server.Ssl by inheriting from Efl.Net.Server.Tcp.

ecore_con: add an intermediate Efl.Net.Server.Ip

eolian gen: simplify generate_normal_free a bit

efl_ui_focus_manager_calc: correctly specify owned where its needed

Revert "elm focus manager - dont leak child lists when updating focus order"

This reverts commit bd6dcbb044ffea543f27cc69c421317bdea4a14a.

This was not fixing the issue, just ducktaping over a bug in eolian, the
bug is now probebly fixed in eo / eolian.

eolian: add tests for the new owning functions

eolian: generate free function for owned parameters

With this commit owned parameters are freed if the object function
cannot be called, this means bla(NULL,param) will not leak anymore

eo: introduce function definitions with fallback

there can be cases where the call to such a function is never going to
be directed to the actual implementation. In those cases the FallackCall
will be called.

edje entry: fill content info when text is deleted by imf event

Summary:
There was missing info when text is deleted by _edje_entry_imf_event_delete_surrounding_cb().
@fix

Test Plan: N/A

Reviewers: kimcinoo, raster, cedric, jpeg, herdsman

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

evas font: check returned value of FcPatternGet function

Summary:
Evas should add font when only FcPatternGet function gives matched result.
@fix

Test Plan: N/A

Reviewers: raster, cedric, herdsman, jpeg

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

efl ui textpath - remove more unused vars

elm test: rename test_slider_interval.c to test_ui_slider_interval.c

efl ui textpath - remove unused vars due to dead code removal

fix warning

evas cserve - fix validation of cserve mesg size field

fix coverity CID 1039567

evas cseve - check msgsize before using - coverity

fix CID 1039568

elm atspi - fix checking return of eldbus_message_iter_arguments_get

fix coverity CID 1353590

elm config - fix eina value setting to check get return

fix CID 1357139

evas ector gl - fix leak of image data on data put

@fix

fix coverity found leak CID 1374645

evas ector gl - make coverity happy about leaks on data put

fix CID 1374646

eina debug - check write return values and complain on failure

coverity - fix CID 1377521

eina debug - check write return value and complain

makes coverity happy - CID 1377524

efl debugd - check return of write func and report errors

makes coverity happy and reports write errors... fix CID 1377525

efl model - fix checking return of eina_accessor_data_get

coverity - fixes CID 1380995

eio: do not copy memory for nothing.

eio: use TIMESTAMP when meaningful.

eio: migrate efl.io.manager.ls to use Eina_Future.

ecore: allow other thread to write local data on a thread to increase utility.

elm widget - event legacy propagation - remove coverity warning

fix CID 1379920 - event_flags is actually never NULL or undefined in
the function logic. it's always set to point to the specific event
flags field depending on struct type or the function will return
before using the pointer.

efl text markup - fix hex check to be or not and - found by coverity

fix CID 1381491

efl ui focusable - remove pointless check of ptr already checked

coverity fix CID 1381492

eolian: fix tests after extra checks

eolian: validate inner types of complex types

This makes sure the inner types are all correctly defined and
that freefuncs are actually correctly filled.

eolian-cxx: Add optional parameter to lambda for unnamed proxy instantiation