Kaydet (Commit) 77ba9d7e authored tarafından Rohan Kumar's avatar Rohan Kumar Kaydeden (comit) Björn Michaelsen

tdf#91794 remove OSL_DEBUG_LEVEL > 1 conditionals

I replaced OSL_DEBUG_LEVEL > 1 to OSL_DEBUG_LEVEL > 0 conditionals
as suggested in easyhack. I didn't convert those fprintf statements
because they were required for debug output formatting according to me

Change-Id: I54d5b0873f527fd04d736eeb01d18f73eb9be74f
Reviewed-on: https://gerrit.libreoffice.org/23069Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarBjörn Michaelsen <bjoern.michaelsen@canonical.com>
üst aba0c20f
...@@ -1639,7 +1639,7 @@ throw (uno::RuntimeException, std::exception) ...@@ -1639,7 +1639,7 @@ throw (uno::RuntimeException, std::exception)
} // namespace sfx2 } // namespace sfx2
#if OSL_DEBUG_LEVEL > 1 #if OSL_DEBUG_LEVEL > 0
#include <stdio.h> #include <stdio.h>
......
...@@ -725,17 +725,6 @@ void SfxObjectShell::SetTitle ...@@ -725,17 +725,6 @@ void SfxObjectShell::SetTitle
} }
#if OSL_DEBUG_LEVEL > 1
OUString X(const OUString &rRet)
{
if ( rRet.isEmpty() )
return OUString( "-empty-" );
return rRet;
}
#else
#define X(ret) ret
#endif
OUString SfxObjectShell::GetTitle OUString SfxObjectShell::GetTitle
( (
...@@ -818,15 +807,15 @@ OUString SfxObjectShell::GetTitle ...@@ -818,15 +807,15 @@ OUString SfxObjectShell::GetTitle
if ( IsTemplate() ) if ( IsTemplate() )
pThis->SetTitle( aTitle ); pThis->SetTitle( aTitle );
bRecur = false; bRecur = false;
return X(aTitle); return aTitle;
} }
else if (SFX_TITLE_APINAME == nMaxLength ) else if (SFX_TITLE_APINAME == nMaxLength )
return X(GetAPIName()); return GetAPIName();
// Special case templates: // Special case templates:
if( IsTemplate() && !pImp->aTitle.isEmpty() && if( IsTemplate() && !pImp->aTitle.isEmpty() &&
( nMaxLength == SFX_TITLE_CAPTION || nMaxLength == SFX_TITLE_PICKLIST ) ) ( nMaxLength == SFX_TITLE_CAPTION || nMaxLength == SFX_TITLE_PICKLIST ) )
return X(pImp->aTitle); return pImp->aTitle;
// Picklist/Caption is mapped // Picklist/Caption is mapped
if ( pMed && ( nMaxLength == SFX_TITLE_CAPTION || nMaxLength == SFX_TITLE_PICKLIST ) ) if ( pMed && ( nMaxLength == SFX_TITLE_CAPTION || nMaxLength == SFX_TITLE_PICKLIST ) )
...@@ -836,7 +825,7 @@ OUString SfxObjectShell::GetTitle ...@@ -836,7 +825,7 @@ OUString SfxObjectShell::GetTitle
// considered. (See below, analysis of aTitleMap_Impl) // considered. (See below, analysis of aTitleMap_Impl)
const SfxStringItem* pNameItem = SfxItemSet::GetItem<SfxStringItem>(pMed->GetItemSet(), SID_DOCINFO_TITLE, false); const SfxStringItem* pNameItem = SfxItemSet::GetItem<SfxStringItem>(pMed->GetItemSet(), SID_DOCINFO_TITLE, false);
if ( pNameItem ) if ( pNameItem )
return X( pNameItem->GetValue() ); return pNameItem->GetValue();
} }
// Still unnamed? // Still unnamed?
...@@ -845,7 +834,7 @@ OUString SfxObjectShell::GetTitle ...@@ -845,7 +834,7 @@ OUString SfxObjectShell::GetTitle
{ {
// Title already set? // Title already set?
if ( !pImp->aTitle.isEmpty() ) if ( !pImp->aTitle.isEmpty() )
return X(pImp->aTitle); return pImp->aTitle;
// must it be numbered? // must it be numbered?
OUString aNoName(SFX2_RESSTR(STR_NONAME)); OUString aNoName(SFX2_RESSTR(STR_NONAME));
...@@ -856,7 +845,7 @@ OUString SfxObjectShell::GetTitle ...@@ -856,7 +845,7 @@ OUString SfxObjectShell::GetTitle
} }
// Document called "noname" for the time being // Document called "noname" for the time being
return X(aNoName); return aNoName;
} }
const INetURLObject aURL( IsDocShared() ? GetSharedFileURL() : OUString( GetMedium()->GetName() ) ); const INetURLObject aURL( IsDocShared() ? GetSharedFileURL() : OUString( GetMedium()->GetName() ) );
...@@ -875,10 +864,9 @@ OUString SfxObjectShell::GetTitle ...@@ -875,10 +864,9 @@ OUString SfxObjectShell::GetTitle
{ {
OUString aName( aURL.HasMark() ? INetURLObject( aURL.GetURLNoMark() ).PathToFileName() : aURL.PathToFileName() ); OUString aName( aURL.HasMark() ? INetURLObject( aURL.GetURLNoMark() ).PathToFileName() : aURL.PathToFileName() );
if ( nMaxLength == SFX_TITLE_FULLNAME ) if ( nMaxLength == SFX_TITLE_FULLNAME )
return X( aName ); return aName;
else if ( nMaxLength == SFX_TITLE_FILENAME ) else if ( nMaxLength == SFX_TITLE_FILENAME )
return X( aURL.getName( INetURLObject::LAST_SEGMENT, return aURL.getName(INetURLObject::LAST_SEGMENT, true, INetURLObject::DECODE_WITH_CHARSET);
true, INetURLObject::DECODE_WITH_CHARSET ) );
else if ( pImp->aTitle.isEmpty() ) else if ( pImp->aTitle.isEmpty() )
pImp->aTitle = aURL.getBase( INetURLObject::LAST_SEGMENT, pImp->aTitle = aURL.getBase( INetURLObject::LAST_SEGMENT,
true, INetURLObject::DECODE_WITH_CHARSET ); true, INetURLObject::DECODE_WITH_CHARSET );
...@@ -892,10 +880,10 @@ OUString SfxObjectShell::GetTitle ...@@ -892,10 +880,10 @@ OUString SfxObjectShell::GetTitle
{ {
OUString aRet( "..." ); OUString aRet( "..." );
aRet += aComplete.copy( aComplete.getLength() - nMaxLength + 3, nMaxLength - 3 ); aRet += aComplete.copy( aComplete.getLength() - nMaxLength + 3, nMaxLength - 3 );
return X( aRet ); return aRet;
} }
else else
return X( aComplete ); return aComplete;
} }
else if ( nMaxLength == SFX_TITLE_FILENAME ) else if ( nMaxLength == SFX_TITLE_FILENAME )
{ {
...@@ -903,10 +891,10 @@ OUString SfxObjectShell::GetTitle ...@@ -903,10 +891,10 @@ OUString SfxObjectShell::GetTitle
aName = INetURLObject::decode( aName, INetURLObject::DECODE_WITH_CHARSET ); aName = INetURLObject::decode( aName, INetURLObject::DECODE_WITH_CHARSET );
if( aName.isEmpty() ) if( aName.isEmpty() )
aName = aURL.GetURLNoPass(); aName = aURL.GetURLNoPass();
return X(aName); return aName;
} }
else if ( nMaxLength == SFX_TITLE_FULLNAME ) else if ( nMaxLength == SFX_TITLE_FULLNAME )
return X(aURL.GetMainURL( INetURLObject::DECODE_TO_IURI )); return aURL.GetMainURL( INetURLObject::DECODE_TO_IURI );
// Generate Title from file name if possible // Generate Title from file name if possible
if ( pImp->aTitle.isEmpty() ) if ( pImp->aTitle.isEmpty() )
...@@ -918,7 +906,7 @@ OUString SfxObjectShell::GetTitle ...@@ -918,7 +906,7 @@ OUString SfxObjectShell::GetTitle
} }
// Complete Title // Complete Title
return X(pImp->aTitle); return pImp->aTitle;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment