Skip to content

C
core
Kaydet (Commit) 126f1084 authored tarafından Stephan Bergmann's avatar Stephan Bergmann
Dosyalara gözat
Seçenekler

More loplugin:cstylecast: emfio 

auto-rewrite with <https://gerrit.libreoffice.org/#/c/47798/> "Enable
loplugin:cstylecast for some more cases" plus
solenv/clang-format/reformat-formatted-files

Change-Id: Idffd3ef04b007f04b7022e54881254da9b2aa4a0
üst 32bd0425
Hide whitespace changes
Inline Side-by-side
Showing with 26 additions and 26 deletions
emfio/qa/cppunit/emf/EmfImportTest.cxx
Dosyayı görüntüle @ 126f1084
......@@ -88,7 +88,7 @@ void Test::checkRectPrimitive(Primitive2DSequence const & rPrimitive)
void Test::testWorking()
{
Primitive2DSequence aSequenceRect = parseEmf("/emfio/qa/cppunit/emf/data/fdo79679-2.emf");
CPPUNIT_ASSERT_EQUAL(1, (int) aSequenceRect.getLength());
CPPUNIT_ASSERT_EQUAL(1, static_cast<int>(aSequenceRect.getLength()));
checkRectPrimitive(aSequenceRect);
}
......@@ -99,7 +99,7 @@ void Test::TestDrawString()
// first, get the sequence of primitives and dump it
Primitive2DSequence aSequence = parseEmf("/emfio/qa/cppunit/emf/data/TestDrawString.emf");
CPPUNIT_ASSERT_EQUAL(1, (int) aSequence.getLength());
CPPUNIT_ASSERT_EQUAL(1, static_cast<int>(aSequence.getLength()));
Primitive2dXmlDump dumper;
xmlDocPtr pDocument = dumper.dumpAndParse(comphelper::sequenceToContainer<Primitive2DContainer>(aSequence));
CPPUNIT_ASSERT (pDocument);
......@@ -120,7 +120,7 @@ void Test::TestDrawLine()
// first, get the sequence of primitives and dump it
Primitive2DSequence aSequence = parseEmf("/emfio/qa/cppunit/emf/data/TestDrawLine.emf");
CPPUNIT_ASSERT_EQUAL(1, (int) aSequence.getLength());
CPPUNIT_ASSERT_EQUAL(1, static_cast<int>(aSequence.getLength()));
Primitive2dXmlDump dumper;
xmlDocPtr pDocument = dumper.dumpAndParse(comphelper::sequenceToContainer<Primitive2DContainer>(aSequence));
CPPUNIT_ASSERT (pDocument);
......
emfio/source/reader/emfreader.cxx
Dosyayı görüntüle @ 126f1084
......@@ -590,7 +590,7 @@ namespace emfio
{
sal_uInt32 nPoints(0);
mpInputStream->ReadUInt32( nPoints );
aPoints[i] = (sal_uInt16)nPoints;
aPoints[i] = static_cast<sal_uInt16>(nPoints);
}
if ( mpInputStream->good() && ( nGesPoints * (sizeof(T)+sizeof(T)) ) <= ( nEndPos - mpInputStream->Tell() ) )
{
......@@ -681,7 +681,7 @@ namespace emfio
mpInputStream->ReadUInt32( nCommentId );
SAL_INFO ("emfio", "\t\tbegin " << (char)(nCommentId & 0xff) << (char)((nCommentId & 0xff00) >> 8) << (char)((nCommentId & 0xff0000) >> 16) << (char)((nCommentId & 0xff000000) >> 24) << " id: 0x" << std::hex << nCommentId << std::dec);
SAL_INFO ("emfio", "\t\tbegin " << static_cast<char>(nCommentId & 0xff) << static_cast<char>((nCommentId & 0xff00) >> 8) << static_cast<char>((nCommentId & 0xff0000) >> 16) << static_cast<char>((nCommentId & 0xff000000) >> 24) << " id: 0x" << std::hex << nCommentId << std::dec);
if( nCommentId == EMR_COMMENT_EMFPLUS && nRecSize >= 12 )
{
......@@ -761,7 +761,7 @@ namespace emfio
{
mpInputStream->ReadUInt32( nNom1 ).ReadUInt32( nDen1 ).ReadUInt32( nNom2 ).ReadUInt32( nDen2 );
if (nDen1 != 0 && nDen2 != 0)
ScaleWinExt( (double)nNom1 / nDen1, (double)nNom2 / nDen2 );
ScaleWinExt( static_cast<double>(nNom1) / nDen1, static_cast<double>(nNom2) / nDen2 );
else
SAL_WARN("vcl.emf", "ignoring bogus divide by zero");
}
......@@ -778,7 +778,7 @@ namespace emfio
{
mpInputStream->ReadUInt32( nNom1 ).ReadUInt32( nDen1 ).ReadUInt32( nNom2 ).ReadUInt32( nDen2 );
if (nDen1 != 0 && nDen2 != 0)
ScaleDevExt( (double)nNom1 / nDen1, (double)nNom2 / nDen2 );
ScaleDevExt( static_cast<double>(nNom1) / nDen1, static_cast<double>(nNom2) / nDen2 );
else
SAL_WARN("vcl.emf", "ignoring bogus divide by zero");
}
......
emfio/source/reader/mtftools.cxx
Dosyayı görüntüle @ 126f1084
......@@ -272,7 +272,7 @@ namespace emfio
if( rFont.lfStrikeOut )
aFont.SetStrikeout( STRIKEOUT_SINGLE );
aFont.SetOrientation( (short)rFont.lfEscapement );
aFont.SetOrientation( static_cast<short>(rFont.lfEscapement) );
Size aFontSize( Size( rFont.lfWidth, rFont.lfHeight ) );
if ( rFont.lfHeight > 0 )
......@@ -287,8 +287,8 @@ namespace emfio
long nHeight = aMetric.GetAscent() + aMetric.GetDescent();
if (nHeight)
{
double fHeight = ((double)aFontSize.Height() * rFont.lfHeight ) / nHeight;
aFontSize.Height() = (sal_Int32)( fHeight + 0.5 );
double fHeight = (static_cast<double>(aFontSize.Height()) * rFont.lfHeight ) / nHeight;
aFontSize.Height() = static_cast<sal_Int32>( fHeight + 0.5 );
}
}
......@@ -303,7 +303,7 @@ namespace emfio
sal_uInt32 nColor;
mpInputStream->ReadUInt32( nColor );
return Color( (sal_uInt8)nColor, (sal_uInt8)( nColor >> 8 ), (sal_uInt8)( nColor >> 16 ) );
return Color( static_cast<sal_uInt8>(nColor), static_cast<sal_uInt8>( nColor >> 8 ), static_cast<sal_uInt8>( nColor >> 16 ) );
};
Point MtfTools::ImplScale(const Point& rPoint) // Hack to set varying defaults for incompletely defined files.
......@@ -394,8 +394,8 @@ namespace emfio
fY2 *= mnDevHeight;
fX2 += mnDevOrgX;
fY2 += mnDevOrgY; // fX2, fY2 now in device units
fX2 *= (double)mnMillX * 100.0 / (double)mnPixX;
fY2 *= (double)mnMillY * 100.0 / (double)mnPixY;
fX2 *= static_cast<double>(mnMillX) * 100.0 / static_cast<double>(mnPixX);
fY2 *= static_cast<double>(mnMillY) * 100.0 / static_cast<double>(mnPixY);
}
}
break;
......@@ -482,8 +482,8 @@ namespace emfio
fHeight /= mnWinExtY;
fWidth *= mnDevWidth;
fHeight *= mnDevHeight;
fWidth *= (double)mnMillX * 100.0 / (double)mnPixX;
fHeight *= (double)mnMillY * 100.0 / (double)mnPixY;
fWidth *= static_cast<double>(mnMillX) * 100.0 / static_cast<double>(mnPixX);
fHeight *= static_cast<double>(mnMillY) * 100.0 / static_cast<double>(mnPixY);
}
}
break;
......@@ -558,7 +558,7 @@ namespace emfio
{
if ( nIndex & ENHMETA_STOCK_OBJECT )
{
sal_uInt16 nStockId = (sal_uInt8)nIndex;
sal_uInt16 nStockId = static_cast<sal_uInt8>(nIndex);
switch( nStockId )
{
case WHITE_BRUSH :
......@@ -617,7 +617,7 @@ namespace emfio
GDIObj *pGDIObj = nullptr;
if ( (sal_uInt32)nIndex < mvGDIObj.size() )
if ( static_cast<sal_uInt32>(nIndex) < mvGDIObj.size() )
pGDIObj = mvGDIObj[ nIndex ].get();
if ( pGDIObj )
......@@ -760,7 +760,7 @@ namespace emfio
}
}
}
if ( (sal_uInt32)nIndex >= mvGDIObj.size() )
if ( static_cast<sal_uInt32>(nIndex) >= mvGDIObj.size() )
ImplResizeObjectArry( nIndex + 16 );
mvGDIObj[ nIndex ] = std::move(pObject);
......@@ -776,7 +776,7 @@ namespace emfio
{
if ( ( nIndex & ENHMETA_STOCK_OBJECT ) == 0 )
{
if ( (sal_uInt32)nIndex < mvGDIObj.size() )
if ( static_cast<sal_uInt32>(nIndex) < mvGDIObj.size() )
{
mvGDIObj[ nIndex ].reset();
}
......@@ -1738,7 +1738,7 @@ namespace emfio
pSave = rSaveList[i].get();
sal_uInt32 nWinRop = pSave->nWinRop;
sal_uInt8 nRasterOperation = (sal_uInt8)( nWinRop >> 16 );
sal_uInt8 nRasterOperation = static_cast<sal_uInt8>( nWinRop >> 16 );
sal_uInt32 nUsed = 0;
if ( ( nRasterOperation & 0xf ) != ( nRasterOperation >> 4 ) )
......
emfio/source/reader/wmfreader.cxx
Dosyayı görüntüle @ 126f1084
......@@ -244,7 +244,7 @@ namespace emfio
mpInputStream->SetError( SVSTREAM_FILEFORMAT_ERROR );
break;
}
ScaleWinExt( (double)nXNum / nXDenom, (double)nYNum / nYDenom );
ScaleWinExt( static_cast<double>(nXNum) / nXDenom, static_cast<double>(nYNum) / nYDenom );
}
break;
......@@ -269,7 +269,7 @@ namespace emfio
mpInputStream->SetError( SVSTREAM_FILEFORMAT_ERROR );
break;
}
ScaleDevExt( (double)nXNum / nXDenom, (double)nYNum / nYDenom );
ScaleDevExt( static_cast<double>(nXNum) / nXDenom, static_cast<double>(nYNum) / nYDenom );
}
break;
......@@ -796,7 +796,7 @@ namespace emfio
nCount++;
pBmp.reset();
}
Color aColor( (sal_uInt8)( nRed / nCount ), (sal_uInt8)( nGreen / nCount ), (sal_uInt8)( nBlue / nCount ) );
Color aColor( static_cast<sal_uInt8>( nRed / nCount ), static_cast<sal_uInt8>( nGreen / nCount ), static_cast<sal_uInt8>( nBlue / nCount ) );
CreateObject(o3tl::make_unique<WinMtfFillStyle>( aColor, false ));
}
break;
......@@ -1107,7 +1107,7 @@ namespace emfio
}
}
}
else if ( (nNewMagic == static_cast< sal_uInt32 >(0x43464D57)) && (nLen >= 34) && ( (sal_Int32)(nLen + 10) <= (sal_Int32)(mnRecSize * 2) ))
else if ( (nNewMagic == static_cast< sal_uInt32 >(0x43464D57)) && (nLen >= 34) && ( static_cast<sal_Int32>(nLen + 10) <= static_cast<sal_Int32>(mnRecSize * 2) ))
{
sal_uInt32 nComType = 0, nVersion = 0, nFlags = 0, nComRecCount = 0,
nCurRecSize = 0, nRemainingSize = 0, nEMFTotalSize = 0;
......@@ -1252,8 +1252,8 @@ namespace emfio
{
// #n417818#: If we have an external header then overwrite the bounds!
tools::Rectangle aExtRect(0, 0,
(double)mpExternalHeader->xExt * 567 * mnUnitsPerInch / 1440000,
(double)mpExternalHeader->yExt * 567 * mnUnitsPerInch / 1440000);
static_cast<double>(mpExternalHeader->xExt) * 567 * mnUnitsPerInch / 1440000,
static_cast<double>(mpExternalHeader->yExt) * 567 * mnUnitsPerInch / 1440000);
aPlaceableBound = aExtRect;
SAL_INFO("vcl.wmf", "External header size "
......
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