[Libreoffice-commits] .: framework/source sw/source tools/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Mon Oct 22 12:21:21 PDT 2012
framework/source/dispatch/dispatchprovider.cxx | 35 ++++++++++---------------
sw/source/core/bastyp/swcache.cxx | 16 +++++------
sw/source/core/graphic/ndgrf.cxx | 2 -
tools/source/inet/inetstrm.cxx | 2 -
4 files changed, 24 insertions(+), 31 deletions(-)
New commits:
commit c348df5635c45fd446fa9a7e831581164e55be71
Author: Caolán McNamara <caolanm at redhat.com>
Date: Mon Oct 22 20:18:50 2012 +0100
loplugin: various warnings
Change-Id: I605bffb518d804ee69edda5e7641e13e4c14ef01
diff --git a/framework/source/dispatch/dispatchprovider.cxx b/framework/source/dispatch/dispatchprovider.cxx
index f570dd2..1fb6019 100644
--- a/framework/source/dispatch/dispatchprovider.cxx
+++ b/framework/source/dispatch/dispatchprovider.cxx
@@ -246,12 +246,11 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt
// Why is "top" here handled too? Because the desktop is the topest frame. Normaly it's superflous
// to use this target - but we can handle it in the same manner then "_self".
//-----------------------------------------------------------------------------------------------------
- else
- if (
- (sTargetFrameName==SPECIALTARGET_SELF) ||
- (sTargetFrameName==SPECIALTARGET_TOP ) ||
- (sTargetFrameName.isEmpty())
- )
+ else if (
+ (sTargetFrameName==SPECIALTARGET_SELF) ||
+ (sTargetFrameName==SPECIALTARGET_TOP ) ||
+ (sTargetFrameName.isEmpty())
+ )
{
xDispatcher = implts_searchProtocolHandler(aURL);
}
@@ -274,10 +273,9 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt
css::uno::Reference< css::frame::XDispatchProvider > xProvider( xFoundFrame, css::uno::UNO_QUERY );
xDispatcher = xProvider->queryDispatch(aURL,SPECIALTARGET_SELF,0);
}
- else
// if it couldn't be found - but creation was allowed
// use special dispatcher for creatio or froward it to the browser
- if (nSearchFlags & css::frame::FrameSearchFlag::CREATE)
+ else if (nSearchFlags & css::frame::FrameSearchFlag::CREATE)
xDispatcher = implts_getOrCreateDispatchHelper( E_CREATEDISPATCHER, xDesktop, sTargetFrameName, nSearchFlags );
}
@@ -324,8 +322,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryFrame
// I.II) "_menubar"
// Special mode on frame or task to receive the local menu. Not supported by findFrame()
//-----------------------------------------------------------------------------------------------------
- else
- if (sTargetName==SPECIALTARGET_MENUBAR)
+ else if (sTargetName==SPECIALTARGET_MENUBAR)
{
xDispatcher = implts_getOrCreateDispatchHelper( E_MENUDISPATCHER, xFrame );
}
@@ -335,8 +332,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryFrame
// Special sub frame of a top frame only. Search or create it. ... OK it's currently a little bit HACKI.
// Only the sfx (means the controller) can create it it.
//-----------------------------------------------------------------------------------------------------
- else
- if (sTargetName==SPECIALTARGET_BEAMER)
+ else if (sTargetName==SPECIALTARGET_BEAMER)
{
css::uno::Reference< css::frame::XDispatchProvider > xBeamer( xFrame->findFrame( SPECIALTARGET_BEAMER, css::frame::FrameSearchFlag::CHILDREN | css::frame::FrameSearchFlag::SELF ), css::uno::UNO_QUERY );
if (xBeamer.is())
@@ -357,8 +353,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryFrame
// I.V) "_parent"
// Our parent frame (if it exist) should handle this URL.
//-----------------------------------------------------------------------------------------------------
- else
- if (sTargetName==SPECIALTARGET_PARENT)
+ else if (sTargetName==SPECIALTARGET_PARENT)
{
css::uno::Reference< css::frame::XDispatchProvider > xParent( xFrame->getCreator(), css::uno::UNO_QUERY );
if (xParent.is())
@@ -371,8 +366,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryFrame
// This request must be forwarded to any parent frame, till we reach a top frame.
// If no parent exist, we can handle itself.
//-----------------------------------------------------------------------------------------------------
- else
- if (sTargetName==SPECIALTARGET_TOP)
+ else if (sTargetName==SPECIALTARGET_TOP)
{
if (xFrame->isTop())
{
@@ -398,11 +392,10 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryFrame
// protocol handler. If this failed too - we check for a loadable content and in case of true
// we load it into the frame by returning specilized dispatch object.
//-----------------------------------------------------------------------------------------------------
- else
- if (
- (sTargetName==SPECIALTARGET_SELF) ||
- (sTargetName.isEmpty())
- )
+ else if (
+ (sTargetName==SPECIALTARGET_SELF) ||
+ (sTargetName.isEmpty())
+ )
{
// There exist a hard coded interception for special URLs.
if ( aURL.Complete == ".uno:CloseDoc" || aURL.Complete == ".uno:CloseWin" )
diff --git a/sw/source/core/bastyp/swcache.cxx b/sw/source/core/bastyp/swcache.cxx
index d900998..4c77853 100644
--- a/sw/source/core/bastyp/swcache.cxx
+++ b/sw/source/core/bastyp/swcache.cxx
@@ -37,8 +37,8 @@ void SwCache::Check()
return;
// consistency check
- SAL_WARN_IF( pLast->GetNext(), "sw", "Last but not last." );
- SAL_WARN_IF( pRealFirst->GetPrev(), "sw", "First but not first." );
+ SAL_WARN_IF( pLast->GetNext(), "sw.core", "Last but not last." );
+ SAL_WARN_IF( pRealFirst->GetPrev(), "sw.core", "First but not first." );
sal_uInt16 nCnt = 0;
bool bFirstFound = false;
SwCacheObj *pObj = pRealFirst;
@@ -49,20 +49,20 @@ void SwCache::Check()
SwCacheObj *pTmp = pLast;
while ( pTmp && pTmp != pObj )
pTmp = pTmp->GetPrev();
- SAL_WARN_IF( !pTmp, "sw", "Objekt not found." );
+ SAL_WARN_IF( !pTmp, "sw.core", "Objekt not found." );
++nCnt;
if ( pObj == pFirst )
bFirstFound = true;
if ( !pObj->GetNext() )
- SAL_WARN_IF( pObj != pLast, "sw", "Last not Found." );
+ SAL_WARN_IF( pObj != pLast, "sw.core", "Last not Found." );
pObj = pObj->GetNext();
- SAL_WARN_IF( pObj == pRekursive, "sw", "Recursion in SwCache." );
+ SAL_WARN_IF( pObj == pRekursive, "sw.core", "Recursion in SwCache." );
}
- SAL_WARN_IF( !bFirstFound, "sw", "First not Found." );
- SAL_WARN_IF( nCnt + aFreePositions.size() != size(), "sw", "Lost Chain." );
+ SAL_WARN_IF( !bFirstFound, "sw.core", "First not Found." );
+ SAL_WARN_IF( nCnt + aFreePositions.size() != size(), "sw.core", "Lost Chain." );
SAL_WARN_IF(
- size() == nCurMax && nCurMax != aFreePositions.size() + nCnt, "sw",
+ size() == nCurMax && nCurMax != aFreePositions.size() + nCnt, "sw.core",
"Lost FreePositions." );
}
diff --git a/sw/source/core/graphic/ndgrf.cxx b/sw/source/core/graphic/ndgrf.cxx
index 20bbfe2..e5b4d6e 100644
--- a/sw/source/core/graphic/ndgrf.cxx
+++ b/sw/source/core/graphic/ndgrf.cxx
@@ -884,7 +884,7 @@ SwCntntNode* SwGrfNode::MakeCopy( SwDoc* pDoc, const SwNodeIndex& rIdx ) const
catch (const uno::Exception& e)
{
// #i48434#
- SAL_WARN("sw", "<SwGrfNode::MakeCopy(..)> - unhandled exception!" << e.Message);
+ SAL_WARN("sw.core", "<SwGrfNode::MakeCopy(..)> - unhandled exception!" << e.Message);
}
}
else
diff --git a/tools/source/inet/inetstrm.cxx b/tools/source/inet/inetstrm.cxx
index 7392f50..1812a99 100644
--- a/tools/source/inet/inetstrm.cxx
+++ b/tools/source/inet/inetstrm.cxx
@@ -1506,7 +1506,7 @@ int INetMIMEMessageStream::PutMsgLine (const sal_Char *pData, sal_uIntPtr nSize)
return status;
}
else {
- SAL_WARN( "tools", "Boundary not found." );
+ SAL_WARN( "tools.stream", "Boundary not found." );
}
status = INetMessageIOStream::PutMsgLine(
pOldPos, pChar - pOldPos + 1 );
More information about the Libreoffice-commits
mailing list