Fix gcc-14 build error

Let we include vector wrapper for custom actor impl

Change-Id: I320147a2fe622168c8a1e289c3470117de278036
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "Add CommandBufferResourceBinding add resource well" into devel/master

Merge "Automatic RenderTask Ordering" into devel/master

Add CommandBufferResourceBinding add resource well

Looks typo error

Change-Id: I0776658f3e49bf73046ad2e55b3c3014daad8796
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Automatic RenderTask Ordering

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

Add SamplingMode type for high quality sampling

Let we make new enum for lanczos resize method.

Change-Id: I2f7f0db85c33f6d913318b6be90a69217ae68cbe
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reset string shader data cache at event thread if too much shader used

Change-Id: Ie157d613d9318ca378f48d4e34fe75d7230ea4f4
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.42

Change-Id: I8a64f09b2b1436e7da726464cebe0b938e09d7eb

Merge "Allow to animate by difference type if it is scalar value + Convert API for Property::Value" into devel/master

Added resource binding information to command buffers

Change-Id: If9320070962ee1145a1f4b1f4d149942221a3544

Adding render pass & target to CmdBuffer BeginInfo

Change-Id: I79315941628b505ece31c4ba7e92070b2cb46c47

Merge "DALi Version 2.3.41" into devel/master

Merge "Make Renderer use MixColor + Deprecate DevelRenderer's Opacity and VisualRenderer MixColor and PreMultiplyAlpha" into devel/master

DALi Version 2.3.41

Change-Id: I71b6b3bdcb9c49312cd61eb501ece91370511d87

Allow to animate by difference type if it is scalar value + Convert API for Property::Value

Let we allow to set destinaton value of animation not matched with
animatable property.

For example, until now we don't allow to animate POSITION_X as integer.
But after this patch, we can animate by integer type.

Change-Id: Ie332533ffad53b511c697501c3b04d68848734df
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Observer StopperNode and ViewportGuide node at SG::RenderTask

Let we clear the SceneGraph::RenderTask node information what were they don't owned.

Without observing, we might try to access dead node.

The two node - mViewportGuideNode and mStopperNode was not observered, so
let we observe their lifetime, and remove nullptr if they terminated.

Change-Id: I8c55248c625542650e4b386ba538419b5b231546
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make Renderer use MixColor + Deprecate DevelRenderer's Opacity and VisualRenderer MixColor and PreMultiplyAlpha

Let we make MixColor as Renderer property.

And also, make VisualRenderer's MixColor use Renderer's MixColor property directly.

And also, make we don't register preMultipliedAlpha uniform for whole visual renderer.

Due to the Visual MixColor was Vector3 and Renderer MixColor is Vector4,
we need to make VisualMixColor non-animatable.

TODO : Need to change toolkit side codes.

Change-Id: Ida7b788be00ef7b14bba7d86d937a4581726339d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Release graphics contexts before terminate graphics

Since some graphics items deleted after graphics controller shutdown called,
some memory leak trace tool check they are leaking points.

More over, for offscreen application cases,
DestroyResource API was not called at terminate cases.

To make insure those are removed well,
let we call ContextDestroyed for each manager.

Change-Id: I24293d1de7b95a56fb384374c667c69bea5e4180
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Different mouse button types must be handled as separate events." into devel/master

Different mouse button types must be handled as separate events.

Change-Id: I7081ca145f61dd1240cc9f8f2b662001dae3b393

Add BaseObjectObserver

Change-Id: I414126bece5aeb685835c1efa7c6ab8fba9ee67e

Merge "Passing down render target to compile the pipeline" into devel/master

DALi Version 2.3.40

Change-Id: I105101bf3978c4337b77cfbab8c1209dc8f24aed

Passing down render target to compile the pipeline

Change-Id: Ia544d2661ffbef7c6392e550ee8119e9dc993818

Changing vulkan package config libs to match 1.3.280

Change-Id: Ia34fa62e1f21de3aaa5ca87b387230f2dfb1d7ff

Merge "Fixed dali_env to work non-interactively" into devel/master

Fixed dali_env to work non-interactively

Change-Id: I02839104d46578a03775c68f405f8555d03932ec

DALi Version 2.3.39

Change-Id: I654263054fd27e8741452438b0399a5f97a842bf

Merge changes I6482dca0,Icee988fc into devel/master

* changes:
  Add vulkan sdk install options to dali_env
  Fixing divide by zero in ThreadPool

Merge "Implement virtual pre-desturctor for RefObject, BaseObject and Object" into devel/master

Add exclude for sam score

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

Implement virtual pre-desturctor for RefObject, BaseObject and Object

Since WeakHandle<~~> become invalidate after BaseObject::Impl destruct,
WeakHandle<~~>.GetBaseHandle() might return some destructing handle.

If some class A : public BaseObject  is destructing (== refcount is 0)

and during ~A() called, someone try to get WeakHandle<A>.GetHandle()

After that API call, class A's refcount become 1, and if A use done,

A's refcount become 0 again, and destructor call again and again and again...

===

To avoid this kind of issue, let we make BaseObject::Impl Notify first

by virtual function "OnDestroy", and then call destructor.

Similary, Object also have it's own observer system. Let also notify them
destroyed before destructor call.

Change-Id: I1bccf25ec2b902b7051a0fe2bebf4aeb1b914c25
Signed-off-by: Eunkiki Hong <eunkiki.hong@samsung.com>

Add vulkan sdk install options to dali_env

Change-Id: I6482dca0fbd782bb94cd56ae76a1695341320dd1

Fix hash function error at x64

Since RendererKey is not a void* type for __LP64__ case,
we should not shift RendererKey.Value() based on size of Render::Renderer
struct size.

Change-Id: If578d0b5057b93ee69468d8422613471288d9384
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fixing divide by zero in ThreadPool

Change-Id: Icee988fca4b194ad70243676aa816519a64b51f3

Merge "DALi Version 2.3.38" into devel/master

DALi Version 2.3.38

Change-Id: I224ca546735c6207f41db8d39db2097b100e6848

Fix RenderUntil not working well if stopper node is under layer + RenderUntil more stable

For now, RenderUntil consider the layer order at
render-instruction-processor side.

But RenderUntil API only consider root layer cases.

To fix this issue, let we don't check the stopper actor's parenet
is layer or not.

+

Let we make RenderUntil API more stable.

Currently, we only allow given stopper actor is under source actor.
But actually, we don't trace that stopper actor unparent or destroyed
after we call RenderUntil.

Trace that life control is quite heavy. Instead, let we make RenderUntil input
actor added freely, and instead, make the task result will render full scene
if stopper node is not under source node.

Change-Id: I9550c4c77e550d9daa61fe545dc005417a4a3f2a
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(dali_env) Add libmagick++-dev install as well

Required for the visual tests

Change-Id: Ibcd3d0a2a03d05f6ac0d7db82787b544e7b117f7

Merge "DALi Version 2.3.37" into devel/master

DALi Version 2.3.37

Change-Id: Iaab72d254336690e55ac5f8ac2a5e69b258714a6

Merge "(dali_env) Remove --force-yes and use --allow-change-held-packages" into devel/master

Merge "Fix build error for gcc-14" into devel/master

Merge changes I9c9f8abe,I5ef804f5 into devel/master

* changes:
  Change TransformManager dirty flags only if the value is really changed
  Make TransformManager check WorldMatrix dirty, instead local matrix dirty

Fix build error for gcc-14

(Cherry-pick https://review.tizen.org/gerrit/c/platform/core/uifw/dali-core/+/312451 and improve.)

Resolve compile warnings : -Wno-template-id-cdtor, -Wno-dangling-reference

Can't locate XML/Parser.pm in @INC (you may need to install the XML::Parser module) (@INC entries checked: /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.38.2 /usr/local/share/perl/5.38.2 /usr/lib/x86_64-linux-gnu/perl5/5.38 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl-base /usr/lib/x86_64-linux-gnu/perl/5.38 /usr/share/perl/5.38 /usr/local/lib/site_perl) at scripts/output_summary.pl line 6.
BEGIN failed--compilation aborted at scripts/output_summary.pl line 6

Change-Id: Ib00e042a7197d5bcb92b33af8059fc0d5c1b8b92

Change TransformManager dirty flags only if the value is really changed

Let we check the current value and inputed value, so if they are same, ignore.

It will be useful when we don't want to mark dirty for transform compoent property doesn't changed.

TODO : Could we check them also at Vector3?

Change-Id: I9c9f8abe7615ea53856b20706ea9163456105bc3
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Make TransformManager check WorldMatrix dirty, instead local matrix dirty

Since we only need to check parent's world matrix + parent's size
to calculate the world matrix,

Dirtyness of local matrix is not important. Instead, we'd better check
the world matrix changeness.

Change-Id: I5ef804f54300cbad355d7b265e42cd109d2b5d81
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Match trace begin/end tag for DALI_RENDER_FINISHED if their is no target

Change-Id: I7f90ce6e082e8e1495b474a3f3524a4c4a0821e4
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(dali_env) Remove --force-yes and use --allow-change-held-packages

W: --force-yes is deprecated, use one of the options starting with --allow instead.

And I think two option : --allow-downgrades, --allow-change-held-packages is enough

Change-Id: I936f8a0dcd11b9aedb0a2c23d2910119e37c18da
Signed-off-by: Eunkiki Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.3.36" into devel/master

DALi Version 2.3.36

Change-Id: Ibf1efde8eea3cdfeb8737ef084275722e58d6233

The DEPTH value was being retrieved incorrectly. Fix this

Change-Id: Ie0c7556bdaaa661cdfbcf5b298166a94048151fa

Reset memory pool when core terminate

Since we can create new Core without process terminate, (e.g. offscreen application)
We should ensurely clear the memory usage.

Without this clean-up process, some memory might be remained after
core terminate. For example, some OwnerPointer<Node> are exist at Message during
app terminate, it might not be release surely.

Change-Id: Iaef0b16bcd6631aa8a52e9b52bfa39374be770b6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix UTC build error

Change-Id: Ie78675bd4a10301423214d163f4c38fe99599451
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Get the actor who trigger visiblity changed signal" into devel/master

Merge "InheritedVisibilityChangedCallback with window Visibility" into devel/master

Change the trace filter name : DALI_TRACE_UPDATE/RENDER_PROCESS

Now VD performance tracing system prepare the environment value name.

Let we hide the combined keyword at dali core side.

Change-Id: I17dc2cded6b89a1f593335299a092d3034d9ccdf
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix compile error at gcc-13 (Do not use operator< with nullptr)

Change-Id: Ic40ab0752eebbc33f13d1cd7c6f0d14ac424d8c6
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "Print more logs when (animation finished case + previous state)" into devel/master

Merge "Allow to compare IntrusivePtr comparision" into devel/master

Merge "DALi Version 2.3.35" into devel/master

Print more logs when (animation finished case + previous state)

Since we don't print any logs if animation finished naturally,
user might not know whether given rendering loops occured due to animation, or not.

To make ensure the animation state changeness, let we also print log
if animation finished naturally.

Also, print the log before change the animation state. It will be useful
when we trace the animation state changing.

Change-Id: I642057b56b86b2aca27874995ee1ff9adcfae30e
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.35

Change-Id: I463177231febbb34e2e1ef3d8607ebe3f11fdc6a

Merge "Make bake flag don't chage the set dirty flag for VisualRenderer" into devel/master

Get the actor who trigger visiblity changed signal

Since we cannot change the parameter of Visibility changed signal,
we need to make another API to get the actor who trigger visibility changed.

Change-Id: I990338910efc72fda941a66b0c9a802b22ff7f49
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add GetLastPanGestureState

Change-Id: Ica2861de9274023bb6f0c7b8c79c105224badea8

Allow to compare IntrusivePtr comparision

Let we allow to compare IntrusivePtr operator<.

After this patch, we can use std::set<IntrusivePtr<T>> or map, without
custom comparisoin functor.

Change-Id: Ifce5ad1fe951837d21766daba4cda1546c263262
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make bake flag don't chage the set dirty flag for VisualRenderer

Since VisualRendererProperty try to call AnimatablePropertyBase::OnBake directly,
it might change the overlaped bake flags.

To avoid this issue, let we call AnimableProperty<T>::OnBake, instead of
supersuper class

Change-Id: Ic57c34834b46e889f438b7b19a8a592e77a7fbcb
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

InheritedVisibilityChangedCallback with window Visibility

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

Make Actor's name don't use ConstString

Since ConstString register into internal string pool
and never be released,
Their might be occured some memory limits if user use randomized name.

Until now, we use raw string comparision at FindChildByName.
So their is no performance issue after this patch.

TODO : How could we resolve same issue for RegisterProperty?

Change-Id: Ia4258d204e986c117f19489a075e3c5afff9fab9
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.3.34" into devel/master

DALi Version 2.3.34

Change-Id: I41cd021c91e5e4ca71e5aa9bc36b9dfe3b974b54

Merge "Cache string based shader if required" into devel/master

Merge "Trigger Program cleaning by the size of cache container" into devel/master

Make bake flag dont change set dirty flag

Since we can bake componet side, it may have problem
if component property baked, and full property is set.

In this case, property call reset to base value only 1 times.
So it will show flickering.

To fix this issue, let we make mDirtyFlag |= BAKED_FLAG instead of
mDirtyFlag = BAKED_FLAG

Change-Id: I7269ac79ab1ec7796f62eaea438ee47dd104e963
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Trigger Program cleaning by the size of cache container

Current logic try to clean program cache every 10 seconds.
That mean, we need to re-create program.
It might make some unneccessary performance slow-down actions.

To avoid it, let we try to GC only if the number of program cache size
is bigger than threshold, not always.

And also, for every GC time, we delete only 5 programs incrementally per each frame.
So make we less-block the rendering time.

It will be useful for real world app who don't use custom shader.

Change-Id: Ie4e3bfb3486984673caa79750f55d813f256c0bf
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Cache string based shader if required

Let we cache string based shader's ShaderData.
It will be useful to avoid use multiple ShaderData creation
even if we use same source code.

Change-Id: Ifeaa9c2fa47b402690dab9b650b01ce874002069
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Do not calculate transform if there is nothing changed as transform" into devel/master

Add internal UTC to use ShaderFactory.SaveShader

Since there was no UTC relative with it, platform abstraction code become broken.

To make code clean, let we implement something internal UTC about shader cache.

Change-Id: I83f10b5a7231af9f6a68c33336c92bc1f7ea1a68
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Do not calculate transform if there is nothing changed as transform

Since we can know every cases when the local-world matrix changeness,
we can "skip" if there is nothing changed during current frame.

Transform calculation is kind of heavy operation.
So if there is any timing that we can skip it, it could be
increase performance for non-transform cases (a.k.a Lottie, AniamtedImage, Particle)

Change-Id: Ie7b6fdd8099810e599bf696fca267e48c7f83066
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge changes Ia910ced0,Ifa949d9b into devel/master

* changes:
  Print log if TypeInfo asserted + Do not assert if we call SetProperty for readonly + Add keyword LIKELY / UNLIKELY
  Remove useless macro

Merge "DALi Version 2.3.33" into devel/master

DALi Version 2.3.33

Change-Id: I42061d53504977f1cd2b39eb984ecc9b9625f166

Fix secondary command buffer Begin flag

Change-Id: I52db5f89da394444bcb248ff97ed0a2b0bf2b6da

Adding Begin/End commands to graphics API

Change-Id: Iff0f91133f4a8646f17ccaebf6d7d95e84c8fc17

Merge "Cleaned up render pass creation comments" into devel/master

(TestApplication) Fix SVACE issue

Change-Id: I654a48b18ef90fb7c0a4737ac9dc273f4c3e661a

Merge changes I03cb1f37,I66ce8530,Iad1ace55,Ib9eef3cf into devel/master

* changes:
  Making test-render-surface.h/cpp common across repos
  Updating framebuffer to use renderpass
  Minor test harness updates
  Updating graphics-api for Vulkan

Cleaned up render pass creation comments

Change-Id: I92a7a4c4beb236fd2caa9207c7726079374bc480

DALi Version 2.3.32

Change-Id: If9a370530f2668add53306b58726d6cfb7c5f60a

Calculate Has with std::string_view

Since convert from std::string_view to std::string
might requred memory copy.

To avoid this overhead, let we make a API that calculate
hash as string_view, instead of full string

Change-Id: Ie5455a5001e1c1f423023708a4988d53badff74c
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Print log if TypeInfo asserted + Do not assert if we call SetProperty for readonly + Add keyword LIKELY / UNLIKELY

Change-Id: Ia910ced0e022f95a262cd508b02b31917b1e3a83
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Making test-render-surface.h/cpp common across repos

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

Updating framebuffer to use renderpass

Vulkan framebuffer requires a renderpass to be defined before
the framebuffer can be created. Currently in gles backend, this
order is irrelevant, and is not defined in the graphics api.

Added Graphics::RenderPass vector to the
Graphics::FramebufferCreateInfo - this vector may have more than one
compatible render pass in it. The first render pass will be used to
create the framebuffer, but BeginRenderPass may use any of the defined
render passes with that framebuffer (as long as they are compatible in
vulkan, i.e. only differ by layout or load/store ops).

Change-Id: I66ce8530a34bcb1da57461db5644771efe0af48c

Minor test harness updates

Adding test-render-surface to dali-core

Change-Id: Iad1ace557fd4bab95a5e5cae5cf8281a9f4d2c64

Updating graphics-api for Vulkan

Change-Id: Ib9eef3cf2901b4fae094891d9ac82939c5be9d49

Remove useless macro

Since we use IndexedMap for a long time and it doesn't show any problem, let we remove this macro

Change-Id: Ifa949d9b4a2b40238770d63b170d9dd462ff0f95
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.31

Change-Id: Ifa20f9c8ef8faea482e55bcfd24cf71f1460af05

Merge "Print log if worker thread destruction occured for UI items." into devel/master

Print log if worker thread destruction occured for UI items.

Change-Id: I15db9dd5d6763b403b4b5015c95e3491c18443dc
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>