[Libreoffice-commits] core.git: sw/qa vcl/headless vcl/osx xmloff/source

[Jan-Marek Glogowski (via logerrit)]

 sw/qa/core/uwriter.cxx           |    7 -------
 vcl/headless/svpframe.cxx        |    6 ------
 vcl/osx/a11yfactory.mm           |    6 ------
 xmloff/source/style/impastpl.cxx |    2 --
 4 files changed, 21 deletions(-)

New commits:
commit 55dcb060ea5feef0478f7abcaa4da9cbe69bd440
Author:     Jan-Marek Glogowski <glo...@fbihome.de>
AuthorDate: Tue Jan 4 09:26:42 2022 +0100
Commit:     Jan-Marek Glogowski <glo...@fbihome.de>
CommitDate: Tue Jan 4 11:26:47 2022 +0100

    Drop all commented SAL_DEBUG calls
    
    Change-Id: I3be7153d27b83766303519e73d1a355ba327871a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127929
    Tested-by: Jenkins
    Reviewed-by: Jan-Marek Glogowski <glo...@fbihome.de>

diff --git a/sw/qa/core/uwriter.cxx b/sw/qa/core/uwriter.cxx
index c43d068152b6..c2175aea2f44 100644
--- a/sw/qa/core/uwriter.cxx
+++ b/sw/qa/core/uwriter.cxx
@@ -1552,12 +1552,6 @@ namespace
             { return GetPrevInRing(); }
         bool lonely() const
             { return unique(); }
-#if 0
-        void debug()
-        {
-            SAL_DEBUG("TestRing at: " << this << " prev: " << GetPrev() << " 
next: " << GetNext());
-        }
-#endif
     };
 }
 
@@ -1603,7 +1597,6 @@ void SwDocTest::testIntrusiveRing()
     {
         TestRing* pRing = &r;
         CPPUNIT_ASSERT(pRing);
-        //pRing->debug();
     }
     const TestRing* pConstRing = &aRing1;
     for(const TestRing& r: pConstRing->GetRingContainer()) // this should fail 
without r being const
diff --git a/vcl/headless/svpframe.cxx b/vcl/headless/svpframe.cxx
index 5f40377f5b86..c6c8a08e29c0 100644
--- a/vcl/headless/svpframe.cxx
+++ b/vcl/headless/svpframe.cxx
@@ -55,7 +55,6 @@ SvpSalFrame::SvpSalFrame( SvpSalInstance* pInstance,
     m_nMaxWidth( 0 ),
     m_nMaxHeight( 0 )
 {
-    // SAL_DEBUG("SvpSalFrame::SvpSalFrame: " << this);
 #ifdef IOS
     // Nothing
 #elif defined ANDROID
@@ -86,7 +85,6 @@ SvpSalFrame::~SvpSalFrame()
 
     if( s_pFocusFrame == this )
     {
-        // SAL_DEBUG("SvpSalFrame::~SvpSalFrame: losing focus: " << this);
         s_pFocusFrame = nullptr;
         // call directly here, else an event for a destroyed frame would be 
dispatched
         CallCallback( SalEvent::LoseFocus, nullptr );
@@ -129,7 +127,6 @@ void SvpSalFrame::GetFocus()
     {
         if( s_pFocusFrame )
             s_pFocusFrame->LoseFocus();
-        // SAL_DEBUG("SvpSalFrame::GetFocus(): " << this);
         s_pFocusFrame = this;
         m_pInstance->PostEvent( this, nullptr, SalEvent::GetFocus );
     }
@@ -139,7 +136,6 @@ void SvpSalFrame::LoseFocus()
 {
     if( s_pFocusFrame == this )
     {
-        // SAL_DEBUG("SvpSalFrame::LoseFocus: " << this);
         m_pInstance->PostEvent( this, nullptr, SalEvent::LoseFocus );
         s_pFocusFrame = nullptr;
     }
@@ -225,7 +221,6 @@ void SvpSalFrame::Show( bool bVisible, bool bNoActivate )
 
     if (bVisible)
     {
-        // SAL_DEBUG("SvpSalFrame::Show: showing: " << this);
         m_bVisible = true;
         m_pInstance->PostEvent( this, nullptr, SalEvent::Resize );
         if( ! bNoActivate )
@@ -233,7 +228,6 @@ void SvpSalFrame::Show( bool bVisible, bool bNoActivate )
     }
     else
     {
-        // SAL_DEBUG("SvpSalFrame::Show: hiding: " << this);
         m_bVisible = false;
         LoseFocus();
     }
diff --git a/vcl/osx/a11yfactory.mm b/vcl/osx/a11yfactory.mm
index 560d3f97357a..e9a5da854607 100644
--- a/vcl/osx/a11yfactory.mm
+++ b/vcl/osx/a11yfactory.mm
@@ -165,19 +165,13 @@ static bool enabled = false;
             id parent = [aWrapper 
accessibilityAttributeValue:NSAccessibilityParentAttribute];
             if (parent) {
                 if ([parent isKindOfClass:[NSView class]]) {
-                    // SAL_DEBUG("Wrapper INIT: " << [[aWrapper description] 
UTF8String] << " ==> " << [[parent description] UTF8String]);
                     NSView *parentView = static_cast<NSView *>(parent);
                     [parentView addSubview:aWrapper positioned:NSWindowBelow 
relativeTo:nil];
                 } else if ([parent 
isKindOfClass:NSClassFromString(@"SalFrameWindow")]) {
                     NSWindow *window = static_cast<NSWindow *>(parent);
                     NSView *salView = [window contentView];
-                    // SAL_DEBUG("Wrapper INIT SAL: " << [[aWrapper 
description] UTF8String] << " ==> " << [[salView description] UTF8String]);
                     [salView addSubview:aWrapper positioned:NSWindowBelow 
relativeTo:nil];
-                } else {
-                    // SAL_DEBUG("Wrapper INIT: !! " << [[aWrapper 
description] UTF8String] << " !==>! " << [[parent description] UTF8String] << 
"!!");
                 }
-            } else {
-                // SAL_DEBUG("Wrapper INIT: " << [[aWrapper description] 
UTF8String] << " ==> NO PARENT");
             }
         }
     }
diff --git a/xmloff/source/style/impastpl.cxx b/xmloff/source/style/impastpl.cxx
index fd7c84dc1642..42565e08cf85 100644
--- a/xmloff/source/style/impastpl.cxx
+++ b/xmloff/source/style/impastpl.cxx
@@ -400,7 +400,6 @@ void SvXMLAutoStylePoolP_Impl::RegisterName( XmlStyleFamily 
nFamily, const OUStr
     XMLAutoStyleFamily aTemp(nFamily);
     auto const iter = m_FamilySet.find(aTemp);
     assert(iter != m_FamilySet.end()); // family must be known
-    // SAL_DEBUG("SvXMLAutoStylePoolP_Impl::RegisterName: " << nFamily << ", 
'" << rName << "'");
     (*iter)->maNameSet.insert(rName);
 }
 
@@ -621,7 +620,6 @@ void SvXMLAutoStylePoolP_Impl::exportXML(
             OUString oldName = aExpStyles[i].mpProperties->GetName();
             sal_Int32 dashIx = oldName.indexOf('-');
             OUString newName = (dashIx > 0 ? oldName.copy(0, dashIx) : 
oldName) + OUString::number(i);
-            // SAL_DEBUG("renaming '" << oldName << "' -> '" << newName << 
"'");
             aExpStyles[i].mpProperties->SetName(newName);
         }
     }