Remove Helgrind warning when Node flags are illegally accessed from event and update... 26/46726/1
authorKimmo Hoikka <kimmo.hoikka@samsung.com>
Tue, 25 Aug 2015 09:19:24 +0000 (10:19 +0100)
committerKimmo Hoikka <kimmo.hoikka@samsung.com>
Tue, 25 Aug 2015 09:19:24 +0000 (10:19 +0100)
Change-Id: I45c483fe5db2d3cad32144f120b7d7311867658c

dali/internal/event/actors/actor-impl.cpp

index ea6f1d0..f327252 100644 (file)
@@ -1631,12 +1631,11 @@ bool Actor::RayActorTest( const Vector4& rayOrigin, const Vector4& rayDir, Vecto
   if( OnStage() &&
   NULL != mNode )
   {
-    BufferIndex bufferIndex( GetEventThreadServices().GetEventBufferIndex() );
-
     // Transforms the ray to the local reference system.
-
     // Calculate the inverse of Model matrix
     Matrix invModelMatrix( false/*don't init*/);
+
+    BufferIndex bufferIndex( GetEventThreadServices().GetEventBufferIndex() );
     // need to use the components as world matrix is only updated for actors that need it
     invModelMatrix.SetInverseTransformComponents( mNode->GetWorldScale( bufferIndex ), mNode->GetWorldOrientation( bufferIndex ), mNode->GetWorldPosition( bufferIndex ) );
 
@@ -2112,10 +2111,9 @@ bool Actor::IsNodeConnected() const
 {
   bool connected( false );
 
-  if( OnStage() &&
-  NULL != mNode )
+  if( OnStage() && ( NULL != mNode ) )
   {
-    if( mNode->IsRoot() || mNode->GetParent() )
+    if( IsRoot() || mNode->GetParent() )
     {
       connected = true;
     }