cui/source/dialogs/whatsnewtabpage.cxx |   18 +++++++++---------
 cui/source/inc/whatsnewtabpage.hxx     |    6 +++---
 2 files changed, 12 insertions(+), 12 deletions(-)

New commits:
commit 4daf094be97a6e8df5dcb28089450684d0a35e33
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Fri Jun 13 10:44:35 2025 +0200
Commit:     Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>
CommitDate: Fri Aug 1 08:48:13 2025 +0200

    cui: Don't use "p" prefix for non-pointers
    
    Change-Id: I63be3cc6aa46b64f42c5e39a7cb7cebea2eba4a3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186451
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
    (cherry picked from commit 8e6c53f057e48c0f5bf9c2ea14dd24e6cbd80209)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188649
    Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>
    (cherry picked from commit d8745db2e830fc45288e412c0918b08dd1e22159)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188668
    Reviewed-by: ๆž—ๅšไป (Buo-ren Lin) <buo.ren....@gmail.com>
    Reviewed-by: Stephan Bergmann <stephan.bergm...@collabora.com>
    Tested-by: Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>

diff --git a/cui/source/dialogs/whatsnewtabpage.cxx 
b/cui/source/dialogs/whatsnewtabpage.cxx
index 01ed7d416635..6bfd152a5085 100644
--- a/cui/source/dialogs/whatsnewtabpage.cxx
+++ b/cui/source/dialogs/whatsnewtabpage.cxx
@@ -52,8 +52,8 @@ AnimatedBrand::AnimatedBrand()
 {
     OUString aURL(u"$BRAND_BASE_DIR/" LIBO_SHARE_SHELL_FOLDER 
"/animatedbrand.gif"_ustr);
     rtl::Bootstrap::expandMacros(aURL);
-    GraphicFilter::LoadGraphic(aURL, OUString(), m_pGraphic);
-    m_pGraphicSize = m_pGraphic.GetSizePixel();
+    GraphicFilter::LoadGraphic(aURL, OUString(), m_aGraphic);
+    m_aGraphicSize = m_aGraphic.GetSizePixel();
 }
 
 void AnimatedBrand::Paint(vcl::RenderContext& rRenderContext, const 
tools::Rectangle& rRect)
@@ -62,20 +62,20 @@ void AnimatedBrand::Paint(vcl::RenderContext& 
rRenderContext, const tools::Recta
     rRenderContext.SetBackground(COL_WHITE);
     rRenderContext.Erase();
 
-    const Point aGraphicPosition((rRect.GetWidth() - m_pGraphicSize.Width()),
-                                 (rRect.GetHeight() - m_pGraphicSize.Height()) 
>> 1);
+    const Point aGraphicPosition((rRect.GetWidth() - m_aGraphicSize.Width()),
+                                 (rRect.GetHeight() - m_aGraphicSize.Height()) 
>> 1);
 #ifndef MACOSX
-    if (m_pGraphic.IsAnimated() && MiscSettings::IsAnimatedOthersAllowed())
-        m_pGraphic.StartAnimation(rRenderContext, aGraphicPosition, 
m_pGraphicSize);
+    if (m_aGraphic.IsAnimated() && MiscSettings::IsAnimatedOthersAllowed())
+        m_aGraphic.StartAnimation(rRenderContext, aGraphicPosition, 
m_aGraphicSize);
     else
 #endif
-        m_pGraphic.Draw(rRenderContext, aGraphicPosition, m_pGraphicSize);
+        m_aGraphic.Draw(rRenderContext, aGraphicPosition, m_aGraphicSize);
 
     tools::Rectangle aTextRect;
-    if (m_pGraphic.isAvailable())
+    if (m_aGraphic.isAvailable())
     {
         aTextRect.SetPos(Point(8, 8));
-        aTextRect.SetSize(Size(aGraphicPosition.getX(), 
m_pGraphicSize.Height() - 20));
+        aTextRect.SetSize(Size(aGraphicPosition.getX(), 
m_aGraphicSize.Height() - 20));
     }
     else
         aTextRect = rRect;
diff --git a/cui/source/inc/whatsnewtabpage.hxx 
b/cui/source/inc/whatsnewtabpage.hxx
index 7eeeb0337182..5a10a57f2b3e 100644
--- a/cui/source/inc/whatsnewtabpage.hxx
+++ b/cui/source/inc/whatsnewtabpage.hxx
@@ -17,13 +17,13 @@ class AnimatedBrand : public weld::CustomWidgetController
 {
 private:
     bool m_bIsFirstStart;
-    Graphic m_pGraphic;
-    Size m_pGraphicSize;
+    Graphic m_aGraphic;
+    Size m_aGraphicSize;
     virtual void Paint(vcl::RenderContext& rRenderContext, const 
tools::Rectangle&) override;
 
 public:
     AnimatedBrand();
-    const Size& GetGraphicSize() const { return m_pGraphicSize; };
+    const Size& GetGraphicSize() const { return m_aGraphicSize; };
     void SetIsFirstStart(const bool bIsFirstStart) { m_bIsFirstStart = 
bIsFirstStart; };
 };
 

Reply via email to