DALi Version 2.0.0

Change-Id: I0a32b1d26aa223dd728b36188726693084461e35

RenderTaskProcessor::AddRenderablesForTask() optimization.

- pulled loop-invariant conditional for which RenderableContainer to add
  to, out of the loop;
- replaced branch for assigning true, with compound or-assignment of the
  result of the check;

Change-Id: I2f36f2b7a5eb95da6e3cc04bddfbede0f8437b69
Signed-off-by: György Straub <g.straub@partner.samsung.com>

Implement GetNanoseconds() with std::chrono.

* Platform independent.

Change-Id: I484dbfb5a0b6115a6ea8cd6c645420985e9aae0d
Signed-off-by: Victor Cebollada <v.cebollada@samsung.com>

Improve the logic using Erase-Remove Idiom.

instead of erasing each element found and restarting the
loop again . first find all element and move them to the end
of list and then erase them once using std::erase(std::remove_if()).

This will improve the cache locality as this will avoid lot ofpointer chaseing.
Also avoids expensing Dali::Vector::end() call.

Change-Id: I306de3f017ef23ea7d4d1e6e5202940f84fcf872

Merge "use string_view instead of const string literals." into devel/master

use string_view instead of const string literals.

as constexpr stringview calculates the length of the string during
compiletime. the == comparison is fast as it first checks whether
the size of the both string are same before calling the expensive
compare() function.

Change-Id: I996faed4d82e5c478f5f55c31cfe25581aa4bbc9

refactor dali-animation.

- refactor keyframeChannel class.
- refactor KeyFrameBaseSpec to avoid heap allocation.
- keep a templated form of GetSpecialization.
- in the keyframeanimation functor keep a copy of keyframeChannel instead of heap allocated object.
- remove AnimatorFunctorBase class and replace it with std::function to avoid heap allocation when possible.
- structure packing of AnimatorBase

Change-Id: I416df4601116547c79f04b2cf8e0bc95e622b869

Added test-harness timeout check

Change-Id: I82276c59de57bc29a2e6a2e895db26e5072a61e0

DALi Version 1.9.36

Change-Id: Ie87e252297e782248c1d5222a3cabc58e2c8f76a

Change property name from TouchDelegateArea to TouchArea.

Change-Id: I6b2af477ffb8523d7e98ffbc250cbb2deb59c5cc

Updated patch-coverage.pl script

Change-Id: I01c1e0e7a6959596bea5ff3318e964eda230b519

use algorithm and range for

Change-Id: I8f276553114fecf8c9f6b60696e2dcd274388958

Merge "DALi Version 1.9.35" into devel/master

DALi Version 1.9.35

Change-Id: I39bd529cc7846aebdebe06c06145977b3dd6e269

Merge "use modern construct '= default' for special functions." into devel/master

Merge "Add TouchDelegateArea property." into devel/master

Adding new patch-coverage script

Change-Id: Iaf227f70e10f2af1db928c0f0614619dbee53d5d

Add TouchDelegateArea property.

TouchDelegateArea can reset the actor's touchable area.
This is usefull when the actor is small, but it should have a larger touch area.

for example

      Actor actor = Actor::New();
      actor.SetProperty(Actor::Property::SIZE, Vector2(10.0f, 10.0f));
      actor.SetProperty(DevelActor::Property::TOUCH_DELEGATE_AREA, Vector2(200.0f, 200.0f));
      actor.TouchedSignal().Connect(OnTouchCallback);

The actor is small, If you want to set the touch area to a larger area,
you can use the TOUCH_DELEGATE_AREA property.

Change-Id: Ic4e3683e09d1636bc61719ea1e83bcd05a1c4153

Merge "refactor Property::Map interface." into devel/master

Merge "add  base type of enum to reduce class size." into devel/master

Merge "DALi Version 1.9.34" into devel/master

DALi Version 1.9.34

Change-Id: I5f8723bdcce4d478589a8dc72cea88180882d69f

Merge "Skip rendering if no animation is currently active" into devel/master

Add intercept touch event

The Touch event calls the TouchEvent callback by going back from the last child actor to the parent via hitTest.
InterceptTouchEvent checks the touch event in the parent first.
Returning false from interceptTouchEvent allows child actors to receive TouchEvents.
If it returns true, the actor will receive a TouchEvent.

for example

   Actor parent = Actor::New();
   Actor actor = Actor::New();
   parent.Add(actor);
   actor.TouchedSignal().Connect(&application, functor);
   parent.TouchedSignal().Connect(&application, parentFunctor);

The callbacks are called in the order functor -> parentFunctor.

If you connect interceptTouchSignal to parentActor.

   Dali::DevelActor::InterceptTouchedSignal(parent).Connect(&application, interceptFunctor);

When interceptFunctor returns false, it is called in the same order functor -> parentFunctor.
If it returns true, it means that the TouchEvent was intercepted.
So the child actor will not be able to receive touch events.
Only the parentFunctor is called.

Change-Id: Ib6887adbcee59168a7caf7f36bcc400500c626e8

Fix doxygen comments

Change-Id: If8c7b64bbf197370ccdcf71d959846a477cee8d1

Merge "Fix PropertySetSignalType signal interface signature" into devel/master

use modern construct '= default' for special functions.

This patch is auto generated by clang-tidy with modernize-use-equals-default.
excluded automated-tests

Change-Id: I50cf90eb5684144a95b263774f3d8b3be8f4518c

Merge changes I2769d6ae,I11060630,I5c96789c,I1d0e89be,I331a838c, ... into devel/master

* changes:
  Add template keyword for android compiler
  refactor SignalBase and CallbackBase using variadic template.
  refactor Signal class.
  refactor ConnectionTracker class.
  refactor SlotConnection class.
  refactor CallbackBase class.
  refactor BaseSignal class.
  refactor  SignalConnection class.

DALi Version 1.9.33

Change-Id: I055725b0491f27960da25b48a94d6ee3d67dff89

Fix PropertySetSignalType signal interface signature

because Property::Value class is not a regular type each
time we pass by value it takes a deep copy(2 allocation)
this interface change will fix that issue.

Change-Id: I1c54c1fac00d0387f474c831df4eacf017dbd54b

Skip rendering if no animation is currently active

Change-Id: Ie14194eeee047d908f36ec4c509e3863cc227cb8

Merge "skipping the render items outside of the partial rendering merged area." into devel/master

Add template keyword for android compiler

Android compiler needs template keyword added
to be able to find Emit / EmitReturns methods.

Not needed on latest GCC on Ubuntu or target.

Change-Id: I2769d6ae7e4aa6e56b26361885b8edc9a9277a35

add  base type of enum to reduce class size.

with base type and structure packing the Animation class size reduced by 16byte.
Also added class member-initializer rather than updating default value in constructor.

Change-Id: I0153e8aefc6bb0128f185996389ba7d85f689049

refactor Property::Map interface.

- take std::string and Property::Value by value.
- use std::string_view in the find interface.

Change-Id: I8bcedb15f1b84663520a79b0e4a724e0cb28487b

refactor SignalBase and CallbackBase using variadic template.

Variadic template and parameter pack makes the Api list much shorter
and enables for future optimization.

Removed assertion from Callback::Execute if an empty callback has
been setup.

Fixed negative test case.

Change-Id: I11060630ef4b82fc9966610b5c63ef2fa27b3779

Merge "added Erase and EraseIf function to dali-vector" into devel/master

added Erase and EraseIf function to dali-vector

These functions uses erase-remove idiom for efficient
removal of more than one item from vector.
The signature is same as the standard erase() and erase_if()
function added to c++20.

Change-Id: I2d6769cde5b6acfbb03447b6576491021ab14fe1

skipping the render items outside of the partial rendering merged area.

Change-Id: I7a9019fc75600f71a5580c4a4956b6324399a2a3

refactor Signal class.

- current Signal<> class takes 3 Pointer size . 24byte in 64bit system and 12byte in 32bit system.
- The Actor class uses 11 siganls which takes 250bytes almost 50% of the Actor class size.
- With this patch the Signal<> object now occupies 1 pointer size. and allocates only when
  user connects a signal.

Change-Id: I5c96789ca995e50bf54890d1d9048dc4e1b276bb

refactor ConnectionTracker class.

- keep Dali::Vector<SlotConnection> instead of Dali::Vector<SlotConnection*>, this will
  reduce heap allocation as well as performance.

Change-Id: I1d0e89be58a41f89087708d8a59f2e0cbfa454df

refactor SlotConnection class.

- make SlotConnection a trivial type so that we can keep in container by value.

Change-Id: I331a838ccef8f9fecfd1a2b26830fb3ced801ee4

refactor CallbackBase class.

- Don't allocate objectdata separately as we allways heap allocate the CallbackBase object.
- keep operator==() in the header to give compiler opertunity for inlining.

Change-Id: Id17214b4c91d27849f67c953d526e4d3789a2cb4

refactor BaseSignal class.

- replace Dali::Vector<SignalConnection*> with std::vector<SignalConnection>
- Fixed Empty() and GetConnectionCount() implementaion complexity from O(n) to O(1).
- move trivial functions to header to make them inline.
- use erase-remove idiom to cleanup the empty connections.

Change-Id: I941bcf6b0b27a14d29a316f005f37bfca44c3530

refactor  SignalConnection class.

- make SignalConnection class a move-only type.
- with this change we can use this object as a valuetype
instead of pointer in the std::vector.

- this will reduce the heap allocation as well as search
  inside the container.

Change-Id: I87a0a4b38e979be23bfec75076fbda97ea73e932

Merge "Added small buffer optimization to Property::Value." into devel/master

Added small buffer optimization to Property::Value.

With SBO optimization we don't need to allocate memory
for the Pimpl pointer as we will be creating the object
inplace in th Storage buffer.
After this change the Property::Value size will be 16byte
instead of 8. I think the overhead is minimal comparing to
the number of allocation this class does in a dali application.

Note: with the SBO and union trick the implementaion is little
bit complex than before. but as this class is core for property
and animation and used every where. the complexity is justified
in my opinion.

Change-Id: Ibe14fe92f15c2ab93395c9731bd1fca902e98259

keep a static  vector to avoid memory allocation

Change-Id: I3ada71c7fd7d5d4ede47f769cc81723e22ac3a77

DALi Version 1.9.32

Change-Id: I0e3ec20ed93df4dffc7a6604ae3ef069036f1af7

Merge "partial rendering  - When calculating the area, a difference of about 1px occurs.    so change roundf to floor" into devel/master

Merge "partial rendering" into devel/master

Const correctness improvements for Property::Value.

GetArray() and GetMap() return non-const pointers, which allow const
objects being modified. While DALi itself doesn't seem to abuse this
in client code it may still lead to hard to trace bugs or UB at worst.

It makes sense that the modification of an element of a Property::Map
or Array that a Property::Value holds, means modification of the
Property::Value itself.

- GetArray/Map() const has been changed to return pointer-to-consts;
- a non-const counterpart has been added to each;

Change-Id: I187770bbb3cf355fbf1df95c909bad3ddc8cdb23
Signed-off-by: György Straub <g.straub@partner.samsung.com>

Refactor Property::Value::Impl class

- now Impl class object is 16byte for both 32 and 64 bit system.
- inplace constuct the object for type bool , int, float , vector2 , vector3 , Property::Map and Property::Array
- dosen't delete the Impl pointer when type dosen't match.
- all the constructos take argument by value (sink argument).
- marked move constructor and assignment operator as noexcept.

Change-Id: Ic5f4fb8956b67b00a7a68ba8f02d6963fb457221

partial rendering
 - When calculating the area, a difference of about 1px occurs.
   so change roundf to floor

Change-Id: Id07ecd26456288884585ca58bc2d33e4f8bddb7e

refcator demangler to avoid temporary vector and string creation.

-this patch reduces heap allocation by 130,000

Change-Id: Ic095319c11aea9e5235683f409b7ee846087d702

Ensuring CustomActor behaviour flags retain expected values

Change-Id: Ica035f04ac71d0f514af0dbe3920f235bd775907

DALi Version 1.9.31

Change-Id: Ia9d748e582577f2acb0bbfc85968fa9d12e8f92a

partial rendering

1. Modify dirtyRects to have scene for multi window support.
  - Looking at a single dirtyRect in multiple windows causes problems.

2. Add Scissor for partial rendering

Change-Id: Iec8455c2b948d4344ab805d4426e2a17a447af99

Merge "Not use texture bound cache in unexpected cases" into devel/master

Fix interface to avoid heap allocation.

- take Property::Value and std::string by value.
- use vector::push_back(T&&) version to avoid copy.
- this patch reduces 300,000 memory allocation.

Change-Id: I2da12f4b18cef5967ada18d6c9c58d2bc406d8ea

Merge "Include the algorithm header file" into devel/master

Include the algorithm header file

It is necessary by std::min and std::max. This fixes dali-demo build on
Windows.

Change-Id: Id10dd27f2b90b675ba68c1504f829853ace5071d

Not use texture bound cache in unexpected cases

Change-Id: I7866750a920a5c9da8c554088f72af501052a239

Merge "Fix interface to take sink argument by value." into devel/master

Merge "Internal::Actor Refactoring" into devel/master

Fix interface to take sink argument by value.

 -Fix RegisterProperty() interface to take name and property by value.
 -Fix SetProperty() interface to take property by value.
 -Fix RegisterProperty() interface to take name and property by value.

Change-Id: I04231da346a62d3aa56892453d36ed0c0b1657f5

DALi Version 1.9.30

Change-Id: Ib069886ed3b8bdf7d159e2fe5771b22d8de00816

Merge "Fix texture bound cache issue" into devel/master

Merge "Added API to apply native texture modification to shader" into devel/master

Internal::Actor Refactoring

Change-Id: Ibeb50bb81a8941fb1e72f8ca8d2db9c998085e7c

Fix texture bound cache issue

ActiveTexture should be called if mActiveTextureUnit is different.
We don't need to check it in BindTextureForUnit().

And the mBoundTextureId should store ids per target.

Change-Id: I30d93784577a2ce261ef047f2ec36c6adb766b0a

Merge "Fix AddUniformMapping() interface to take uniform name by value." into devel/master

Fix AddUniformMapping() interface to take uniform name by value.

this will avoid temporary string allocation.

Change-Id: I21e358150c3ac40496ae62c23562a77551e66acd

refactor PropertyMetadata table.

- keep string_view insted of const char* for performance.
- make both table constexpr so that we can sanity check during compile time.

Change-Id: I13a424b3f9c5fadd52ea02889ba6d687f8f41346

Merge "Ensure animation is not played when an actor is not on scene" into devel/master

Added API to apply native texture modification to shader

If a given texture is using a Native Image, then update the shader
using the custom prefix and sampler from that NativeImage.

Change-Id: Ic94e5b6eb90ad453d9e7e288837964634f20a2db
Signed-off-by: David Steele <david.steele@samsung.com>

DALi Version 1.9.29

Change-Id: I971d9664063dc8f7a9c178735103c39fa64eb4ed

Ensure animation is not played when an actor is not on scene

Change-Id: Iadfe60520ad14de6db2c1867bc6df223fa5bf10a

Formatting automated-tests

Change-Id: Ic0fd2e6eef75ad8af21511ba6b9383fa287009a4

Formatted API

Change-Id: I46852b09a374bb8115a2ce12959800f030e54c60

clang-format: added .clang-format with rules adhere to dali coding guidelines.

added ./format file which will apply the coding guideline to the latest commit at HEAD^ and
reformats  the files if it violates Dali style  guideline.

Change-Id: Ic265bdab2f43123eba72dd3317f2fda39248c2e4

Export Setter APIs in DevelKeyEvent

Change-Id: Id14c93d01b2cfad68ba2bb9a01b6fad457ab6231

Adding 'override', removing 'virtual' from overriding functions' declarations in tests.

Change-Id: I8954231725d60faf8caac222db73ebf7b5a4bfae
Signed-off-by: György Straub <g.straub@partner.samsung.com>

Merge "Add devel api to create valid handle for Gesture classes" into devel/master

Merge "Changed 'virtual' function override declarations to 'override' in automated-tests." into devel/master

Add devel api to create valid handle for Gesture classes

Change-Id: I63f180b330ad0da44f09ad0e323d89ec78101ae0

Merge "Fix indenting" into devel/master

Merge "use modern construct 'nullptr' instead of 'NULL' or '0'" into devel/master

Merge "Add Integration API to Create public event type" into devel/master

Merge "fix deep copy of the vector." into devel/master

Merge "use modern construct 'override' in the derive class." into devel/master

Merge "Make Update() function of frame callback is called for every frame." into devel/master

fix deep copy of the vector.

as the function returns the vector by const& and our temporary
object dosen't match the return type we make a deep copy of the vector.

Change-Id: Ie402d85962b138ed08bf00cbb3a8453190b8f812

use modern construct 'nullptr' instead of 'NULL' or '0'

This patch is auto generated by clang-tidy with modernize-use-nullptr check.

Change-Id: Iafb79e884e79ec02ebdf32ea687f519aceb1f4dc

use modern construct 'override' in the derive class.

this patch is auto generated by clang-tidy with modernize-use-override.

Change-Id: Id2a6abb4a6ce9725bd870dc6a846e8cea33b37ae

Fix indenting

Change-Id: I352dcbb67b54f928ee20695f63f009afa58f8cd7

Make Update() function of frame callback is called for every frame.

 - Currently updateScene flag is not be true even though frame callback is existed.
 - So, make updateScene true when frame callback processor is existed.

Change-Id: If5003ab5be3fa9293ef94ed2482e6cd36ac5166e
Signed-off-by: Seungho, Baek <sbsh.baek@samsung.com>

Use modern construct 'using' instead of typedef.

this patch is auto generated by clang-tidy using 'modernize-use-using' check.

Change-Id: I269203c391d6a676db65a28ac7dd4b19301bdbd1

Changed 'virtual' function override declarations to 'override' in automated-tests.

Change-Id: I39b03b7d75ceb50c20ca2b964eb65fa488827040
Signed-off-by: György Straub <g.straub@partner.samsung.com>

[dali_1.9.28] Merge branch 'devel/master'

Change-Id: I3f7cd18342490e72c5c7b05349a25f3036facde4