[Libreoffice-commits] .: 5 commits - sdext/source vcl/generic vcl/unx

[Stephan Bergmann]

 sdext/source/minimizer/graphiccollector.cxx   |    2 --
 sdext/source/pdfimport/tree/pdfiprocessor.cxx |    1 -
 sdext/source/presenter/PresenterHelpView.cxx  |    2 --
 sdext/source/presenter/PresenterToolBar.cxx   |    2 --
 vcl/generic/fontmanager/fontmanager.cxx       |   24 ------------------------
 vcl/unx/generic/window/salframe.cxx           |    2 +-
 6 files changed, 1 insertion(+), 32 deletions(-)

New commits:
commit 5ebd4bd13c9beb6776407abdcf7a6ff56a76b575
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Mon Jan 16 13:55:28 2012 +0100

    Clang -Wunused-variable.

diff --git a/sdext/source/presenter/PresenterHelpView.cxx b/sdext/source/presenter/PresenterHelpView.cxx
index c7ca5c4..b314dcc 100644
--- a/sdext/source/presenter/PresenterHelpView.cxx
+++ b/sdext/source/presenter/PresenterHelpView.cxx
@@ -375,8 +375,6 @@ void PresenterHelpView::ProcessString (
     PresenterConfigurationAccess::GetProperty(rsProperties, A2S("Left")) >>= sLeftText;
     OUString sRightText;
     PresenterConfigurationAccess::GetProperty(rsProperties, A2S("Right")) >>= sRightText;
-
-    const awt::Rectangle aWindowBox (mxWindow->getPosSize());
     mpTextContainer->push_back(
         ::boost::shared_ptr<Block>(
             new Block(sLeftText, sRightText, mpFont->mxFont, mnMaximalWidth)));
diff --git a/sdext/source/presenter/PresenterToolBar.cxx b/sdext/source/presenter/PresenterToolBar.cxx
index 905c2f3..23a19cd 100644
--- a/sdext/source/presenter/PresenterToolBar.cxx
+++ b/sdext/source/presenter/PresenterToolBar.cxx
@@ -832,8 +832,6 @@ geometry::RealSize2D PresenterToolBar::CalculatePartSize (
 
     if (mxWindow.is())
     {
-        const awt::Rectangle aWindowBox (mxWindow->getPosSize());
-
         // Calculate the summed width of all elements.
         ElementContainerPart::const_iterator iElement;
         for (iElement=rpPart->begin(); iElement!=rpPart->end(); ++iElement)
commit b6b22079f4214fddaa39514a43446e6b3ce6fef0
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Mon Jan 16 13:49:01 2012 +0100

    Clang -Wsign-compare.

diff --git a/vcl/unx/generic/window/salframe.cxx b/vcl/unx/generic/window/salframe.cxx
index f075d99..9ab7a1a 100644
--- a/vcl/unx/generic/window/salframe.cxx
+++ b/vcl/unx/generic/window/salframe.cxx
@@ -2129,7 +2129,7 @@ void X11SalFrame::SetScreenNumber( unsigned int nNewScreen )
             Show( sal_True );
         maGeometry.nDisplayScreenNumber = nNewScreen;
     }
-    else if( sal_Int32(nNewScreen) < GetDisplay()->GetXScreenCount() )
+    else if( nNewScreen < GetDisplay()->GetXScreenCount() )
     {
         bool bVisible = bMapped_;
         if( bVisible )
commit 2091a7fe8739eae4ca4d6ccd23db27e8e7de7203
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Mon Jan 16 13:46:10 2012 +0100

    Clang -Wunused-variable.

diff --git a/sdext/source/minimizer/graphiccollector.cxx b/sdext/source/minimizer/graphiccollector.cxx
index 5e5e5f5..d6c76ab 100644
--- a/sdext/source/minimizer/graphiccollector.cxx
+++ b/sdext/source/minimizer/graphiccollector.cxx
@@ -376,8 +376,6 @@ void ImpCountGraphicObjects( const Reference< XComponentContext >& rxMSF, const
             // now check for a fillstyle
             Reference< XPropertySet > xEmptyPagePropSet;
             Reference< XPropertySet > xShapePropertySet( xShape, UNO_QUERY_THROW );
-            awt::Size aLogicalSize( xShape->getSize() );
-
             FillStyle eFillStyle;
             if ( xShapePropertySet->getPropertyValue( TKGet( TK_FillStyle ) ) >>= eFillStyle )
             {
commit 5a410d2e5a80ae1ea8a11300ddd2d8486e7e095b
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Mon Jan 16 12:09:41 2012 +0100

    Clang -Wunneeded-internal-declaration.

diff --git a/vcl/generic/fontmanager/fontmanager.cxx b/vcl/generic/fontmanager/fontmanager.cxx
index 578cc23..b4a5a4b 100644
--- a/vcl/generic/fontmanager/fontmanager.cxx
+++ b/vcl/generic/fontmanager/fontmanager.cxx
@@ -2587,30 +2587,6 @@ bool PrintFontManager::getMetrics( fontID nFontID, sal_Unicode minCharacter, sal
 
 // -------------------------------------------------------------------------
 
-static bool createWriteablePath( const ByteString& rPath )
-{
-    bool bSuccess = false;
-
-    if( access( rPath.GetBuffer(), W_OK ) )
-    {
-        int nPos = rPath.SearchBackward( '/' );
-        if( nPos != STRING_NOTFOUND )
-            while( nPos > 0 && rPath.GetChar( nPos ) == '/' )
-                nPos--;
-
-        if( nPos != STRING_NOTFOUND && nPos != 0 && createWriteablePath( rPath.Copy( 0, nPos+1 ) ) )
-        {
-            bSuccess = mkdir( rPath.GetBuffer(), 0777 ) ? false : true;
-        }
-    }
-    else
-        bSuccess = true;
-
-    return bSuccess;
-}
-
-// -------------------------------------------------------------------------
-
 // TODO: move most of this stuff into the central font-subsetting code
 bool PrintFontManager::createFontSubset(
                                         FontSubsetInfo& rInfo,
commit 9892e4c8d91fa1cbae0ced6b64c9735d9657a9b8
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Mon Jan 16 12:06:26 2012 +0100

    Clang -Wunused-variable.

diff --git a/sdext/source/pdfimport/tree/pdfiprocessor.cxx b/sdext/source/pdfimport/tree/pdfiprocessor.cxx
index eb490d5..e6e9b33 100644
--- a/sdext/source/pdfimport/tree/pdfiprocessor.cxx
+++ b/sdext/source/pdfimport/tree/pdfiprocessor.cxx
@@ -492,7 +492,6 @@ GraphicsContext& PDFIProcessor::getTransformGlyphContext( CharGlyph& rGlyph )
     geometry::Matrix2D          rFontMatrix = rGlyph.getFontMatrix();
 
     rtl::OUString tempStr( 32 );
-    geometry::RealRectangle2D aRect(rRect);
 
     basegfx::B2DHomMatrix aFontMatrix;
     basegfx::unotools::homMatrixFromMatrix(