vcl/inc/animate/AnimationRenderer.hxx    |   10 ++---
 vcl/source/animate/Animation.cxx         |    2 -
 vcl/source/animate/AnimationRenderer.cxx |   62 +++++++++++++++----------------
 3 files changed, 37 insertions(+), 37 deletions(-)

New commits:
commit 91d77d65190e7cf11dd7cd0b29d5de6b66061faf
Author:     Chris Sherlock <chris.sherloc...@gmail.com>
AuthorDate: Fri Jul 1 17:16:21 2022 +1000
Commit:     Tomaž Vajngerl <qui...@gmail.com>
CommitDate: Wed Aug 10 07:53:08 2022 +0200

    vcl: AnimationRenderer::getOutPos() -> GetOriginPosition()
    
    Change-Id: I23275aa9915ea47cb91a8511b219869a4d62b96a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137352
    Tested-by: Jenkins
    Reviewed-by: Tomaž Vajngerl <qui...@gmail.com>

diff --git a/vcl/inc/animate/AnimationRenderer.hxx 
b/vcl/inc/animate/AnimationRenderer.hxx
index bc98f9ac3093..95de08014a21 100644
--- a/vcl/inc/animate/AnimationRenderer.hxx
+++ b/vcl/inc/animate/AnimationRenderer.hxx
@@ -50,11 +50,11 @@ private:
     Animation*      mpParent;
     VclPtr<OutputDevice>  mpRenderContext;
     tools::Long     mnRendererId;
-    Point           maPt;
+    Point           maOriginPt;
     Point           maDispPt;
     Point           maRestPt;
-    Size            maSz;
-    Size            maSzPix;
+    Size            maLogicalSize;
+    Size            maSizePx;
     Size            maDispSz;
     Size            maRestSz;
     vcl::Region     maClip;
@@ -82,9 +82,9 @@ public:
 
     void            getPosSize( const AnimationBitmap& rAnm, Point& rPosPix, 
Size& rSizePix );
 
-    const Point&    getOutPos() const { return maPt; }
+    const Point&    getOriginPosition() const { return maOriginPt; }
 
-    const Size&     getOutSizePix() const { return maSzPix; }
+    const Size&     getOutSizePix() const { return maSizePx; }
 
     void            pause( bool bIsPaused ) { mbIsPaused = bIsPaused; }
     bool            isPaused() const { return mbIsPaused; }
diff --git a/vcl/source/animate/Animation.cxx b/vcl/source/animate/Animation.cxx
index 0922dec2bf37..831532d5977a 100644
--- a/vcl/source/animate/Animation.cxx
+++ b/vcl/source/animate/Animation.cxx
@@ -181,7 +181,7 @@ bool Animation::Start(OutputDevice& rOut, const Point& 
rDestPt, const Size& rDes
 
             if (itAnimView != maRenderers.end())
             {
-                if ((*itAnimView)->getOutPos() == rDestPt
+                if ((*itAnimView)->getOriginPosition() == rDestPt
                     && (*itAnimView)->getOutSizePix() == 
rOut.LogicToPixel(rDestSz))
                 {
                     (*itAnimView)->repaint();
diff --git a/vcl/source/animate/AnimationRenderer.cxx 
b/vcl/source/animate/AnimationRenderer.cxx
index e4ce509a694d..9bb5efc38ae9 100644
--- a/vcl/source/animate/AnimationRenderer.cxx
+++ b/vcl/source/animate/AnimationRenderer.cxx
@@ -32,10 +32,10 @@ AnimationRenderer::AnimationRenderer( Animation* pParent, 
OutputDevice* pOut,
                             OutputDevice* pFirstFrameOutDev ) :
         mpParent        ( pParent ),
         mpRenderContext ( pFirstFrameOutDev ? pFirstFrameOutDev : pOut ),
-        mnRendererId     ( nRendererId ),
-        maPt            ( rPt ),
-        maSz            ( rSz ),
-        maSzPix         ( mpRenderContext->LogicToPixel( maSz ) ),
+        mnRendererId    ( nRendererId ),
+        maOriginPt      ( rPt ),
+        maLogicalSize   ( rSz ),
+        maSizePx        ( mpRenderContext->LogicToPixel( maLogicalSize ) ),
         maClip          ( mpRenderContext->GetClipRegion() ),
         mpBackground    ( VclPtr<VirtualDevice>::Create() ),
         mpRestore       ( VclPtr<VirtualDevice>::Create() ),
@@ -43,40 +43,40 @@ AnimationRenderer::AnimationRenderer( Animation* pParent, 
OutputDevice* pOut,
         meLastDisposal  ( Disposal::Back ),
         mbIsPaused      ( false ),
         mbIsMarked      ( false ),
-        mbIsMirroredHorizontally         ( maSz.Width() < 0 ),
-        mbIsMirroredVertically         ( maSz.Height() < 0 )
+        mbIsMirroredHorizontally( maLogicalSize.Width() < 0 ),
+        mbIsMirroredVertically( maLogicalSize.Height() < 0 )
 {
     Animation::ImplIncAnimCount();
 
     // Mirrored horizontally?
     if( mbIsMirroredHorizontally )
     {
-        maDispPt.setX( maPt.X() + maSz.Width() + 1 );
-        maDispSz.setWidth( -maSz.Width() );
-        maSzPix.setWidth( -maSzPix.Width() );
+        maDispPt.setX( maOriginPt.X() + maLogicalSize.Width() + 1 );
+        maDispSz.setWidth( -maLogicalSize.Width() );
+        maSizePx.setWidth( -maSizePx.Width() );
     }
     else
     {
-        maDispPt.setX( maPt.X() );
-        maDispSz.setWidth( maSz.Width() );
+        maDispPt.setX( maOriginPt.X() );
+        maDispSz.setWidth( maLogicalSize.Width() );
     }
 
     // Mirrored vertically?
     if( mbIsMirroredVertically )
     {
-        maDispPt.setY( maPt.Y() + maSz.Height() + 1 );
-        maDispSz.setHeight( -maSz.Height() );
-        maSzPix.setHeight( -maSzPix.Height() );
+        maDispPt.setY( maOriginPt.Y() + maLogicalSize.Height() + 1 );
+        maDispSz.setHeight( -maLogicalSize.Height() );
+        maSizePx.setHeight( -maSizePx.Height() );
     }
     else
     {
-        maDispPt.setY( maPt.Y() );
-        maDispSz.setHeight( maSz.Height() );
+        maDispPt.setY( maOriginPt.Y() );
+        maDispSz.setHeight( maLogicalSize.Height() );
     }
 
     // save background
-    mpBackground->SetOutputSizePixel( maSzPix );
-    mpRenderContext->SaveBackground(*mpBackground, maDispPt, maDispSz, 
maSzPix);
+    mpBackground->SetOutputSizePixel( maSizePx );
+    mpRenderContext->SaveBackground(*mpBackground, maDispPt, maDispSz, 
maSizePx);
 
     // Initialize drawing to actual position
     drawToIndex( mpParent->ImplGetCurPos() );
@@ -111,13 +111,13 @@ void AnimationRenderer::getPosSize( const 
AnimationBitmap& rAnimationBitmap, Poi
 
     // calculate x scaling
     if( rAnmSize.Width() > 1 )
-        fFactX = static_cast<double>( maSzPix.Width() - 1 ) / ( 
rAnmSize.Width() - 1 );
+        fFactX = static_cast<double>( maSizePx.Width() - 1 ) / ( 
rAnmSize.Width() - 1 );
     else
         fFactX = 1.0;
 
     // calculate y scaling
     if( rAnmSize.Height() > 1 )
-        fFactY = static_cast<double>( maSzPix.Height() - 1 ) / ( 
rAnmSize.Height() - 1 );
+        fFactY = static_cast<double>( maSizePx.Height() - 1 ) / ( 
rAnmSize.Height() - 1 );
     else
         fFactY = 1.0;
 
@@ -132,11 +132,11 @@ void AnimationRenderer::getPosSize( const 
AnimationBitmap& rAnimationBitmap, Poi
 
     // Mirrored horizontally?
     if( mbIsMirroredHorizontally )
-        rPosPix.setX( maSzPix.Width() - 1 - aPt2.X() );
+        rPosPix.setX( maSizePx.Width() - 1 - aPt2.X() );
 
     // Mirrored vertically?
     if( mbIsMirroredVertically )
-        rPosPix.setY( maSzPix.Height() - 1 - aPt2.Y() );
+        rPosPix.setY( maSizePx.Height() - 1 - aPt2.Y() );
 }
 
 void AnimationRenderer::drawToIndex( sal_uLong nIndex )
@@ -156,7 +156,7 @@ void AnimationRenderer::drawToIndex( sal_uLong nIndex )
     if (!maClip.IsNull())
         xOldClip = pRenderContext->GetClipRegion();
 
-    aVDev->SetOutputSizePixel( maSzPix, false );
+    aVDev->SetOutputSizePixel( maSizePx, false );
     nIndex = std::min( nIndex, static_cast<sal_uLong>(mpParent->Count()) - 1 );
 
     for( sal_uLong i = 0; i <= nIndex; i++ )
@@ -165,7 +165,7 @@ void AnimationRenderer::drawToIndex( sal_uLong nIndex )
     if (xOldClip)
         pRenderContext->SetClipRegion( maClip );
 
-    pRenderContext->DrawOutDev( maDispPt, maDispSz, Point(), maSzPix, *aVDev );
+    pRenderContext->DrawOutDev( maDispPt, maDispSz, Point(), maSizePx, *aVDev 
);
     if (pGuard)
         pGuard->SetPaintRect(tools::Rectangle(maDispPt, maDispSz));
 
@@ -231,8 +231,8 @@ void AnimationRenderer::draw( sal_uLong nIndex, 
VirtualDevice* pVDev )
         if( !pVDev )
         {
             pDev = VclPtr<VirtualDevice>::Create();
-            pDev->SetOutputSizePixel( maSzPix, false );
-            pDev->DrawOutDev( Point(), maSzPix, maDispPt, maDispSz, 
*pRenderContext );
+            pDev->SetOutputSizePixel( maSizePx, false );
+            pDev->DrawOutDev( Point(), maSizePx, maDispPt, maDispSz, 
*pRenderContext );
         }
         else
             pDev = pVDev;
@@ -242,7 +242,7 @@ void AnimationRenderer::draw( sal_uLong nIndex, 
VirtualDevice* pVDev )
         {
             meLastDisposal = Disposal::Back;
             maRestPt = Point();
-            maRestSz = maSzPix;
+            maRestSz = maSizePx;
         }
 
         // restore
@@ -280,7 +280,7 @@ void AnimationRenderer::draw( sal_uLong nIndex, 
VirtualDevice* pVDev )
             if (xOldClip)
                 pRenderContext->SetClipRegion( maClip );
 
-            pRenderContext->DrawOutDev( maDispPt, maDispSz, Point(), maSzPix, 
*pDev );
+            pRenderContext->DrawOutDev( maDispPt, maDispSz, Point(), maSizePx, 
*pDev );
             if (pGuard)
                 pGuard->SetPaintRect(tools::Rectangle(maDispPt, maDispSz));
 
@@ -300,7 +300,7 @@ void AnimationRenderer::repaint()
 {
     const bool bOldPause = mbIsPaused;
 
-    mpRenderContext->SaveBackground(*mpBackground, maDispPt, maDispSz, 
maSzPix);
+    mpRenderContext->SaveBackground(*mpBackground, maDispPt, maDispSz, 
maSizePx);
 
     mbIsPaused = false;
     drawToIndex( mnActIndex );
@@ -311,8 +311,8 @@ AnimationData* AnimationRenderer::createAnimationData() 
const
 {
     AnimationData* pDataItem = new AnimationData;
 
-    pDataItem->aStartOrg = maPt;
-    pDataItem->aStartSize = maSz;
+    pDataItem->aStartOrg = maOriginPt;
+    pDataItem->aStartSize = maLogicalSize;
     pDataItem->pOutDev = mpRenderContext;
     pDataItem->pRendererData = const_cast<AnimationRenderer *>(this);
     pDataItem->nRendererId = mnRendererId;

Reply via email to