Avoid Svace MEMORY_LEAK warning in callback.cpp

Change-Id: I7be6559a82035993e8607b282b6ecc47ec365264
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

DALi Version 1.2.40

Change-Id: I707fcf7f3e9584a854a3ef9d5a4906f21dedd8a5

(Animation) Ensure TimePeriod is taken into account when setting properties on the event-thread as well

Also added some more test cases to test the AnimateTo event side properties

Change-Id: Ibeeb8f01316034715eb9121dfec61c18e18eeb5d

Enable -Wold-style-cast to ensure more consideration is given to types and their conversions

Change-Id: I8b8e92f09434e52ad62353d54a2846fae96fb3c3

Merge "Enable -Woverloaded-virtual to enforce correct C++ behaviour when overriding virtuals" into devel/master

Merge "Enable -Wnon-virtual-dtor to avoid incorrect C++ code sneaking in" into devel/master

Merge changes I84e1ce43,I76b93d51 into devel/master

* changes:
  (Animation) Reorder animation variables according to size & remove unused
  Ensure cached values of properties animated using AnimateTo are updated

Enable -Woverloaded-virtual to enforce correct C++ behaviour when overriding virtuals

Change-Id: I36aae7341320f43957639473d7f6d933ff815172

Enable -Wnon-virtual-dtor to avoid incorrect C++ code sneaking in

Change-Id: Ic70f3e0c688ca3f6ea191842ae0f7f4271330b19

Fix support for NativeImage

- Fixed program to recognise uniforms with type: GL_SAMPLER_EXTERNAL_OES
- Fixed Framebuffer renderer to use GL_TEXTURE_EXTERNAL_OES

Change-Id: I337ab3f461aa43fd10b311895b2f186b2462e617

(Animation) Reorder animation variables according to size & remove unused

Change-Id: I84e1ce4388ec0b31d4077a51fe0f8769dc0464bf

Ensure cached values of properties animated using AnimateTo are updated

Change-Id: I76b93d51c807c6be75ed419e01b0fa5fdb575c08

Merge "Remove duplicated Erase iteration code by introducing EraseObject method in OwnerContainer" into devel/master

Merge "Initialize 'mDepthTreeDirty' member in stage-impl.cpp" into devel/master

Merge "Ensure animations take TimePeriod into account when running animators" into devel/master

Fix the USE_ASSIGNED_SIZE resize policy to not overwrite the actor's original policy

OnRelayout is called during the relayout process, and if the actor has calculated
the size of child actors it will add them to the relayout container with their
desired size and set the ResizePolicy::USE_ASSIGNED_SIZE resize policy on them.
However, by doing this, it overwrites the original resize policy of the actor,
so the size calculation of the actor could be incorrect during any further
relayout process.

This patch fixes the above issue by setting an internal flag instead of overwriting
the original resize policy in order to notify the size negotiation algorithm to
assign the size of child actors as calculated during OnRelayout.

Change-Id: Ib1b2e4cfc2a39cfd653e64291e86508351d15606

Ensure animations take TimePeriod into account when running animators

Issue is that when animating the same property using animators, the final value would be whatever animator came last
rather than the taking the end time of the animator into account.

Now, we order the animators according to the end time.

Change-Id: Ic51513dda942caaf9a9c7b339b5bd41431f5116c

Merge "Build dali with C++11" into devel/master

DALi Version 1.2.39

Change-Id: I23da6f7cae22f0dfe7c0f278bf5e650703368b26

Remove duplicated Erase iteration code by introducing EraseObject method in OwnerContainer

Change-Id: I3317c7e1347ee0f07b6648202bf3a640498134a0

Initialize 'mDepthTreeDirty' member in stage-impl.cpp

Change-Id: I44fa68e50c273e2b0be4f0f6a7136a5e6bcf88ce
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Remove Gesture container as there is only ever one pan gesture scene object

- scene pan gesture is created in core construction and never dies so no need to send an add/remove message or have a container to iterate; just ownerpointer is enough

Change-Id: Idd7b8be54cade3206778382410287d8e3a290b7f

Updated Renderer lifecycle test case to test all objects.

Renderer Add/Remove positive test case doesn't actually test the
renderer lifecycle. Added extra scoping and calls to Application.Render
to test the lifecyle of scenegraph and render objects as well.

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

Build dali with C++11

Change-Id: I8059c47f591edf89c94c53ffeb38bd5a91564c6c

DALi Version 1.2.38

Change-Id: I13320d43a6018759925692ebf46f1f45f3a5252e

Remove obsolete template and harmonize UpdateManager code

- typedef containers in same naming convention
- use helpers to iterate containers
- remove obsolete object owner container and related unnecessary templated code
- put method definitions in declaration order
- data members in size order

Change-Id: I650f7b6714390504294d8d31111890fea8bc9059

Merge "Remove dead code for render needing an update (no such case at the moment)" into devel/master

Merge "(Actor) Fix mismatch between GetTargetSize method and Size property" into devel/master

Added new mechanism to order actor depths

At the end of each frame, if the actor tree was changed, or the
sibling order of any actor was modified, then the actor tree is
traversed to determine the new depth orders.

Currently, the depth order is written back to each actor's mSortedDepth,
and the nodes are updated with a single message containing the nodes
in depth order. (Could be used in a future patch to reduce renderer sorting)

Created a memory pool for managing the lifetime of the depth nodes that
exists for the lifetime of the application.

Change-Id: Iaed004eb8dc790fcc0f6f1b4822037054193b552

(Actor) Fix mismatch between GetTargetSize method and Size property

Change-Id: I797cab204a260164af314e92d6d40277c643d483

Remove dead code for render needing an update (no such case at the moment)

Change-Id: I68816d3e629fbb87edb1f6c66ebfd205140947e6

DALi Version 1.2.37

Change-Id: I38702c7138df65247b61195c358ce83701ce4e6f

Merge "Synchronous Set/Get behaviour for default properties" into devel/master

Synchronous Set/Get behaviour for default properties

Change-Id: I70cf839cc6d41eb9f18de6be92b1c33ce30cb41b

Cleanup: Remove handling for default shader as it has been long gone from dali-core

Change-Id: I8fe8fe593b562e45f8c9a2730a81fdafdd224977

Merge "Add LICENSE.BSD-3-Clause file" into devel/master

Merge "SVACE issue resolved" into devel/master

Add LICENSE.BSD-3-Clause file

- BSD 2.0 is the wrong expression of BSD-3-Clause. Fixed it.

Change-Id: I7467c2e57ec016a3af757e59479cd7058a72668f
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Added Visibility Change Signal & added macro to print meaningful line numbers in test checking functions

Change-Id: I4120f711f78446153cb9552018f4fc1d6e3ddfd3

Merge "Adding deviceClass to KeyEvent Integ" into devel/master

Merge "Synchronous Set/Get Behaviour for custom & animatable properties" into devel/master

DALi Version 1.2.36

Change-Id: Ic9e0e7bfec6cfc850864664480a4a07af5a9d7a9

SVACE issue resolved

Change-Id: I085825149a1e9611824798571322d23ae0e4e890

Adding deviceClass to KeyEvent Integ

Change-Id: I8cf899bf17d93d0462571b01d69697bf58a6ff22

Synchronous Set/Get Behaviour for custom & animatable properties

TODO: Default properties
      Updating the property value when animating

Change-Id: I3bd09cc4c6f89edffcf4f95d8461b9a54e76d840

Ensured consistency of scene graph and render items

There was a timing issue where UpdateManager's updateScene could be
false, but the messageQueue flushed messages to the process queue just
prior to ProcessSMessages being called which updated the scene graph.

Have ensured that ProcessMessages returns true if there were updates
to the scene graph, and changed UpdateManager to process the scene
appropriately.

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

Key event copy and assignment ops added

Change-Id: Ia2057b4fe23e60e2fb44c293fc1680cbe6434c97

DALi Version 1.2.35

Change-Id: I28215672d98cf5a84ea8bb979f5b9a9adaa2c6f7

Adding deviceName to KeyEvent Integ

Change-Id: If40b705d8e5ba0e42808365afecdf5fb2ead43ae

Merge "Adding Dali::KeyEvent convertor" into devel/master

Merge "Fix disconnection issue of signal" into devel/master

Adding Dali::KeyEvent convertor

Change-Id: I369dfdffa8171611b5796a0576eb5633268d2300

Fix disconnection issue of signal

The item is not removed from mSignalConnections until the sinal is emitted.
Changed to remove the item if the function is not called from Emit().

Change-Id: I0288b68dac3f601790aa16e77ab78b22e437ae44

Avoid Svace DEREF_OF_NULL warning in constraint-impl.h

(reduces the number of False positives)

Change-Id: I3d1895288865f2e19c44ec279131e7303ce7d915

Merge "DALi Version 1.2.34" into devel/master

DALi Version 1.2.34

Change-Id: Iaf895fa2d2bcd0aebc805707e7e8f9c5103c92b4

Added stage-devel to support KeyEvent propagation

To support KeyEvent propagation, made stage-devel.

In stage-devel, there is one new signal.

 Signal< bool( const Dali::KeyEvent& ) > KeyEventGeneratedSignal
 : Emitted when KeyEvent is generated

And also, now when Propagation result is false, then emit
Stage::KeyEventSignal for app & KeyboardFocus.

Change-Id: Ibd95ba2e082f4d3ee43ddf51b5ec6c09010164fb
Signed-off-by: minho.sun <minho.sun@samsung.com>

Ensure SCREEN_POSITION value takes POSITION_USES_ANCHOR_POINT property into account

Change-Id: I685f336c8ff28e6af4f88d3f5ac7ded804207c23

change dead link to new link

Change dead Native API reference link to new link

Change-Id: I3247a29ffeba6cbb128bca7c88400672b077f711
Signed-off-by: minho.sun <minho.sun@samsung.com>

DALi Version 1.2.33

Change-Id: I5629a48c2021e06435d05c5dd8ac4a64c27b71bf

Add doxygen comment in WeightObject namespace

- Add a comment to show WeightObject namespace in the doxygen file.

Change-Id: I5cc51499f76791376924145fafe6a75f6ef23328
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Merge "Valgrind detected TextureSet leak and invalid access" into devel/master

Merge changes Ibcc99fc9,I8d1202b2 into devel/master

* changes:
  Remove manual memory management from Core implementation.
  Fix potencial bug with owner pointer.

Valgrind detected TextureSet leak and invalid access

Memory leak:

If a public API TextureSet gets deleted, it sends a
message to update-manager (RemoveTextureSet).
The scene graph side TextureSet gets removed from
update manager, but it never gets deleted.

Invalid memory access:

Currently when a scene graph TextureSet gets deleted
the renderers using that TextureSet are not notified.

So when a Render gets deleted it would call
TextureSet->RemoveObserver( this );

on an already deleted TextureSet.

DALi however doesn't crash at this point,
because the TextureSet is stored in a memory
 pool so the object is still reachable.

Change-Id: Icf0f5a3e55d3ba7537f40db08accad04ae4440f7

Remove manual memory management from Core implementation.

Change-Id: Ibcc99fc9235a8c0f35a14440372d519e2fab7b04

Merge "Fix to avoid DEREF_AFTER_NULL Svace issue in any.h" into devel/master

Fix potencial bug with owner pointer.

Change-Id: I8d1202b29bb71ce779502e4ce19c01a89207b696

Merge "packaging/dali.spec use license macro" into devel/master

(Renderer) Fix memory leak when using blend-color

Change-Id: Iaf7a0ae464fa0c354e1111f9313972349a2b9b45

DALi Version 1.2.32

Change-Id: I578f410ee51348c1650dbd4b9f92008951f5865b

packaging/dali.spec use license macro

Change-Id: I2c4666e6c0b4662190f41f4945b5496dacc54b3b

Fix to avoid DEREF_AFTER_NULL Svace issue in any.h

Change-Id: I4e87718579bc208fccf498bc3204b109303abb84

Merge "Target Values of animation only set on ::Play()" into devel/master

Ensure width & height properties can be set individually

Change-Id: Ie8d7ee230a87f2152c8ad6320839c46bc70a2a5f

Target Values of animation only set on ::Play()

Change-Id: I834e151b4f06a08def523cf9d915fcafa79fd806

DALi Version 1.2.31

Change-Id: I409fc5db91f42d484d34ceee94e314b263b10b23

Added Getters for Animator Connectors base class

Change-Id: I90c6afe299dd565261fe733c4fbe3c53df1988d2

DALi Version 1.2.30

Change-Id: If1de12dcfcde05eabb9cd9647dbdbca115c7ef7b

Added a property to allow an actor to ignore the anchor-point for its position

In that scenario, TOP_LEFT is assumed.

Change-Id: Id57829b9488b0f3d3546354d51900941ca92affa

Merge "Fix memory leaks detected by Valgrind" into devel/master

Added Get Screen Position functionality to Actor

Screen coordinates TOP_LEFT is 0,0
This property will return the position of the Actor relative to the TOP_LEFT
The returned position is the position of the actor's Anchor point.
Actor Scaling is supported.
Actor Rotation is not supported.

Change-Id: I8519f2b82ab9f9ac8eec197e783a66dcf5ba949c

Fix memory leaks detected by Valgrind

Fixes the following memory leaks reported by Valgrind
that can occur when messages with raw pointers are sent
from event -> update (and update -> render)
and dali shutdowns before update or render has had chance
to process those messages

- AddSampler ( sampler is lost )
- AddPropertyBuffer ( property buffer is lost )
- SetPropertyBufferFormat( property buffer format is lost )
- AddGeometry ( geometry data  is lost )
- AddTexture ( texture data is lost )

event -> update
- AddCameraMessage ( Camera is lost )
- AddObjectMessage ( Object is lost )
- AddPropertyNotificationMessage ( PropertyNotification is lost )
- AddGestureMessage ( PanGesture is lost )
- AddSamplerMessage( sampler is lost )
- AddPropertyBuffer( property buffer is lost )
- SetPropertyBufferFormat( property buffer format is lost )
- SetPropertyBufferData( property data is lost )
- AddGeometry( geometry is lost)

Also removes PropertyBuffer data and format from being duplicated
on the event side.

Change-Id: Iac6d446bd2b5372cea7ec101e2e289ef3f284b4c

Merge "Added checks to Sibling Order Raise/Lower API" into devel/master

Merge "Valgrind fix for uninitialized values in Render::Sampler" into devel/master

Added checks to Sibling Order Raise/Lower API

Added checks to ensure target actor and actor raising or lowering belong to the same parent

Also checks that both actors are parented.

Change-Id: I346ead160a72cad12192cf4f6b771cf82bb43930

DALi Version 1.2.29

Change-Id: I4589137dbd69c6c6cbbdece518324b146a58b45a

Valgrind fix for uninitialized values in Render::Sampler

Valgrind reported conditional jump depends on uninit values in:
  if( mSampler.mBitField != oldSampler.mBitField ) // Texture::ApplySampler

Problem was down to Sampler having a anonymous union:

  union
  {
    unsigned int mBitfield;

    struct
    {
      FilterMode  mMinificationFilter   : 4;    ///< The minify filter
      FilterMode  mMagnificationFilter  : 4;    ///< The magnify filter
      WrapMode    mSWrapMode            : 4;    ///< The horizontal wrap mode
      WrapMode    mTWrapMode            : 4;    ///< The vertical wrap mode
      WrapMode    mRWrapMode            : 4;    ///< The vertical wrap mode
    };
  };

Fundamentally the size of the bit field struct
can be larger than the size of the unsigned int mBitfield, so using mBitfield to compare Samplers
isn't reliable.

The issue valgrind picked up on is, only 20 bits of the
mBitfield are being set in the sampler constructor
  Sampler()
  :mMinificationFilter(FilterMode::DEFAULT),
   mMagnificationFilter(FilterMode::DEFAULT),
   mSWrapMode(WrapMode::DEFAULT),
   mTWrapMode(WrapMode::DEFAULT),
   mRWrapMode(WrapMode::DEFAULT)
  {}

Solution is to remove the union

Change-Id: Id66afc05d21377a1363448ef10654cd1ab4bb365

Merge "Fix Message Buffer leak and added DEBUG_ASSERT for leaking nodes" into devel/master

Merge "(Actor) Fix documentation in sibling order methods" into devel/master

(Actor) Fix documentation in sibling order methods

Change-Id: Ib3ffe6e8c8f5e1f249f0a6f5e5439e174f883a4f

Check memory being free'd from a memory pool, belongs to it.

Memory leak detected by Valgrind due to Scene Graph Layers
being deleted from a memory pool, even though they were heap
allocated.

Assert currently fires on shutting down dali-demo, and when
using text-editor.example

Leak fixed in
https://review.tizen.org/gerrit/#/c/115115/

Change-Id: I1254b56a3f3be17d662fb0622ebe407874df71cc

Fix Message Buffer leak and added DEBUG_ASSERT for leaking nodes

Current message queue is not cleared when update is destroyed
This is 1 cause of Node leak.
Second cause of Node leak is InstallRootMessage passes a layer
as a raw pointer. If update doesn't run ( doesn't run in the test
cases), then we leak the layer (node).

Also added DEBUG_ASSERT to check for leaking scene graph Nodes

Change-Id: Ibfbb73ee840cbeaf33104da6769945cf9b3aafbc

Fix memory leak, scene graph layers are never deleted from memory

Reported by Valgrind.
We have a memory pool for Nodes which works fine
for creating /deleting nodes.

Scene Graph Layers inherit from Nodes, but don't use
the memory pool.

However when either a Node or a Layer is deleted, the overloaded
Node operator delete( void* ptr ) is called, which tries to free
the Node from the memory pool.
Unfortunately for Layer it was never in the memory pool so no
memory is free'd.

Fix includes:
node destructor is protected
layer destructor is private
node delete operator removed

Change-Id: Icfe83f35b4f29d5b774cde392ff976ad299481c5

Adding Raise Lower API to Actor

Devel API to allow the raising and lowering of the Actor's sibbling level.

Change-Id: I86c03d43d962441fd295dbf024092d42e78d9c0e

Merge "Removed redundant resource loading & rendering code" into devel/master

DALi Version 1.2.28

Change-Id: If7991a9a2a637f145136669dffc6908f927661c1

Add Opacity property for Actor

Change-Id: I7fefcf77e512b4d0954666d4187d116dc5d518bd

Removed redundant resource loading & rendering code

Change-Id: I369e0e907f452125eb97059f853e9e16fa31f158

Test harness: Moved CompareType<> templates to own header

Change-Id: I367a21562a7c6362a15a13bfcc6fc8fffeb37c3b

Merge "DALi Version 1.2.27" into devel/master