Clean up the code to build successfully on macOS

This commit does some code cleanup to successfully build dali-core in
macOS using the Apple provided clang compiler.

The clang version shipped with macOS doesn't have the
-Wno-class-memaccess and the -Wno-cast-function-type switches, so we
test their existence before adding them to the compiler options.

Mutex now has move constructor and assigment operator, so classes with
mutexes members can use their defaults move operations. Classes holding
references cannot have a default move assigment operator.

Virtual functions overriding parent classes are now declared with the
`override` keyword.

MAXIMUM_TIME_DIFF_ALLOWED is a global internal variable nerver used, so
it was removed.

.gitignore was updated to include some artifacts from coc-nvim
extension.

Change-Id: I76201f867240a0822e7e202c2edcbfa07b2dffd6

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

Change-Id: I9ad15755abccc4c455918b049bec38c1bcb84b4d

DALi Version 1.9.34

Change-Id: I5f8723bdcce4d478589a8dc72cea88180882d69f

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

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_1.9.33] Merge branch 'devel/master'

Change-Id: I58ae05dfbcbb8b9a80a3548de1c051444e9c9f8f

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

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

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_1.9.32] Merge branch 'devel/master'

Change-Id: I273fcca7812387ae4d62a455aa69c43ea8f95c9f

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_1.9.31] Merge branch 'devel/master'

Change-Id: Id3ff79a9e7c09c3aa7b4e7bba1a854913556b762

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_1.9.30] Merge branch 'devel/master'

Change-Id: Ie458776623bc8eabd38c5d20534e1bc5b596c500

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_1.9.29] Merge branch 'devel/master'

Change-Id: I80d089c67fbb756a2d86e45b67d5ed0883f08130

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

DALi Version 1.9.28

Change-Id: I1dc2f217b6ef5c25114c7dc1e03bd2cee4055d19

Add Integration API to Create public event type

Add intergration API to support GlWindow.
It is used gl-window-impl in dali-adaptor.

Change-Id: I296ed587911135c3fea616f7114352f5a6af6a9e

Merge "Removed On(...)Event()" into devel/master

Removed On(...)Event()

Change-Id: Ia14b294a1fbdde5bf82cac93ff3101285f86727f

change Object::OnPropertySet() interface signature.

the old interface was taking Property::Value object by value which
was making a deep copy (2 allocation). so changed the interface to
take it as a const& object.

Change-Id: I6cd67f6900ea61ea47154f9345a9ffe8f949f816

Moved Gesture::State and -::Type to gesture-enumerations.h.

- GestureState is now an enum class;
- Their values have been changed to upper case in compliance with the
  DALi coding standards;
- Added tests for Pinch- & TapGestureProcessor abort cases;

Change-Id: Iae9414e0f771535dffe6373e448c8769131f1b6d

Merge "Renamed TouchSignal to TouchedSignal" into devel/master

Renamed TouchSignal to TouchedSignal

Change-Id: Iee038875f65be610804e4209c0be22be9914df02

Add devel API to create an initialized HoverEvent handle

Change-Id: I8860e77d8abce7d2097cdd146316dff936303086

Merge "Added move semantics to TouchEvent & fixed some other internal event classes" into devel/master

Added move semantics to TouchEvent & fixed some other internal event classes

Change-Id: Iaff0eb50d4cdb9b7e481cfe5def5ad850c58eb92

Merge "Moved (DEFAULT|DEBUG)_BACKGROUND_COLOR out of Stage and over to constants.h" into devel/master

Refactoring Gestures Class

Change-Id: I5e11996bc2a02f7fb0e9ebe958d8c8b794912635

Moved (DEFAULT|DEBUG)_BACKGROUND_COLOR out of Stage and over to constants.h

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

Renaming of enum values for coding standards compliance.

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

Move TouchPoint to Devel API

Change-Id: I8c9bc63e159f64593998281e1cff85ef83c13752

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

Change-Id: I46a29786191b3e10f8205eea2b56c4637f553c26

DALi Version 1.9.27

Change-Id: I1452176bdf4ece1a22c77e0c7127fd5fb53955ac

Merge "Changed const variables to constexpr where possible." into devel/master