[Libreoffice-commits] core.git: filter/source

Noel Grandin noel at peralex.com
Mon Apr 28 04:19:38 PDT 2014


 filter/source/graphicfilter/icgm/cgm.cxx |   31 -------------------------------
 filter/source/graphicfilter/icgm/cgm.hxx |    6 +-----
 2 files changed, 1 insertion(+), 36 deletions(-)

New commits:
commit a3abc602578270582ab7d8df9ef34ef9503234de
Author: Noel Grandin <noel at peralex.com>
Date:   Fri Apr 25 16:00:39 2014 +0200

    remove some dodgy defines from the ICGM filter
    
    One of these looks it was somebodys temporary debugging code.
    The others constants are are always defined,
    so the #ifdef blocks are pointless.
    
    Change-Id: I4817e9ce268ab58f5e329830e2e332e2b4ebf4dc
    Reviewed-on: https://gerrit.libreoffice.org/9163
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/filter/source/graphicfilter/icgm/cgm.cxx b/filter/source/graphicfilter/icgm/cgm.cxx
index 780b5af..d168101 100644
--- a/filter/source/graphicfilter/icgm/cgm.cxx
+++ b/filter/source/graphicfilter/icgm/cgm.cxx
@@ -32,8 +32,6 @@
 
 using namespace ::com::sun::star;
 
-#ifdef CGM_EXPORT_IMPRESS
-
 CGM::CGM( sal_uInt32 nMode, uno::Reference< frame::XModel > & rModel )
     : mnOutdx(28000)
     , mnOutdy(21000)
@@ -66,20 +64,16 @@ CGM::CGM( sal_uInt32 nMode, uno::Reference< frame::XModel > & rModel )
     , mnElementClass(0)
     , mnElementID(0)
     , mnElementSize(0)
-#ifdef CGM_EXPORT_META
     , mpVirDev(NULL)
     , mpGDIMetaFile(NULL)
-#endif
 {
     pElement = new CGMElements( *this );
     pCopyOfE = new CGMElements( *this );
 }
-#endif
 
 CGM::~CGM()
 {
 
-#ifdef CGM_EXPORT_META
     if ( mpGraphic )
     {
         mpGDIMetaFile->Stop();
@@ -88,7 +82,6 @@ CGM::~CGM()
         delete mpVirDev;
         *mpGraphic = Graphic( *mpGDIMetaFile );
     }
-#endif
     for( size_t i = 0, n = maDefRepList.size(); i < n; ++i )
         delete maDefRepList[ i ];
     maDefRepList.clear();
@@ -594,13 +587,6 @@ void CGM::ImplMapPoint( FloatPoint& rFloatPoint )
 
 void CGM::ImplDoClass()
 {
-#ifdef CGM_USER_BREAKPOINT
-#ifdef WNT
-#define CGM_BREAK_ACTION    0xffffffff
-    if ( mnActCount == CGM_BREAK_ACTION )
-        _asm int 0x3;
-#endif
-#endif
     switch ( mnElementClass )
     {
         case 0 : ImplDoClass0(); break;
@@ -695,14 +681,6 @@ bool CGM::Write( SvStream& rIStm )
         rIStm.SeekRel( 1 );
     ImplDoClass();
 
-
-#ifdef CGM_USER_BREAKPOINT
-#ifdef WNT
-    if ( !mbStatus || mnParaSize && ( mnElementSize != mnParaSize ) )
-        _asm int 0x3;
-#endif
-#endif
-
     return mbStatus;
 };
 
@@ -731,7 +709,6 @@ ImportCGM( OUString& rFileName, uno::Reference< frame::XModel > & rXModel, sal_u
                         sal_uInt64 const nInSize = pIn->remainingSize();
                         pIn->Seek( 0 );
 
-#ifdef CGM_EXPORT_IMPRESS
                         uno::Reference< task::XStatusIndicator >  aXStatInd;
                         sal_uInt32  nNext = 0;
                         sal_uInt32  nAdd = nInSize / 20;
@@ -740,14 +717,9 @@ ImportCGM( OUString& rFileName, uno::Reference< frame::XModel > & rXModel, sal_u
                         bProgressBar = aXStatInd.is();
                         if ( bProgressBar )
                             aXStatInd->start( "CGM Import" , nInSize );
-#endif
 
                         while ( pCGM->IsValid() && ( pIn->Tell() < nInSize ) && !pCGM->IsFinished() )
                         {
-
-#ifdef CGM_EXPORT_IMPRESS
-
-
                             if ( bProgressBar )
                             {
                                 sal_uInt32 nCurrentPos = pIn->Tell();
@@ -757,7 +729,6 @@ ImportCGM( OUString& rFileName, uno::Reference< frame::XModel > & rXModel, sal_u
                                     nNext = nCurrentPos + nAdd;
                                 }
                             }
-#endif
 
                             if ( pCGM->Write( *pIn ) == false )
                                 break;
@@ -766,10 +737,8 @@ ImportCGM( OUString& rFileName, uno::Reference< frame::XModel > & rXModel, sal_u
                         {
                             nStatus = pCGM->GetBackGroundColor() | 0xff000000;
                         }
-#ifdef CGM_EXPORT_IMPRESS
                         if ( bProgressBar )
                             aXStatInd->end();
-#endif
                     }
                 }
             }
diff --git a/filter/source/graphicfilter/icgm/cgm.hxx b/filter/source/graphicfilter/icgm/cgm.hxx
index 3bdd0c3..25ab80a 100644
--- a/filter/source/graphicfilter/icgm/cgm.hxx
+++ b/filter/source/graphicfilter/icgm/cgm.hxx
@@ -22,8 +22,6 @@
 
 #include <com/sun/star/frame/XModel.hpp>
 
-#undef CGM_USER_BREAKPOINT
-
 #define CGM_IMPORT_CGM      0x00000001
 #define CGM_EXPORT_IMPRESS  0x00000100
 #define CGM_EXPORT_META     0x00000200
@@ -61,7 +59,7 @@ class CGM
         double              mnYFraction;
         bool                mbAngReverse;           // AngularDirection
 
-        Graphic*            mpGraphic;              // ifdef CGM_EXPORT_META
+        Graphic*            mpGraphic;
 
         bool                mbStatus;
         bool                mbMetaFile;
@@ -139,10 +137,8 @@ class CGM
                             ~CGM();
 
                             CGM( sal_uInt32 nMode, ::com::sun::star::uno::Reference< ::com::sun::star::frame::XModel > & rModel );
-#ifdef CGM_EXPORT_META
         VirtualDevice*      mpVirDev;
         GDIMetaFile*        mpGDIMetaFile;
-#endif
         sal_uInt32              GetBackGroundColor();
         bool                IsValid() const { return mbStatus; };
         bool                IsFinished() const { return mbIsFinished; };


More information about the Libreoffice-commits mailing list