From a79e6a7cf1ce3be46e4339a54b013ddaa534dd39 Mon Sep 17 00:00:00 2001 From: Noel Date: Tue, 15 Dec 2020 09:38:19 +0200 Subject: use views to parse rather than allocating OUString Change-Id: If0a848c64ce8077d1681661873629c83307cf8b2 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107736 Tested-by: Jenkins Reviewed-by: Noel Grandin --- xmloff/source/chart/SchXMLAxisContext.cxx | 24 ++-- xmloff/source/chart/SchXMLChartContext.cxx | 23 ++-- xmloff/source/chart/SchXMLLegendContext.cxx | 17 ++- xmloff/source/chart/SchXMLPlotAreaContext.cxx | 27 +++-- xmloff/source/chart/SchXMLPlotAreaContext.hxx | 2 +- .../chart/SchXMLRegressionCurveObjectContext.cxx | 11 +- xmloff/source/chart/SchXMLTableContext.cxx | 8 +- xmloff/source/core/xmluconv.cxx | 72 +++++++++++- xmloff/source/draw/QRCodeContext.cxx | 2 +- xmloff/source/draw/XMLNumberStyles.cxx | 11 +- xmloff/source/draw/animationimport.cxx | 127 ++++++++++---------- xmloff/source/draw/animimp.cxx | 20 ++-- xmloff/source/draw/eventimp.cxx | 30 +++-- xmloff/source/draw/ximp3dobject.cxx | 12 +- xmloff/source/draw/ximp3dscene.cxx | 11 +- xmloff/source/draw/ximpcustomshape.cxx | 130 ++++++++++----------- xmloff/source/draw/ximppage.cxx | 10 +- xmloff/source/draw/ximpshap.cxx | 2 +- xmloff/source/draw/ximpshow.cxx | 28 +++-- xmloff/source/draw/ximpstyl.cxx | 27 ++--- xmloff/source/style/DashStyle.cxx | 30 +++-- xmloff/source/style/GradientStyle.cxx | 24 ++-- xmloff/source/style/HatchStyle.cxx | 13 +-- xmloff/source/style/TransGradientStyle.cxx | 20 ++-- xmloff/source/style/XMLFootnoteSeparatorImport.cxx | 15 ++- xmloff/source/style/xmlnumfi.cxx | 86 +++++++------- xmloff/source/style/xmlnumi.cxx | 78 ++++++------- xmloff/source/style/xmltabi.cxx | 26 ++--- .../source/text/XMLCalculationSettingsContext.cxx | 4 +- .../source/text/XMLChangedRegionImportContext.cxx | 7 +- .../text/XMLFootnoteConfigurationImportContext.cxx | 2 +- xmloff/source/text/XMLFootnoteImportContext.cxx | 2 +- .../XMLIndexBibliographyConfigurationContext.cxx | 2 +- .../text/XMLIndexChapterInfoEntryContext.cxx | 2 +- xmloff/source/text/XMLIndexTOCContext.cxx | 2 +- xmloff/source/text/XMLIndexTOCStylesContext.cxx | 2 +- xmloff/source/text/XMLIndexTabStopEntryContext.cxx | 11 +- .../XMLLineNumberingSeparatorImportContext.cxx | 2 +- .../source/text/XMLSectionFootnoteConfigImport.cxx | 14 +-- xmloff/source/text/XMLSectionImportContext.cxx | 21 ++-- .../text/XMLSectionSourceDDEImportContext.cxx | 2 +- xmloff/source/text/XMLTextColumnsContext.cxx | 28 ++--- xmloff/source/text/XMLTextFrameContext.cxx | 96 ++++++++------- .../source/text/XMLTextFrameHyperlinkContext.cxx | 11 +- xmloff/source/text/XMLTextListBlockContext.cxx | 9 +- xmloff/source/text/XMLTextShapeImportHelper.cxx | 7 +- .../source/text/XMLTrackedChangesImportContext.cxx | 2 +- xmloff/source/text/txtdropi.cxx | 12 +- xmloff/source/text/txtfldi.cxx | 7 +- xmloff/source/text/txtparai.cxx | 37 +++--- xmloff/source/text/txtvfldi.cxx | 2 +- 51 files changed, 585 insertions(+), 585 deletions(-) (limited to 'xmloff') diff --git a/xmloff/source/chart/SchXMLAxisContext.cxx b/xmloff/source/chart/SchXMLAxisContext.cxx index 17ba48af5c9e..dd6c1fd62596 100644 --- a/xmloff/source/chart/SchXMLAxisContext.cxx +++ b/xmloff/source/chart/SchXMLAxisContext.cxx @@ -235,30 +235,29 @@ void SchXMLAxisContext::startFastElement( sal_Int32 /*nElement*/, // parse attributes for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString aValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(CHART, XML_DIMENSION): { SchXMLAxisDimension nEnumVal; - if( SvXMLUnitConverter::convertEnum( nEnumVal, aValue, aXMLAxisDimensionMap )) + if( SvXMLUnitConverter::convertEnum( nEnumVal, aIter.toString(), aXMLAxisDimensionMap )) m_aCurrentAxis.eDimension = nEnumVal; } break; case XML_ELEMENT(CHART, XML_NAME): - m_aCurrentAxis.aName = aValue; + m_aCurrentAxis.aName = aIter.toString(); break; case XML_ELEMENT(CHART, XML_AXIS_TYPE): case XML_ELEMENT(CHART_EXT, XML_AXIS_TYPE): sal_uInt16 nEnumVal; - if( SvXMLUnitConverter::convertEnum( nEnumVal, aValue, aXMLAxisTypeMap )) + if( SvXMLUnitConverter::convertEnum( nEnumVal, aIter.toString(), aXMLAxisTypeMap )) { m_nAxisType = nEnumVal; m_bAxisTypeImported = true; } break; case XML_ELEMENT(CHART, XML_STYLE_NAME): - m_aAutoStyleName = aValue; + m_aAutoStyleName = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -611,7 +610,7 @@ css::uno::Reference< css::xml::sax::XFastContextHandler > SchXMLAxisContext::cre switch (aIter.getToken()) { case XML_ELEMENT(CHART, XML_CLASS): - if( IsXMLToken( aIter.toString(), XML_MINOR ) ) + if( IsXMLToken( aIter, XML_MINOR ) ) bIsMajor = false; break; case XML_ELEMENT(CHART, XML_STYLE_NAME): @@ -819,7 +818,7 @@ DateScaleContext::DateScaleContext( namespace { -sal_Int32 lcl_getTimeUnit( const OUString& rValue ) +sal_Int32 lcl_getTimeUnit( const sax_fastparser::FastAttributeList::FastAttributeIter& rValue ) { sal_Int32 nTimeUnit = css::chart::TimeUnit::DAY; if( IsXMLToken( rValue, XML_DAYS ) ) @@ -846,12 +845,11 @@ void DateScaleContext::startFastElement( sal_Int32 /*nElement*/, for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString aValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(CHART, XML_BASE_TIME_UNIT): { - aIncrement.TimeResolution <<= lcl_getTimeUnit(aValue); + aIncrement.TimeResolution <<= lcl_getTimeUnit(aIter); bSetNewIncrement = true; } break; @@ -859,7 +857,7 @@ void DateScaleContext::startFastElement( sal_Int32 /*nElement*/, { chart::TimeInterval aInterval(1,0); aIncrement.MajorTimeInterval >>= aInterval; - ::sax::Converter::convertNumber( aInterval.Number, aValue ); + ::sax::Converter::convertNumber( aInterval.Number, aIter.toView() ); aIncrement.MajorTimeInterval <<= aInterval; bSetNewIncrement = true; } @@ -868,7 +866,7 @@ void DateScaleContext::startFastElement( sal_Int32 /*nElement*/, { chart::TimeInterval aInterval(1,0); aIncrement.MajorTimeInterval >>= aInterval; - aInterval.TimeUnit = lcl_getTimeUnit(aValue); + aInterval.TimeUnit = lcl_getTimeUnit(aIter); aIncrement.MajorTimeInterval <<= aInterval; bSetNewIncrement = true; } @@ -877,7 +875,7 @@ void DateScaleContext::startFastElement( sal_Int32 /*nElement*/, { chart::TimeInterval aInterval(1,0); aIncrement.MinorTimeInterval >>= aInterval; - ::sax::Converter::convertNumber( aInterval.Number, aValue ); + ::sax::Converter::convertNumber( aInterval.Number, aIter.toView() ); aIncrement.MinorTimeInterval <<= aInterval; bSetNewIncrement = true; } @@ -886,7 +884,7 @@ void DateScaleContext::startFastElement( sal_Int32 /*nElement*/, { chart::TimeInterval aInterval(1,0); aIncrement.MinorTimeInterval >>= aInterval; - aInterval.TimeUnit = lcl_getTimeUnit(aValue); + aInterval.TimeUnit = lcl_getTimeUnit(aIter); aIncrement.MinorTimeInterval <<= aInterval; bSetNewIncrement = true; } diff --git a/xmloff/source/chart/SchXMLChartContext.cxx b/xmloff/source/chart/SchXMLChartContext.cxx index 8b4f79582138..9c97c64a5a29 100644 --- a/xmloff/source/chart/SchXMLChartContext.cxx +++ b/xmloff/source/chart/SchXMLChartContext.cxx @@ -319,17 +319,17 @@ void SchXMLChartContext::startFastElement( sal_Int32 /*nElement*/, for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString aValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(LO_EXT, XML_DATA_PILOT_SOURCE): - msDataPilotSource = aValue; + msDataPilotSource = aIter.toString(); break; case XML_ELEMENT(XLINK, XML_HREF): - m_aXLinkHRefAttributeToIndicateDataProvider = aValue; + m_aXLinkHRefAttributeToIndicateDataProvider = aIter.toString(); break; case XML_ELEMENT(CHART, XML_CLASS): { + OUString aValue = aIter.toString(); OUString sClassName; sal_uInt16 nClassPrefix = GetImport().GetNamespaceMap().GetKeyByAttrValueQName( @@ -365,24 +365,24 @@ void SchXMLChartContext::startFastElement( sal_Int32 /*nElement*/, case XML_ELEMENT(SVG, XML_WIDTH): case XML_ELEMENT(SVG_COMPAT, XML_WIDTH): GetImport().GetMM100UnitConverter().convertMeasureToCore( - maChartSize.Width, aValue ); + maChartSize.Width, aIter.toView() ); break; case XML_ELEMENT(SVG, XML_HEIGHT): case XML_ELEMENT(SVG_COMPAT, XML_HEIGHT): GetImport().GetMM100UnitConverter().convertMeasureToCore( - maChartSize.Height, aValue ); + maChartSize.Height, aIter.toView() ); break; case XML_ELEMENT(CHART, XML_STYLE_NAME): - sAutoStyleName = aValue; + sAutoStyleName = aIter.toString(); break; case XML_ELEMENT(CHART, XML_COLUMN_MAPPING): - msColTrans = aValue; + msColTrans = aIter.toString(); break; case XML_ELEMENT(CHART, XML_ROW_MAPPING): - msRowTrans = aValue; + msRowTrans = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -1176,14 +1176,13 @@ void SchXMLTitleContext::startFastElement( sal_Int32 /*nElement*/, for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString aValue = aIter.toString(); switch (aIter.getToken()) { case XML_ELEMENT(SVG, XML_X): case XML_ELEMENT(SVG_COMPAT, XML_X): { GetImport().GetMM100UnitConverter().convertMeasureToCore( - aPosition.X, aValue ); + aPosition.X, aIter.toView() ); bHasXPosition = true; break; } @@ -1191,12 +1190,12 @@ void SchXMLTitleContext::startFastElement( sal_Int32 /*nElement*/, case XML_ELEMENT(SVG_COMPAT, XML_Y): { GetImport().GetMM100UnitConverter().convertMeasureToCore( - aPosition.Y, aValue ); + aPosition.Y, aIter.toView() ); bHasYPosition = true; break; } case XML_ELEMENT(CHART, XML_STYLE_NAME): - msAutoStyleName = aValue; + msAutoStyleName = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); diff --git a/xmloff/source/chart/SchXMLLegendContext.cxx b/xmloff/source/chart/SchXMLLegendContext.cxx index 758863a688c6..4cc10dfe7a9e 100644 --- a/xmloff/source/chart/SchXMLLegendContext.cxx +++ b/xmloff/source/chart/SchXMLLegendContext.cxx @@ -85,13 +85,12 @@ void SchXMLLegendContext::startFastElement( sal_Int32 /*nElement*/, for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString aValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(CHART, XML_LEGEND_POSITION): try { - if( SchXMLEnumConverter::getLegendPositionConverter().importXML( aValue, aAny, GetImport().GetMM100UnitConverter() ) ) + if( SchXMLEnumConverter::getLegendPositionConverter().importXML( aIter.toString(), aAny, GetImport().GetMM100UnitConverter() ) ) xLegendProps->setPropertyValue("Alignment", aAny ); } catch(const beans::UnknownPropertyException&) @@ -102,7 +101,7 @@ void SchXMLLegendContext::startFastElement( sal_Int32 /*nElement*/, case XML_ELEMENT(LO_EXT, XML_OVERLAY): try { - bOverlay = aValue.toBoolean(); + bOverlay = aIter.toBoolean(); xLegendProps->setPropertyValue("Overlay", uno::makeAny(bOverlay)); } catch(const beans::UnknownPropertyException&) @@ -113,20 +112,20 @@ void SchXMLLegendContext::startFastElement( sal_Int32 /*nElement*/, case XML_ELEMENT(SVG, XML_X): case XML_ELEMENT(SVG_COMPAT, XML_X): GetImport().GetMM100UnitConverter().convertMeasureToCore( - aLegendPos.X, aValue ); + aLegendPos.X, aIter.toView() ); bHasXPosition = true; break; case XML_ELEMENT(SVG, XML_Y): case XML_ELEMENT(SVG_COMPAT, XML_Y): GetImport().GetMM100UnitConverter().convertMeasureToCore( - aLegendPos.Y, aValue ); + aLegendPos.Y, aIter.toView() ); bHasYPosition = true; break; case XML_ELEMENT(CHART, XML_STYLE_NAME): - sAutoStyleName = aValue; + sAutoStyleName = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_LEGEND_EXPANSION): - SchXMLEnumConverter::getLegendPositionConverter().importXML( aValue, aAny, GetImport().GetMM100UnitConverter() ); + SchXMLEnumConverter::getLegendPositionConverter().importXML( aIter.toString(), aAny, GetImport().GetMM100UnitConverter() ); bHasExpansion = (aAny>>=nLegendExpansion); break; case XML_ELEMENT(STYLE, XML_LEGEND_EXPANSION_ASPECT_RATIO): @@ -135,14 +134,14 @@ void SchXMLLegendContext::startFastElement( sal_Int32 /*nElement*/, case XML_ELEMENT(SVG_COMPAT, XML_WIDTH): case XML_ELEMENT(CHART_EXT, XML_WIDTH): GetImport().GetMM100UnitConverter().convertMeasureToCore( - aLegendSize.Width, aValue ); + aLegendSize.Width, aIter.toView() ); bHasWidth = true; break; case XML_ELEMENT(SVG, XML_HEIGHT): case XML_ELEMENT(SVG_COMPAT, XML_HEIGHT): case XML_ELEMENT(CHART_EXT, XML_HEIGHT): GetImport().GetMM100UnitConverter().convertMeasureToCore( - aLegendSize.Height, aValue ); + aLegendSize.Height, aIter.toView() ); bHasHeight = true; break; default: diff --git a/xmloff/source/chart/SchXMLPlotAreaContext.cxx b/xmloff/source/chart/SchXMLPlotAreaContext.cxx index 985a4a633212..2d870e9b2f8e 100644 --- a/xmloff/source/chart/SchXMLPlotAreaContext.cxx +++ b/xmloff/source/chart/SchXMLPlotAreaContext.cxx @@ -225,23 +225,23 @@ void SchXMLPlotAreaContext::startFastElement (sal_Int32 /*nElement*/, case XML_ELEMENT(SVG_COMPAT, XML_WIDTH): case XML_ELEMENT(SVG, XML_HEIGHT): case XML_ELEMENT(SVG_COMPAT, XML_HEIGHT): - m_aOuterPositioning.readPositioningAttribute( aIter.getToken(), aValue ); + m_aOuterPositioning.readPositioningAttribute( aIter.getToken(), aIter.toView() ); break; case XML_ELEMENT(CHART, XML_STYLE_NAME): - msAutoStyleName = aValue; + msAutoStyleName = aIter.toString(); break; case XML_ELEMENT(TABLE, XML_CELL_RANGE_ADDRESS): - mrChartAddress = lcl_ConvertRange( aValue, xNewDoc ); + mrChartAddress = lcl_ConvertRange( aIter.toString(), xNewDoc ); // indicator for getting data from the outside m_rbHasRangeAtPlotArea = true; break; case XML_ELEMENT(CHART, XML_DATA_SOURCE_HAS_LABELS): { - if( aValue == ::xmloff::token::GetXMLToken( ::xmloff::token::XML_BOTH )) + if( IsXMLToken(aIter, XML_BOTH) ) mrColHasLabels = mrRowHasLabels = true; - else if( aValue == ::xmloff::token::GetXMLToken( ::xmloff::token::XML_ROW )) + else if( IsXMLToken(aIter, XML_ROW) ) mrRowHasLabels = true; - else if( aValue == ::xmloff::token::GetXMLToken( ::xmloff::token::XML_COLUMN )) + else if( IsXMLToken(aIter, XML_COLUMN) ) mrColHasLabels = true; } break; @@ -643,14 +643,13 @@ void SchXMLDataLabelContext::startFastElement( { for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(SVG, XML_X): case XML_ELEMENT(SVG_COMPAT, XML_X): { sal_Int32 nResultValue; - GetImport().GetMM100UnitConverter().convertMeasureToCore(nResultValue, sValue); + GetImport().GetMM100UnitConverter().convertMeasureToCore(nResultValue, aIter.toView()); mrDataLabelStyle.mo_nLabelAbsolutePosX = nResultValue; break; } @@ -658,12 +657,12 @@ void SchXMLDataLabelContext::startFastElement( case XML_ELEMENT(SVG_COMPAT, XML_Y): { sal_Int32 nResultValue; - GetImport().GetMM100UnitConverter().convertMeasureToCore(nResultValue, sValue); + GetImport().GetMM100UnitConverter().convertMeasureToCore(nResultValue, aIter.toView()); mrDataLabelStyle.mo_nLabelAbsolutePosY = nResultValue; break; } case XML_ELEMENT(CHART, XML_STYLE_NAME): - mrDataLabelStyle.msStyleName = sValue; + mrDataLabelStyle.msStyleName = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -744,7 +743,7 @@ void SchXMLDataPointContext::startFastElement (sal_Int32 /*Element*/, break; case XML_ELEMENT(LO_EXT, XML_HIDE_LEGEND): { - bool bHideLegend = aIter.toString().toBoolean(); + bool bHideLegend = aIter.toBoolean(); if (bHideLegend) { uno::Sequence deletedLegendEntriesSeq; @@ -818,7 +817,7 @@ bool SchXMLPositionAttributesHelper::isAutomatic() const return m_bAutoSize || m_bAutoPosition; } -void SchXMLPositionAttributesHelper::readPositioningAttribute( sal_Int32 nAttributeToken, std::u16string_view rValue ) +void SchXMLPositionAttributesHelper::readPositioningAttribute( sal_Int32 nAttributeToken, std::string_view rValue ) { if( !IsTokenInNamespace(nAttributeToken, XML_NAMESPACE_SVG) && !IsTokenInNamespace(nAttributeToken, XML_NAMESPACE_SVG_COMPAT) ) return; @@ -854,7 +853,7 @@ void SchXMLPositionAttributesHelper::readPositioningAttribute( sal_Int32 nAttrib break; } default: - XMLOFF_WARN_UNKNOWN_ATTR("xmloff", nAttributeToken, OUString(rValue)); + XMLOFF_WARN_UNKNOWN_ATTR("xmloff", nAttributeToken, OUString::fromUtf8(rValue)); } } @@ -887,7 +886,7 @@ void SchXMLCoordinateRegionContext::startFastElement (sal_Int32 /*Element*/, { // parse attributes for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) - m_rPositioning.readPositioningAttribute( aIter.getToken(), aIter.toString() ); + m_rPositioning.readPositioningAttribute( aIter.getToken(), aIter.toView() ); } SchXMLWallFloorContext::SchXMLWallFloorContext( diff --git a/xmloff/source/chart/SchXMLPlotAreaContext.hxx b/xmloff/source/chart/SchXMLPlotAreaContext.hxx index aeda126bd33d..137bbb45d6e8 100644 --- a/xmloff/source/chart/SchXMLPlotAreaContext.hxx +++ b/xmloff/source/chart/SchXMLPlotAreaContext.hxx @@ -60,7 +60,7 @@ public: explicit SchXMLPositionAttributesHelper( SvXMLImport& rImporter ); ~SchXMLPositionAttributesHelper(); - void readPositioningAttribute( sal_Int32 nAttributeToken, std::u16string_view rValue ); + void readPositioningAttribute( sal_Int32 nAttributeToken, std::string_view rValue ); void readAutomaticPositioningProperties( XMLPropStyleContext const * pPropStyleContext, const SvXMLStylesContext* pStylesCtxt ); bool hasPosSize() const; diff --git a/xmloff/source/chart/SchXMLRegressionCurveObjectContext.cxx b/xmloff/source/chart/SchXMLRegressionCurveObjectContext.cxx index fcb95607b6aa..aac03261fb29 100644 --- a/xmloff/source/chart/SchXMLRegressionCurveObjectContext.cxx +++ b/xmloff/source/chart/SchXMLRegressionCurveObjectContext.cxx @@ -117,29 +117,28 @@ void SchXMLEquationContext::startFastElement (sal_Int32 /*nElement*/, for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString aValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(SVG, XML_X): case XML_ELEMENT(SVG_COMPAT, XML_X): rImport.GetMM100UnitConverter().convertMeasureToCore( - aPosition.X, aValue ); + aPosition.X, aIter.toView() ); bHasXPos = true; break; case XML_ELEMENT(SVG, XML_Y): case XML_ELEMENT(SVG_COMPAT, XML_Y): rImport.GetMM100UnitConverter().convertMeasureToCore( - aPosition.Y, aValue ); + aPosition.Y, aIter.toView() ); bHasYPos = true; break; case XML_ELEMENT(CHART, XML_DISPLAY_EQUATION): - (void)::sax::Converter::convertBool(bShowEquation, aValue); + (void)::sax::Converter::convertBool(bShowEquation, aIter.toView()); break; case XML_ELEMENT(CHART, XML_DISPLAY_R_SQUARE): - (void)::sax::Converter::convertBool(bShowRSquare, aValue); + (void)::sax::Converter::convertBool(bShowRSquare, aIter.toView()); break; case XML_ELEMENT(CHART, XML_STYLE_NAME): - sAutoStyleName = aValue; + sAutoStyleName = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); diff --git a/xmloff/source/chart/SchXMLTableContext.cxx b/xmloff/source/chart/SchXMLTableContext.cxx index 8b128461bf35..ac5bcdbc2515 100644 --- a/xmloff/source/chart/SchXMLTableContext.cxx +++ b/xmloff/source/chart/SchXMLTableContext.cxx @@ -261,7 +261,7 @@ void SchXMLTableContext::startFastElement (sal_Int32 /*nElement*/, mrTable.aTableNameOfFile = aIter.toString(); break; case XML_ELEMENT(TABLE, XML_PROTECTED): - if ( IsXMLToken( aIter.toString(), XML_TRUE ) ) + if ( IsXMLToken( aIter, XML_TRUE ) ) { mrTable.bProtected = true; } @@ -574,7 +574,6 @@ SchXMLTableCellContext::~SchXMLTableCellContext() void SchXMLTableCellContext::startFastElement (sal_Int32 /*nElement*/, const css::uno::Reference< css::xml::sax::XFastAttributeList >& xAttrList) { - OUString aValue; OUString aCellContent; SchXMLCellType eValueType = SCH_CELL_TYPE_UNKNOWN; @@ -583,10 +582,9 @@ void SchXMLTableCellContext::startFastElement (sal_Int32 /*nElement*/, switch(aIter.getToken()) { case XML_ELEMENT(OFFICE, XML_VALUE_TYPE): - aValue = aIter.toString(); - if( IsXMLToken( aValue, XML_FLOAT ) ) + if( IsXMLToken( aIter, XML_FLOAT ) ) eValueType = SCH_CELL_TYPE_FLOAT; - else if( IsXMLToken( aValue, XML_STRING ) ) + else if( IsXMLToken( aIter, XML_STRING ) ) eValueType = SCH_CELL_TYPE_STRING; break; diff --git a/xmloff/source/core/xmluconv.cxx b/xmloff/source/core/xmluconv.cxx index 98aa54df7b51..6b5a6b65634e 100644 --- a/xmloff/source/core/xmluconv.cxx +++ b/xmloff/source/core/xmluconv.cxx @@ -177,6 +177,16 @@ bool SvXMLUnitConverter::convertMeasureToCore( sal_Int32& nValue, nMin, nMax ); } +/** convert string to measure using optional min and max values*/ +bool SvXMLUnitConverter::convertMeasureToCore( sal_Int32& nValue, + std::string_view rString, + sal_Int32 nMin, sal_Int32 nMax ) const +{ + return ::sax::Converter::convertMeasure( nValue, rString, + m_pImpl->m_eCoreMeasureUnit, + nMin, nMax ); +} + /** convert measure to string */ void SvXMLUnitConverter::convertMeasureToXML( OUStringBuffer& rString, sal_Int32 nMeasure ) const @@ -501,8 +511,40 @@ static bool lcl_getPositions(const OUString& _sValue,OUString& _rContentX,OUStri _rContentZ = _sValue.copy(nPos, nFound - nPos); return true; +} + +static bool lcl_getPositions(std::string_view _sValue,OUString& _rContentX,OUString& _rContentY,OUString& _rContentZ) +{ + if(_sValue.empty() || _sValue[0] != '(') + return false; + + size_t nPos(1); + size_t nFound = _sValue.find(' ', nPos); + + if(nFound == std::string_view::npos || nFound <= nPos) + return false; + + _rContentX = OUString::fromUtf8(_sValue.substr(nPos, nFound - nPos)); + + nPos = nFound + 1; + nFound = _sValue.find(' ', nPos); + + if(nFound == std::string_view::npos || nFound <= nPos) + return false; + + _rContentY = OUString::fromUtf8(_sValue.substr(nPos, nFound - nPos)); + + nPos = nFound + 1; + nFound = _sValue.find(')', nPos); + + if(nFound == std::string_view::npos || nFound <= nPos) + return false; + + _rContentZ = OUString::fromUtf8(_sValue.substr(nPos, nFound - nPos)); + return true; } + /** convert string to ::basegfx::B3DVector */ bool SvXMLUnitConverter::convertB3DVector( ::basegfx::B3DVector& rVector, const OUString& rValue ) { @@ -531,6 +573,34 @@ bool SvXMLUnitConverter::convertB3DVector( ::basegfx::B3DVector& rVector, const return ( eStatus == rtl_math_ConversionStatus_Ok ); } +/** convert string to ::basegfx::B3DVector */ +bool SvXMLUnitConverter::convertB3DVector( ::basegfx::B3DVector& rVector, std::string_view rValue ) +{ + OUString aContentX,aContentY,aContentZ; + if ( !lcl_getPositions(rValue,aContentX,aContentY,aContentZ) ) + return false; + + rtl_math_ConversionStatus eStatus; + + rVector.setX(::rtl::math::stringToDouble(aContentX, '.', + ',', &eStatus)); + + if( eStatus != rtl_math_ConversionStatus_Ok ) + return false; + + rVector.setY(::rtl::math::stringToDouble(aContentY, '.', + ',', &eStatus)); + + if( eStatus != rtl_math_ConversionStatus_Ok ) + return false; + + rVector.setZ(::rtl::math::stringToDouble(aContentZ, '.', + ',', &eStatus)); + + + return ( eStatus == rtl_math_ConversionStatus_Ok ); +} + /** convert ::basegfx::B3DVector to string */ void SvXMLUnitConverter::convertB3DVector( OUStringBuffer &rBuffer, const ::basegfx::B3DVector& rVector ) { @@ -545,7 +615,7 @@ void SvXMLUnitConverter::convertB3DVector( OUStringBuffer &rBuffer, const ::base /** convert string to Position3D */ bool SvXMLUnitConverter::convertPosition3D( drawing::Position3D& rPosition, - const OUString& rValue ) + std::string_view rValue ) { OUString aContentX,aContentY,aContentZ; if ( !lcl_getPositions(rValue,aContentX,aContentY,aContentZ) ) diff --git a/xmloff/source/draw/QRCodeContext.cxx b/xmloff/source/draw/QRCodeContext.cxx index 978737b34131..f217fcf17ffa 100644 --- a/xmloff/source/draw/QRCodeContext.cxx +++ b/xmloff/source/draw/QRCodeContext.cxx @@ -65,7 +65,7 @@ QRCodeContext::QRCodeContext(SvXMLImport& rImport, sal_Int32 /*nElement*/, case XML_ELEMENT(LO_EXT, XML_QRCODE_BORDER): { sal_Int32 nAttrVal; - if (sax::Converter::convertNumber(nAttrVal, aIter.toString(), 0)) + if (sax::Converter::convertNumber(nAttrVal, aIter.toView(), 0)) aQRCode.Border = nAttrVal; break; } diff --git a/xmloff/source/draw/XMLNumberStyles.cxx b/xmloff/source/draw/XMLNumberStyles.cxx index db2f93e8b607..3d4407d8a5df 100644 --- a/xmloff/source/draw/XMLNumberStyles.cxx +++ b/xmloff/source/draw/XMLNumberStyles.cxx @@ -531,18 +531,16 @@ SdXMLNumberFormatMemberImportContext::SdXMLNumberFormatMemberImportContext( for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); - switch (aIter.getToken()) { case XML_ELEMENT(NUMBER, XML_DECIMAL_PLACES): - mbDecimal02 = IsXMLToken( sValue, XML_2 ); + mbDecimal02 = IsXMLToken( aIter, XML_2 ); break; case XML_ELEMENT(NUMBER, XML_STYLE): - mbLong = IsXMLToken( sValue, XML_LONG ); + mbLong = IsXMLToken( aIter, XML_LONG ); break; case XML_ELEMENT(NUMBER, XML_TEXTUAL): - mbTextual = IsXMLToken( sValue, XML_TRUE ); + mbTextual = IsXMLToken( aIter, XML_TRUE ); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -591,9 +589,8 @@ SdXMLNumberFormatImportContext::SdXMLNumberFormatImportContext( SdXMLImport& rIm for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); if( aIter.getToken() == XML_ELEMENT(NUMBER, XML_AUTOMATIC_ORDER) ) - mbAutomatic = IsXMLToken( sValue, XML_TRUE ); + mbAutomatic = IsXMLToken( aIter, XML_TRUE ); else XMLOFF_WARN_UNKNOWN("xmloff", aIter); } diff --git a/xmloff/source/draw/animationimport.cxx b/xmloff/source/draw/animationimport.cxx index 6c8672fa38b4..2813160d7691 100644 --- a/xmloff/source/draw/animationimport.cxx +++ b/xmloff/source/draw/animationimport.cxx @@ -122,10 +122,10 @@ AnimationsImportHelperImpl::AnimationsImportHelperImpl( SvXMLImport& rImport ) { } -static bool isDouble( const OUString& rValue ) +static bool isDouble( std::string_view rValue ) { - sal_Int32 nLength = rValue.getLength(); - const sal_Unicode * pStr = rValue.getStr(); + sal_Int32 nLength = rValue.size(); + const char * pStr = rValue.data(); while( nLength ) { if( (*pStr >= '0' && *pStr <= '9') || *pStr == '-' || *pStr == '.' || *pStr == '+' || *pStr == 'e' || *pStr == 'E' ) @@ -557,7 +557,6 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: sal_Int16 nEnum; for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString rValue = aIter.toString(); auto nToken = aIter.getToken(); switch( nToken ) { @@ -565,28 +564,28 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_BEGIN): case XML_ELEMENT(SMIL_SO52, XML_BEGIN): { - mxNode->setBegin( mpHelper->convertTiming( rValue ) ); + mxNode->setBegin( mpHelper->convertTiming( aIter.toString() ) ); } break; case XML_ELEMENT(SMIL, XML_DUR): case XML_ELEMENT(SMIL_COMPAT, XML_DUR): case XML_ELEMENT(SMIL_SO52, XML_DUR): { - mxNode->setDuration( mpHelper->convertTiming( rValue ) ); + mxNode->setDuration( mpHelper->convertTiming( aIter.toString() ) ); } break; case XML_ELEMENT(SMIL, XML_END): case XML_ELEMENT(SMIL_COMPAT, XML_END): case XML_ELEMENT(SMIL_SO52, XML_END): { - mxNode->setEnd( mpHelper->convertTiming( rValue ) ); + mxNode->setEnd( mpHelper->convertTiming( aIter.toString() ) ); } break; case XML_ELEMENT(SMIL, XML_FILL): case XML_ELEMENT(SMIL_COMPAT, XML_FILL): case XML_ELEMENT(SMIL_SO52, XML_FILL): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_Fill ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_Fill ) ) mxNode->setFill( nEnum ); } break; @@ -594,7 +593,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_FILLDEFAULT): case XML_ELEMENT(SMIL_SO52, XML_FILLDEFAULT): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_FillDefault ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_FillDefault ) ) mxNode->setFillDefault( nEnum ); } break; @@ -602,7 +601,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_RESTART): case XML_ELEMENT(SMIL_SO52, XML_RESTART): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_Restart ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_Restart ) ) mxNode->setRestart( nEnum ); } break; @@ -610,7 +609,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_RESTARTDEFAULT): case XML_ELEMENT(SMIL_SO52, XML_RESTARTDEFAULT): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_RestartDefault ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_RestartDefault ) ) mxNode->setRestartDefault( nEnum ); } break; @@ -618,16 +617,16 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_ACCELERATE): case XML_ELEMENT(SMIL_SO52, XML_ACCELERATE): { - if( isDouble( rValue ) ) - mxNode->setAcceleration( rValue.toDouble() ); + if( isDouble( aIter.toView() ) ) + mxNode->setAcceleration( aIter.toDouble() ); } break; case XML_ELEMENT(SMIL, XML_DECELERATE): case XML_ELEMENT(SMIL_COMPAT, XML_DECELERATE): case XML_ELEMENT(SMIL_SO52, XML_DECELERATE): { - if( isDouble( rValue ) ) - mxNode->setDecelerate( rValue.toDouble() ); + if( isDouble( aIter.toView() ) ) + mxNode->setDecelerate( aIter.toDouble() ); } break; case XML_ELEMENT(SMIL, XML_AUTOREVERSE): @@ -635,7 +634,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_SO52, XML_AUTOREVERSE): { bool bTemp; - if (::sax::Converter::convertBool( bTemp, rValue )) + if (::sax::Converter::convertBool( bTemp, aIter.toView() )) mxNode->setAutoReverse( bTemp ); } break; @@ -643,21 +642,21 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_REPEATCOUNT): case XML_ELEMENT(SMIL_SO52, XML_REPEATCOUNT): { - mxNode->setRepeatCount( mpHelper->convertTiming( rValue ) ); + mxNode->setRepeatCount( mpHelper->convertTiming( aIter.toString() ) ); } break; case XML_ELEMENT(SMIL, XML_REPEATDUR): case XML_ELEMENT(SMIL_COMPAT, XML_REPEATDUR): case XML_ELEMENT(SMIL_SO52, XML_REPEATDUR): { - mxNode->setRepeatDuration( mpHelper->convertTiming( rValue ) ); + mxNode->setRepeatDuration( mpHelper->convertTiming( aIter.toString() ) ); } break; case XML_ELEMENT(SMIL, XML_ENDSYNC): case XML_ELEMENT(SMIL_COMPAT, XML_ENDSYNC): case XML_ELEMENT(SMIL_SO52, XML_ENDSYNC): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_Endsync ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_Endsync ) ) mxNode->setEndSync( makeAny( nEnum ) ); } break; @@ -666,7 +665,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_OOO, XML_NODE_TYPE): case XML_ELEMENT(PRESENTATION_OASIS, XML_NODE_TYPE): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_EffectNodeType ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_EffectNodeType ) ) aUserData.emplace_back( GetXMLToken( XML_NODE_TYPE ), makeAny( nEnum ) ); } break; @@ -675,7 +674,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_OOO, XML_PRESET_ID): case XML_ELEMENT(PRESENTATION_OASIS, XML_PRESET_ID): { - aUserData.emplace_back( GetXMLToken( XML_PRESET_ID ), makeAny( rValue ) ); + aUserData.emplace_back( GetXMLToken( XML_PRESET_ID ), makeAny( aIter.toString() ) ); } break; case XML_ELEMENT(PRESENTATION, XML_PRESET_SUB_TYPE): @@ -683,7 +682,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_OOO, XML_PRESET_SUB_TYPE): case XML_ELEMENT(PRESENTATION_OASIS, XML_PRESET_SUB_TYPE): { - aUserData.emplace_back( GetXMLToken( XML_PRESET_SUB_TYPE ), makeAny( rValue ) ); + aUserData.emplace_back( GetXMLToken( XML_PRESET_SUB_TYPE ), makeAny( aIter.toString() ) ); } break; case XML_ELEMENT(PRESENTATION, XML_PRESET_CLASS): @@ -691,7 +690,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_OOO, XML_PRESET_CLASS): case XML_ELEMENT(PRESENTATION_OASIS, XML_PRESET_CLASS): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_EffectPresetClass ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_EffectPresetClass ) ) aUserData.emplace_back( GetXMLToken( XML_PRESET_CLASS ), makeAny( nEnum ) ); } break; @@ -700,7 +699,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_OOO, XML_AFTER_EFFECT): { bool bTemp; - if (::sax::Converter::convertBool( bTemp, rValue )) + if (::sax::Converter::convertBool( bTemp, aIter.toView() )) aUserData.emplace_back( GetXMLToken( XML_AFTER_EFFECT ), makeAny( bTemp ) ); } break; @@ -709,7 +708,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: if( nNodeType == AnimationNodeType::AUDIO ) { Reference< XAudio > xAudio( mxNode, UNO_QUERY_THROW ); - xAudio->setSource( makeAny(lcl_GetMediaReference(GetImport(), rValue)) ); + xAudio->setSource( makeAny(lcl_GetMediaReference(GetImport(), aIter.toString())) ); break; } [[fallthrough]]; @@ -718,7 +717,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_TARGETELEMENT): case XML_ELEMENT(SMIL_SO52, XML_TARGETELEMENT): { - Any aTarget( mpHelper->convertTarget( rValue ) ); + Any aTarget( mpHelper->convertTarget( aIter.toString() ) ); if( xAnimate.is() ) { @@ -740,10 +739,10 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( nNodeType == AnimationNodeType::AUDIO ) { - if( isDouble( rValue ) ) + if( isDouble( aIter.toView() ) ) { Reference< XAudio > xAudio( mxNode, UNO_QUERY_THROW ); - xAudio->setVolume( rValue.toDouble() ); + xAudio->setVolume( aIter.toDouble() ); } } } @@ -753,7 +752,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_SO52, XML_MASTER_ELEMENT): case XML_ELEMENT(PRESENTATION_OOO, XML_MASTER_ELEMENT): { - Reference< XAnimationNode > xMaster( GetImport().getInterfaceToIdentifierMapper().getReference( rValue ), UNO_QUERY ); + Reference< XAnimationNode > xMaster( GetImport().getInterfaceToIdentifierMapper().getReference( aIter.toString() ), UNO_QUERY ); aUserData.emplace_back( GetXMLToken( XML_MASTER_ELEMENT ), makeAny( xMaster ) ); } break; @@ -761,7 +760,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(ANIMATION, XML_SUB_ITEM): case XML_ELEMENT(ANIMATION_OOO, XML_SUB_ITEM): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_SubItem ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_SubItem ) ) { if( xAnimate.is() ) { @@ -781,12 +780,12 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xAnimate.is() ) { - OUString aName( rValue ); + OUString aName( aIter.toString() ); const struct ImplAttributeNameConversion* p = getAnimationAttributeNamesConversionList(); while( p->mpAPIName ) { - if( IsXMLToken( aName, p->meXMLToken ) ) + if( IsXMLToken( aIter, p->meXMLToken ) ) { aName = OUString::createFromAscii( p->mpAPIName ); meAttributeName = p->meXMLToken; @@ -805,7 +804,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_VALUES): case XML_ELEMENT(SMIL_SO52, XML_VALUES): { - aValues = rValue; + aValues = aIter.toString(); } break; @@ -813,7 +812,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_FROM): case XML_ELEMENT(SMIL_SO52, XML_FROM): { - aFrom = rValue; + aFrom = aIter.toString(); } break; @@ -821,7 +820,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_BY): case XML_ELEMENT(SMIL_SO52, XML_BY): { - aBy = rValue; + aBy = aIter.toString(); } break; @@ -829,7 +828,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_COMPAT, XML_TO): case XML_ELEMENT(SMIL_SO52, XML_TO): { - aTo = rValue; + aTo = aIter.toString(); } break; @@ -838,7 +837,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_SO52, XML_KEYTIMES): { if( xAnimate.is() ) - xAnimate->setKeyTimes( AnimationsImportHelperImpl::convertKeyTimes( rValue ) ); + xAnimate->setKeyTimes( AnimationsImportHelperImpl::convertKeyTimes( aIter.toString() ) ); } break; @@ -846,19 +845,19 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(ANIMATION_OOO, XML_FORMULA): { if( xAnimate.is() ) - xAnimate->setFormula( rValue ); + xAnimate->setFormula( aIter.toString() ); } break; case XML_ELEMENT(ANIMATION, XML_ID): case XML_ELEMENT(ANIMATION_OOO, XML_ID): { - if (!bHaveXmlId) { sXmlId = rValue; } + if (!bHaveXmlId) { sXmlId = aIter.toString(); } } break; case XML_ELEMENT(XML, XML_ID): { - sXmlId = rValue; + sXmlId = aIter.toString(); bHaveXmlId = true; } break; @@ -869,7 +868,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xAnimate.is() ) { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_CalcMode ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_CalcMode ) ) xAnimate->setCalcMode( nEnum ); } } @@ -880,7 +879,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_SO52, XML_ACCUMULATE): { if( xAnimate.is() ) - xAnimate->setAccumulate( IsXMLToken( rValue, XML_SUM ) ); + xAnimate->setAccumulate( IsXMLToken( aIter, XML_SUM ) ); } break; @@ -893,7 +892,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xAnimate.is() ) { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_AdditiveMode ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_AdditiveMode ) ) xAnimate->setAdditive( nEnum ); } } @@ -904,7 +903,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_SO52, XML_KEYSPLINES): { if( xAnimate.is() ) - xAnimate->setTimeFilter( AnimationsImportHelperImpl::convertTimeFilter( rValue ) ); + xAnimate->setTimeFilter( AnimationsImportHelperImpl::convertTimeFilter( aIter.toString() ) ); } break; @@ -913,7 +912,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimateMotion > xAnimateMotion( mxNode, UNO_QUERY ); if( xAnimateMotion.is() ) - xAnimateMotion->setPath( AnimationsImportHelperImpl::convertPath( rValue ) ); + xAnimateMotion->setPath( AnimationsImportHelperImpl::convertPath( aIter.toString() ) ); } break; @@ -923,7 +922,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimatePhysics > xAnimatePhysics( mxNode, UNO_QUERY ); if( xAnimatePhysics.is() ) - xAnimatePhysics->setStartVelocityX( makeAny(rValue.toDouble()) ); + xAnimatePhysics->setStartVelocityX( makeAny(aIter.toDouble()) ); } break; @@ -933,7 +932,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimatePhysics > xAnimatePhysics( mxNode, UNO_QUERY ); if( xAnimatePhysics.is() ) - xAnimatePhysics->setStartVelocityY( makeAny(rValue.toDouble()) ); + xAnimatePhysics->setStartVelocityY( makeAny(aIter.toDouble()) ); } break; @@ -943,7 +942,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimatePhysics > xAnimatePhysics( mxNode, UNO_QUERY ); if( xAnimatePhysics.is() ) - xAnimatePhysics->setDensity( makeAny(rValue.toDouble()) ); + xAnimatePhysics->setDensity( makeAny(aIter.toDouble()) ); } break; @@ -953,7 +952,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimatePhysics > xAnimatePhysics( mxNode, UNO_QUERY ); if( xAnimatePhysics.is() ) - xAnimatePhysics->setBounciness( makeAny(rValue.toDouble()) ); + xAnimatePhysics->setBounciness( makeAny(aIter.toDouble()) ); } break; @@ -962,7 +961,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimateColor > xAnimateColor( mxNode, UNO_QUERY ); if( xAnimateColor.is() ) - xAnimateColor->setColorInterpolation( IsXMLToken( rValue, XML_HSL ) ? AnimationColorSpace::HSL : AnimationColorSpace::RGB ); + xAnimateColor->setColorInterpolation( IsXMLToken( aIter, XML_HSL ) ? AnimationColorSpace::HSL : AnimationColorSpace::RGB ); } break; @@ -971,7 +970,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { Reference< XAnimateColor > xAnimateColor( mxNode, UNO_QUERY ); if( xAnimateColor.is() ) - xAnimateColor->setDirection( IsXMLToken( rValue, XML_CLOCKWISE ) ); + xAnimateColor->setDirection( IsXMLToken( aIter, XML_CLOCKWISE ) ); } break; @@ -981,7 +980,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: Reference< XAnimateTransform > xTransform( mxNode, UNO_QUERY ); if( xTransform.is() ) { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_TransformType ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_TransformType ) ) { xTransform->setTransformType( nEnum ); switch( nEnum ) @@ -1005,7 +1004,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xTransitionFilter.is() ) { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_TransitionType ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_TransitionType ) ) xTransitionFilter->setTransition( nEnum ); } } @@ -1017,7 +1016,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xTransitionFilter.is() ) { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_TransitionSubType ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_TransitionSubType ) ) xTransitionFilter->setSubtype( nEnum ); } } @@ -1028,7 +1027,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_SO52, XML_MODE): { if( xTransitionFilter.is() ) - xTransitionFilter->setMode( IsXMLToken( rValue, XML_IN ) ); + xTransitionFilter->setMode( IsXMLToken( aIter, XML_IN ) ); } break; @@ -1037,7 +1036,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(SMIL_SO52, XML_DIRECTION): { if( xTransitionFilter.is() ) - xTransitionFilter->setDirection( IsXMLToken( rValue, XML_FORWARD ) ); + xTransitionFilter->setDirection( IsXMLToken( aIter, XML_FORWARD ) ); } break; @@ -1048,7 +1047,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: if( xTransitionFilter.is() ) { sal_Int32 nColor(0); - ::sax::Converter::convertColor(nColor, rValue); + ::sax::Converter::convertColor(nColor, aIter.toView()); xTransitionFilter->setFadeColor(nColor); } } @@ -1057,7 +1056,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(ANIMATION, XML_ITERATE_TYPE): case XML_ELEMENT(ANIMATION_OOO, XML_ITERATE_TYPE): { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_IterateType ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_IterateType ) ) { if( xIter.is() ) xIter->setIterateType( nEnum ); @@ -1070,6 +1069,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xIter.is() ) { + OUString rValue = aIter.toString(); double fInterval = 0.0; if( rValue.match("P") ) { @@ -1083,7 +1083,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: } else { - fInterval = rValue.toDouble(); + fInterval = aIter.toDouble(); } xIter->setIterateInterval( fInterval ); @@ -1095,7 +1095,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: case XML_ELEMENT(PRESENTATION_SO52, XML_GROUP_ID): case XML_ELEMENT(PRESENTATION_OOO, XML_GROUP_ID): { - aUserData.emplace_back( "group-id", makeAny( rValue.toInt32() ) ); + aUserData.emplace_back( "group-id", makeAny( aIter.toInt32() ) ); } break; @@ -1104,7 +1104,7 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: { if( xCommand.is() && nNodeType == AnimationNodeType::COMMAND ) { - if( SvXMLUnitConverter::convertEnum( nEnum, rValue, aAnimations_EnumMap_Command ) ) + if( SvXMLUnitConverter::convertEnum( nEnum, aIter.toString(), aAnimations_EnumMap_Command ) ) { xCommand->setCommand( nEnum ); } @@ -1120,11 +1120,10 @@ void AnimationNodeContext::init_node( const css::uno::Reference< css::xml::sax: || IsTokenInNamespace(nToken, XML_NAMESPACE_PRESENTATION_OASIS) || IsTokenInNamespace(nToken, XML_NAMESPACE_PRESENTATION_OOO)) { - aUserData.emplace_back( SvXMLImport::getNameFromToken(aIter.getToken()), makeAny( rValue ) ); + aUserData.emplace_back( SvXMLImport::getNameFromToken(aIter.getToken()), makeAny( aIter.toString() ) ); } else - SAL_WARN("xmloff", "unknown token '" + SvXMLImport::getNameFromToken(aIter.getToken()) - << "' 0x" << std::hex << aIter.getToken()); + XMLOFF_WARN_UNKNOWN("xmloff", aIter); } } } diff --git a/xmloff/source/draw/animimp.cxx b/xmloff/source/draw/animimp.cxx index 89cf45a37540..ca6d6cb28450 100644 --- a/xmloff/source/draw/animimp.cxx +++ b/xmloff/source/draw/animimp.cxx @@ -395,14 +395,13 @@ XMLAnimationsSoundContext::XMLAnimationsSoundContext( SvXMLImport& rImport, sal_ for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(XLINK, XML_HREF): - pParent->maSoundURL = rImport.GetAbsoluteReference(sValue); + pParent->maSoundURL = rImport.GetAbsoluteReference(aIter.toString()); break; case XML_ELEMENT(PRESENTATION, XML_PLAY_FULL): - pParent->mbPlayFull = IsXMLToken( sValue, XML_TRUE ); + pParent->mbPlayFull = IsXMLToken( aIter, XML_TRUE ); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -450,34 +449,33 @@ XMLAnimationsEffectContext::XMLAnimationsEffectContext( SvXMLImport& rImport, for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(DRAW, XML_SHAPE_ID): - maShapeId = sValue; + maShapeId = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_COLOR): - ::sax::Converter::convertColor(maDimColor, sValue); + ::sax::Converter::convertColor(maDimColor, aIter.toString()); break; case XML_ELEMENT(PRESENTATION, XML_EFFECT): - SvXMLUnitConverter::convertEnum( meEffect, sValue, aXML_AnimationEffect_EnumMap ); + SvXMLUnitConverter::convertEnum( meEffect, aIter.toString(), aXML_AnimationEffect_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_DIRECTION): - SvXMLUnitConverter::convertEnum( meDirection, sValue, aXML_AnimationDirection_EnumMap ); + SvXMLUnitConverter::convertEnum( meDirection, aIter.toString(), aXML_AnimationDirection_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_START_SCALE): { sal_Int32 nScale; - if (::sax::Converter::convertPercent( nScale, sValue )) + if (::sax::Converter::convertPercent( nScale, aIter.toView() )) mnStartScale = static_cast(nScale); break; } case XML_ELEMENT(PRESENTATION, XML_SPEED): - SvXMLUnitConverter::convertEnum( meSpeed, sValue, aXML_AnimationSpeed_EnumMap ); + SvXMLUnitConverter::convertEnum( meSpeed, aIter.toString(), aXML_AnimationSpeed_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_PATH_ID): - maPathShapeId = sValue; + maPathShapeId = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); diff --git a/xmloff/source/draw/eventimp.cxx b/xmloff/source/draw/eventimp.cxx index c71c71043cc2..2f90ed9e5e6c 100644 --- a/xmloff/source/draw/eventimp.cxx +++ b/xmloff/source/draw/eventimp.cxx @@ -106,14 +106,13 @@ XMLEventSoundContext::XMLEventSoundContext( SvXMLImport& rImp, const Reference< { for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(XLINK, XML_HREF): - pParent->maData.msSoundURL = rImp.GetAbsoluteReference(sValue); + pParent->maData.msSoundURL = rImp.GetAbsoluteReference(aIter.toString()); break; case XML_ELEMENT(PRESENTATION, XML_PLAY_FULL): - pParent->maData.mbPlayFull = IsXMLToken( sValue, XML_TRUE ); + pParent->maData.mbPlayFull = IsXMLToken( aIter, XML_TRUE ); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -145,36 +144,35 @@ SdXMLEventContext::SdXMLEventContext( SvXMLImport& rImp, OUString sEventName; for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(PRESENTATION, XML_ACTION): - SvXMLUnitConverter::convertEnum( maData.meClickAction, sValue, aXML_EventActions_EnumMap ); + SvXMLUnitConverter::convertEnum( maData.meClickAction, aIter.toString(), aXML_EventActions_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_EFFECT): - SvXMLUnitConverter::convertEnum( maData.meEffect, sValue, aXML_AnimationEffect_EnumMap ); + SvXMLUnitConverter::convertEnum( maData.meEffect, aIter.toString(), aXML_AnimationEffect_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_DIRECTION): - SvXMLUnitConverter::convertEnum( maData.meDirection, sValue, aXML_AnimationDirection_EnumMap ); + SvXMLUnitConverter::convertEnum( maData.meDirection, aIter.toString(), aXML_AnimationDirection_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_START_SCALE): { sal_Int32 nScale; - if (::sax::Converter::convertPercent( nScale, sValue )) + if (::sax::Converter::convertPercent( nScale, aIter.toView() )) maData.mnStartScale = static_cast(nScale); } break; case XML_ELEMENT(PRESENTATION, XML_SPEED): - SvXMLUnitConverter::convertEnum( maData.meSpeed, sValue, aXML_AnimationSpeed_EnumMap ); + SvXMLUnitConverter::convertEnum( maData.meSpeed, aIter.toString(), aXML_AnimationSpeed_EnumMap ); break; case XML_ELEMENT(PRESENTATION, XML_VERB): - ::sax::Converter::convertNumber( maData.mnVerb, sValue ); + ::sax::Converter::convertNumber( maData.mnVerb, aIter.toView() ); break; case XML_ELEMENT(SCRIPT, XML_EVENT_NAME): { - sEventName = sValue; + sEventName = aIter.toString(); sal_uInt16 nScriptPrefix = - GetImport().GetNamespaceMap().GetKeyByAttrValueQName(sValue, &sEventName); + GetImport().GetNamespaceMap().GetKeyByAttrValueQName(sEventName, &sEventName); maData.mbValid = XML_NAMESPACE_DOM == nScriptPrefix && sEventName == "click"; } break; @@ -182,7 +180,7 @@ SdXMLEventContext::SdXMLEventContext( SvXMLImport& rImp, { // language is not evaluated! OUString aScriptLanguage; - maData.msLanguage = sValue; + maData.msLanguage = aIter.toString(); sal_uInt16 nScriptPrefix = rImp.GetNamespaceMap(). GetKeyByAttrValueQName(maData.msLanguage, &aScriptLanguage); if( XML_NAMESPACE_OOO == nScriptPrefix ) @@ -190,18 +188,18 @@ SdXMLEventContext::SdXMLEventContext( SvXMLImport& rImp, } break; case XML_ELEMENT(SCRIPT, XML_MACRO_NAME): - maData.msMacroName = sValue; + maData.msMacroName = aIter.toString(); break; case XML_ELEMENT(XLINK, XML_HREF): { if ( maData.mbScript ) { - maData.msMacroName = sValue; + maData.msMacroName = aIter.toString(); } else { const OUString &rTmp = - rImp.GetAbsoluteReference(sValue); + rImp.GetAbsoluteReference(aIter.toString()); INetURLObject::translateToInternal( rTmp, maData.msBookmark, INetURLObject::DecodeMechanism::Unambiguous ); } diff --git a/xmloff/source/draw/ximp3dobject.cxx b/xmloff/source/draw/ximp3dobject.cxx index abe34189c90d..c17defc2426b 100644 --- a/xmloff/source/draw/ximp3dobject.cxx +++ b/xmloff/source/draw/ximp3dobject.cxx @@ -97,14 +97,12 @@ SdXML3DCubeObjectShapeContext::SdXML3DCubeObjectShapeContext( { for(auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - OUString sValue = aIter.toString(); - switch(aIter.getToken()) { case XML_ELEMENT(DR3D, XML_MIN_EDGE): { ::basegfx::B3DVector aNewVec; - SvXMLUnitConverter::convertB3DVector(aNewVec, sValue); + SvXMLUnitConverter::convertB3DVector(aNewVec, aIter.toView()); if(aNewVec != maMinEdge) maMinEdge = aNewVec; @@ -113,7 +111,7 @@ SdXML3DCubeObjectShapeContext::SdXML3DCubeObjectShapeContext( case XML_ELEMENT(DR3D, XML_MAX_EDGE): { ::basegfx::B3DVector aNewVec; - SvXMLUnitConverter::convertB3DVector(aNewVec, sValue); + SvXMLUnitConverter::convertB3DVector(aNewVec, aIter.toView()); if(aNewVec != maMaxEdge) maMaxEdge = aNewVec; @@ -176,14 +174,12 @@ SdXML3DSphereObjectShapeContext::SdXML3DSphereObjectShapeContext( { for(auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - OUString sValue = aIter.toString(); - switch(aIter.getToken()) { case XML_ELEMENT(DR3D, XML_CENTER): { ::basegfx::B3DVector aNewVec; - SvXMLUnitConverter::convertB3DVector(aNewVec, sValue); + SvXMLUnitConverter::convertB3DVector(aNewVec, aIter.toView()); if(aNewVec != maCenter) maCenter = aNewVec; @@ -192,7 +188,7 @@ SdXML3DSphereObjectShapeContext::SdXML3DSphereObjectShapeContext( case XML_ELEMENT(DR3D, XML_SIZE): { ::basegfx::B3DVector aNewVec; - SvXMLUnitConverter::convertB3DVector(aNewVec, sValue); + SvXMLUnitConverter::convertB3DVector(aNewVec, aIter.toView()); if(aNewVec != maSphereSize) maSphereSize = aNewVec; diff --git a/xmloff/source/draw/ximp3dscene.cxx b/xmloff/source/draw/ximp3dscene.cxx index 9477481343e8..29a13ba961b3 100644 --- a/xmloff/source/draw/ximp3dscene.cxx +++ b/xmloff/source/draw/ximp3dscene.cxx @@ -49,36 +49,35 @@ SdXML3DLightContext::SdXML3DLightContext( // read attributes for the 3DScene for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(DR3D, XML_DIFFUSE_COLOR): { - ::sax::Converter::convertColor(maDiffuseColor, sValue); + ::sax::Converter::convertColor(maDiffuseColor, aIter.toString()); break; } case XML_ELEMENT(DR3D, XML_DIRECTION): { ::basegfx::B3DVector aVal; - SvXMLUnitConverter::convertB3DVector(aVal, sValue); + SvXMLUnitConverter::convertB3DVector(aVal, aIter.toString()); if (!std::isnan(aVal.getX()) && !std::isnan(aVal.getY()) && !std::isnan(aVal.getZ())) { maDirection = aVal; } else { - SAL_WARN("xmloff", "NaNs found in light direction: " << sValue); + SAL_WARN("xmloff", "NaNs found in light direction: " << aIter.toString()); } break; } case XML_ELEMENT(DR3D, XML_ENABLED): { - (void)::sax::Converter::convertBool(mbEnabled, sValue); + (void)::sax::Converter::convertBool(mbEnabled, aIter.toView()); break; } case XML_ELEMENT(DR3D, XML_SPECULAR): { - (void)::sax::Converter::convertBool(mbSpecular, sValue); + (void)::sax::Converter::convertBool(mbSpecular, aIter.toView()); break; } default: diff --git a/xmloff/source/draw/ximpcustomshape.cxx b/xmloff/source/draw/ximpcustomshape.cxx index 8e23b0ee844f..f53c513d7e26 100644 --- a/xmloff/source/draw/ximpcustomshape.cxx +++ b/xmloff/source/draw/ximpcustomshape.cxx @@ -72,7 +72,7 @@ const SvXMLEnumMapEntry aXML_GluePointEnumMap[] = { XML_TOKEN_INVALID, 0 } }; static void GetBool( std::vector< css::beans::PropertyValue >& rDest, - std::u16string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) + std::string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) { bool bAttrBool; if (::sax::Converter::convertBool( bAttrBool, rValue )) @@ -85,7 +85,7 @@ static void GetBool( std::vector< css::beans::PropertyValue >& rDest, } static void GetInt32( std::vector< css::beans::PropertyValue >& rDest, - std::u16string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) + std::string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) { sal_Int32 nAttrNumber; if (::sax::Converter::convertNumber( nAttrNumber, rValue )) @@ -98,7 +98,7 @@ static void GetInt32( std::vector< css::beans::PropertyValue >& rDest, } static void GetDouble( std::vector< css::beans::PropertyValue >& rDest, - std::u16string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) + std::string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) { double fAttrDouble; if (::sax::Converter::convertDouble( fAttrDouble, rValue )) @@ -135,7 +135,7 @@ static void GetEnum( std::vector< css::beans::PropertyValue >& rDest, } static void GetDoublePercentage( std::vector< css::beans::PropertyValue >& rDest, - const OUString& rValue, const EnhancedCustomShapeTokenEnum eDestProp ) + std::string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) { sal_Int16 const eSrcUnit = ::sax::Converter::GetUnitFromString( rValue, util::MeasureUnit::MM_100TH); @@ -155,7 +155,7 @@ static void GetDoublePercentage( std::vector< css::beans::PropertyValue >& rDest } static void GetB3DVector( std::vector< css::beans::PropertyValue >& rDest, - const OUString& rValue, const EnhancedCustomShapeTokenEnum eDestProp ) + std::string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp ) { ::basegfx::B3DVector aB3DVector; if ( SvXMLUnitConverter::convertB3DVector( aB3DVector, rValue ) ) @@ -424,7 +424,7 @@ static bool GetNextParameter( css::drawing::EnhancedCustomShapeParameter& rParam } static void GetPosition3D( std::vector< css::beans::PropertyValue >& rDest, // e.g. draw:extrusion-viewpoint - const OUString& rValue, const EnhancedCustomShapeTokenEnum eDestProp, + std::string_view rValue, const EnhancedCustomShapeTokenEnum eDestProp, SvXMLUnitConverter& rUnitConverter ) { drawing::Position3D aPosition3D; @@ -866,22 +866,20 @@ void XMLEnhancedCustomShapeContext::startFastElement( sal_Int32 nAttrNumber; for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString rValue = aIter.toString(); - switch( EASGet( aIter.getToken() ) ) { case EAS_type : - GetString( mrCustomShapeGeometry, rValue, EAS_Type ); + GetString( mrCustomShapeGeometry, aIter.toString(), EAS_Type ); break; case EAS_mirror_horizontal : - GetBool( mrCustomShapeGeometry, rValue, EAS_MirroredX ); + GetBool( mrCustomShapeGeometry, aIter.toView(), EAS_MirroredX ); break; case EAS_mirror_vertical : - GetBool( mrCustomShapeGeometry, rValue, EAS_MirroredY ); + GetBool( mrCustomShapeGeometry, aIter.toView(), EAS_MirroredY ); break; case EAS_viewBox : { - SdXMLImExViewBox aViewBox( rValue, GetImport().GetMM100UnitConverter() ); + SdXMLImExViewBox aViewBox( aIter.toString(), GetImport().GetMM100UnitConverter() ); awt::Rectangle aRect( aViewBox.GetX(), aViewBox.GetY(), aViewBox.GetWidth(), aViewBox.GetHeight() ); beans::PropertyValue aProp; aProp.Name = EASGet( EAS_ViewBox ); @@ -890,28 +888,29 @@ void XMLEnhancedCustomShapeContext::startFastElement( } break; case EAS_sub_view_size: - GetSizeSequence( maPath, rValue, EAS_SubViewSize ); + GetSizeSequence( maPath, aIter.toString(), EAS_SubViewSize ); break; case EAS_text_rotate_angle : - GetDouble( mrCustomShapeGeometry, rValue, EAS_TextRotateAngle ); + GetDouble( mrCustomShapeGeometry, aIter.toView(), EAS_TextRotateAngle ); break; case EAS_extrusion_allowed : - GetBool( maPath, rValue, EAS_ExtrusionAllowed ); + GetBool( maPath, aIter.toView(), EAS_ExtrusionAllowed ); break; case EAS_text_path_allowed : - GetBool( maPath, rValue, EAS_TextPathAllowed ); + GetBool( maPath, aIter.toView(), EAS_TextPathAllowed ); break; case EAS_concentric_gradient_fill_allowed : - GetBool( maPath, rValue, EAS_ConcentricGradientFillAllowed ); + GetBool( maPath, aIter.toView(), EAS_ConcentricGradientFillAllowed ); break; case EAS_extrusion : - GetBool( maExtrusion, rValue, EAS_Extrusion ); + GetBool( maExtrusion, aIter.toView(), EAS_Extrusion ); break; case EAS_extrusion_brightness : - GetDoublePercentage( maExtrusion, rValue, EAS_Brightness ); + GetDoublePercentage( maExtrusion, aIter.toView(), EAS_Brightness ); break; case EAS_extrusion_depth : { + OUString rValue = aIter.toString(); sal_Int32 nIndex = 0; css::drawing::EnhancedCustomShapeParameterPair aParameterPair; css::drawing::EnhancedCustomShapeParameter& rDepth = aParameterPair.First; @@ -953,43 +952,43 @@ void XMLEnhancedCustomShapeContext::startFastElement( } break; case EAS_extrusion_diffusion : - GetDoublePercentage( maExtrusion, rValue, EAS_Diffusion ); + GetDoublePercentage( maExtrusion, aIter.toView(), EAS_Diffusion ); break; case EAS_extrusion_number_of_line_segments : - GetInt32( maExtrusion, rValue, EAS_NumberOfLineSegments ); + GetInt32( maExtrusion, aIter.toView(), EAS_NumberOfLineSegments ); break; case EAS_extrusion_light_face : - GetBool( maExtrusion, rValue, EAS_LightFace ); + GetBool( maExtrusion, aIter.toView(), EAS_LightFace ); break; case EAS_extrusion_first_light_harsh : - GetBool( maExtrusion, rValue, EAS_FirstLightHarsh ); + GetBool( maExtrusion, aIter.toView(), EAS_FirstLightHarsh ); break; case EAS_extrusion_second_light_harsh : - GetBool( maExtrusion, rValue, EAS_SecondLightHarsh ); + GetBool( maExtrusion, aIter.toView(), EAS_SecondLightHarsh ); break; case EAS_extrusion_first_light_level : - GetDoublePercentage( maExtrusion, rValue, EAS_FirstLightLevel ); + GetDoublePercentage( maExtrusion, aIter.toView(), EAS_FirstLightLevel ); break; case EAS_extrusion_second_light_level : - GetDoublePercentage( maExtrusion, rValue, EAS_SecondLightLevel ); + GetDoublePercentage( maExtrusion, aIter.toView(), EAS_SecondLightLevel ); break; case EAS_extrusion_first_light_direction : - GetB3DVector( maExtrusion, rValue, EAS_FirstLightDirection ); + GetB3DVector( maExtrusion, aIter.toView(), EAS_FirstLightDirection ); break; case EAS_extrusion_second_light_direction : - GetB3DVector( maExtrusion, rValue, EAS_SecondLightDirection ); + GetB3DVector( maExtrusion, aIter.toView(), EAS_SecondLightDirection ); break; case EAS_extrusion_metal : - GetBool( maExtrusion, rValue, EAS_Metal ); + GetBool( maExtrusion, aIter.toView(), EAS_Metal ); break; case EAS_shade_mode : { drawing::ShadeMode eShadeMode( drawing::ShadeMode_FLAT ); - if( IsXMLToken( rValue, XML_PHONG ) ) + if( IsXMLToken( aIter, XML_PHONG ) ) eShadeMode = drawing::ShadeMode_PHONG; - else if ( IsXMLToken( rValue, XML_GOURAUD ) ) + else if ( IsXMLToken( aIter, XML_GOURAUD ) ) eShadeMode = drawing::ShadeMode_SMOOTH; - else if ( IsXMLToken( rValue, XML_DRAFT ) ) + else if ( IsXMLToken( aIter, XML_DRAFT ) ) eShadeMode = drawing::ShadeMode_DRAFT; beans::PropertyValue aProp; @@ -999,24 +998,24 @@ void XMLEnhancedCustomShapeContext::startFastElement( } break; case EAS_extrusion_rotation_angle : - GetEnhancedParameterPair( maExtrusion, rValue, EAS_RotateAngle ); + GetEnhancedParameterPair( maExtrusion, aIter.toString(), EAS_RotateAngle ); break; case EAS_extrusion_rotation_center : - GetB3DVector( maExtrusion, rValue, EAS_RotationCenter ); + GetB3DVector( maExtrusion, aIter.toView(), EAS_RotationCenter ); break; case EAS_extrusion_shininess : - GetDoublePercentage( maExtrusion, rValue, EAS_Shininess ); + GetDoublePercentage( maExtrusion, aIter.toView(), EAS_Shininess ); break; case EAS_extrusion_skew : - GetEnhancedParameterPair( maExtrusion, rValue, EAS_Skew ); + GetEnhancedParameterPair( maExtrusion, aIter.toString(), EAS_Skew ); break; case EAS_extrusion_specularity : - GetDoublePercentage( maExtrusion, rValue, EAS_Specularity ); + GetDoublePercentage( maExtrusion, aIter.toView(), EAS_Specularity ); break; case EAS_projection : { drawing::ProjectionMode eProjectionMode( drawing::ProjectionMode_PERSPECTIVE ); - if( IsXMLToken( rValue, XML_PARALLEL ) ) + if( IsXMLToken( aIter, XML_PARALLEL ) ) eProjectionMode = drawing::ProjectionMode_PARALLEL; beans::PropertyValue aProp; @@ -1026,20 +1025,20 @@ void XMLEnhancedCustomShapeContext::startFastElement( } break; case EAS_extrusion_viewpoint : - GetPosition3D( maExtrusion, rValue, EAS_ViewPoint, mrUnitConverter ); + GetPosition3D( maExtrusion, aIter.toView(), EAS_ViewPoint, mrUnitConverter ); break; case EAS_extrusion_origin : - GetEnhancedParameterPair( maExtrusion, rValue, EAS_Origin ); + GetEnhancedParameterPair( maExtrusion, aIter.toString(), EAS_Origin ); break; case EAS_extrusion_color : - GetBool( maExtrusion, rValue, EAS_Color ); + GetBool( maExtrusion, aIter.toView(), EAS_Color ); break; case EAS_enhanced_path : - GetEnhancedPath( maPath, rValue ); + GetEnhancedPath( maPath, aIter.toString() ); break; case EAS_path_stretchpoint_x : { - if (::sax::Converter::convertNumber(nAttrNumber, rValue)) + if (::sax::Converter::convertNumber(nAttrNumber, aIter.toView())) { beans::PropertyValue aProp; aProp.Name = EASGet( EAS_StretchX ); @@ -1050,7 +1049,7 @@ void XMLEnhancedCustomShapeContext::startFastElement( break; case EAS_path_stretchpoint_y : { - if (::sax::Converter::convertNumber(nAttrNumber, rValue)) + if (::sax::Converter::convertNumber(nAttrNumber, aIter.toView())) { beans::PropertyValue aProp; aProp.Name = EASGet( EAS_StretchY ); @@ -1060,31 +1059,31 @@ void XMLEnhancedCustomShapeContext::startFastElement( } break; case EAS_text_areas : - GetEnhancedRectangleSequence( maPath, rValue, EAS_TextFrames ); + GetEnhancedRectangleSequence( maPath, aIter.toString(), EAS_TextFrames ); break; case EAS_glue_points : { - sal_Int32 i, nPairs = GetEnhancedParameterPairSequence( maPath, rValue, EAS_GluePoints ); + sal_Int32 i, nPairs = GetEnhancedParameterPairSequence( maPath, aIter.toString(), EAS_GluePoints ); GetImport().GetShapeImport()->moveGluePointMapping( mrxShape, nPairs ); for ( i = 0; i < nPairs; i++ ) GetImport().GetShapeImport()->addGluePointMapping( mrxShape, i + 4, i + 4 ); } break; case EAS_glue_point_type : - GetEnum( maPath, rValue, EAS_GluePointType, *aXML_GluePointEnumMap ); + GetEnum( maPath, aIter.toString(), EAS_GluePointType, *aXML_GluePointEnumMap ); break; case EAS_glue_point_leaving_directions : - GetDoubleSequence( maPath, rValue, EAS_GluePointLeavingDirections ); + GetDoubleSequence( maPath, aIter.toString(), EAS_GluePointLeavingDirections ); break; case EAS_text_path : - GetBool( maTextPath, rValue, EAS_TextPath ); + GetBool( maTextPath, aIter.toView(), EAS_TextPath ); break; case EAS_text_path_mode : { css::drawing::EnhancedCustomShapeTextPathMode eTextPathMode( css::drawing::EnhancedCustomShapeTextPathMode_NORMAL ); - if( IsXMLToken( rValue, XML_PATH ) ) + if( IsXMLToken( aIter, XML_PATH ) ) eTextPathMode = css::drawing::EnhancedCustomShapeTextPathMode_PATH; - else if ( IsXMLToken( rValue, XML_SHAPE ) ) + else if ( IsXMLToken( aIter, XML_SHAPE ) ) eTextPathMode = css::drawing::EnhancedCustomShapeTextPathMode_SHAPE; beans::PropertyValue aProp; @@ -1095,7 +1094,7 @@ void XMLEnhancedCustomShapeContext::startFastElement( break; case EAS_text_path_scale : { - bool bScaleX = IsXMLToken( rValue, XML_SHAPE ); + bool bScaleX = IsXMLToken( aIter, XML_SHAPE ); beans::PropertyValue aProp; aProp.Name = EASGet( EAS_ScaleX ); aProp.Value <<= bScaleX; @@ -1103,10 +1102,10 @@ void XMLEnhancedCustomShapeContext::startFastElement( } break; case EAS_text_path_same_letter_heights : - GetBool( maTextPath, rValue, EAS_SameLetterHeights ); + GetBool( maTextPath, aIter.toView(), EAS_SameLetterHeights ); break; case EAS_modifiers : - GetAdjustmentValues( mrCustomShapeGeometry, rValue ); + GetAdjustmentValues( mrCustomShapeGeometry, aIter.toString() ); break; default: break; @@ -1329,41 +1328,40 @@ css::uno::Reference< css::xml::sax::XFastContextHandler > XMLEnhancedCustomShape std::vector< css::beans::PropertyValue > aHandle; for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( EASGet( aIter.getToken() ) ) { case EAS_handle_mirror_vertical : - GetBool( aHandle, sValue, EAS_MirroredY ); + GetBool( aHandle, aIter.toView(), EAS_MirroredY ); break; case EAS_handle_mirror_horizontal : - GetBool( aHandle, sValue, EAS_MirroredX ); + GetBool( aHandle, aIter.toView(), EAS_MirroredX ); break; case EAS_handle_switched : - GetBool( aHandle, sValue, EAS_Switched ); + GetBool( aHandle, aIter.toView(), EAS_Switched ); break; case EAS_handle_position : - GetEnhancedParameterPair( aHandle, sValue, EAS_Position ); + GetEnhancedParameterPair( aHandle, aIter.toString(), EAS_Position ); break; case EAS_handle_range_x_minimum : - GetEnhancedParameter( aHandle, sValue, EAS_RangeXMinimum ); + GetEnhancedParameter( aHandle, aIter.toString(), EAS_RangeXMinimum ); break; case EAS_handle_range_x_maximum : - GetEnhancedParameter( aHandle, sValue, EAS_RangeXMaximum ); + GetEnhancedParameter( aHandle, aIter.toString(), EAS_RangeXMaximum ); break; case EAS_handle_range_y_minimum : - GetEnhancedParameter( aHandle, sValue, EAS_RangeYMinimum ); + GetEnhancedParameter( aHandle, aIter.toString(), EAS_RangeYMinimum ); break; case EAS_handle_range_y_maximum : - GetEnhancedParameter( aHandle, sValue, EAS_RangeYMaximum ); + GetEnhancedParameter( aHandle, aIter.toString(), EAS_RangeYMaximum ); break; case EAS_handle_polar : - GetEnhancedParameterPair( aHandle, sValue, EAS_Polar ); + GetEnhancedParameterPair( aHandle, aIter.toString(), EAS_Polar ); break; case EAS_handle_radius_range_minimum : - GetEnhancedParameter( aHandle, sValue, EAS_RadiusRangeMinimum ); + GetEnhancedParameter( aHandle, aIter.toString(), EAS_RadiusRangeMinimum ); break; case EAS_handle_radius_range_maximum : - GetEnhancedParameter( aHandle, sValue, EAS_RadiusRangeMaximum ); + GetEnhancedParameter( aHandle, aIter.toString(), EAS_RadiusRangeMaximum ); break; default: break; diff --git a/xmloff/source/draw/ximppage.cxx b/xmloff/source/draw/ximppage.cxx index b9492b96a776..2ddd5140545f 100644 --- a/xmloff/source/draw/ximppage.cxx +++ b/xmloff/source/draw/ximppage.cxx @@ -92,8 +92,6 @@ DrawAnnotationContext::DrawAnnotationContext( SvXMLImport& rImport, const Refere for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); - switch( aIter.getToken() ) { case XML_ELEMENT(SVG, XML_X): @@ -101,7 +99,7 @@ DrawAnnotationContext::DrawAnnotationContext( SvXMLImport& rImport, const Refere { sal_Int32 x; GetImport().GetMM100UnitConverter().convertMeasureToCore( - x, sValue); + x, aIter.toView()); aPosition.X = static_cast(x) / 100.0; break; } @@ -110,7 +108,7 @@ DrawAnnotationContext::DrawAnnotationContext( SvXMLImport& rImport, const Refere { sal_Int32 y; GetImport().GetMM100UnitConverter().convertMeasureToCore( - y, sValue); + y, aIter.toView()); aPosition.Y = static_cast(y) / 100.0; break; } @@ -119,7 +117,7 @@ DrawAnnotationContext::DrawAnnotationContext( SvXMLImport& rImport, const Refere { sal_Int32 w; GetImport().GetMM100UnitConverter().convertMeasureToCore( - w, sValue); + w, aIter.toView()); aSize.Width = static_cast(w) / 100.0; break; } @@ -128,7 +126,7 @@ DrawAnnotationContext::DrawAnnotationContext( SvXMLImport& rImport, const Refere { sal_Int32 h; GetImport().GetMM100UnitConverter().convertMeasureToCore( - h, sValue); + h, aIter.toView()); aSize.Height = static_cast(h) / 100.0; } break; diff --git a/xmloff/source/draw/ximpshap.cxx b/xmloff/source/draw/ximpshap.cxx index 5ae4680332ae..cabf4aa460a1 100644 --- a/xmloff/source/draw/ximpshap.cxx +++ b/xmloff/source/draw/ximpshap.cxx @@ -3420,7 +3420,7 @@ void SdXMLFrameShapeContext::endFastElement(sal_Int32 nElement) switch (aIter.getToken()) { case XML_ELEMENT(PRESENTATION, XML_PLACEHOLDER): - mbIsPlaceholder = IsXMLToken( aIter.toString(), XML_TRUE ); + mbIsPlaceholder = IsXMLToken( aIter, XML_TRUE ); break; case XML_ELEMENT(PRESENTATION, XML_CLASS): maPresentationClass = aIter.toString(); diff --git a/xmloff/source/draw/ximpshow.cxx b/xmloff/source/draw/ximpshow.cxx index 0e227d9cfa64..4fba11fd380a 100644 --- a/xmloff/source/draw/ximpshow.cxx +++ b/xmloff/source/draw/ximpshow.cxx @@ -77,26 +77,24 @@ SdXMLShowsContext::SdXMLShowsContext( SdXMLImport& rImport, const Reference< XFa // read attributes for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); - switch( aIter.getToken() ) { case XML_ELEMENT(PRESENTATION, XML_START_PAGE): { - mxPresProps->setPropertyValue("FirstPage", Any(sValue) ); + mxPresProps->setPropertyValue("FirstPage", Any(aIter.toString()) ); bAll = false; break; } case XML_ELEMENT(PRESENTATION, XML_SHOW): { - maCustomShowName = sValue; + maCustomShowName = aIter.toString(); bAll = false; break; } case XML_ELEMENT(PRESENTATION, XML_PAUSE): { Duration aDuration; - if (!::sax::Converter::convertDuration(aDuration, sValue)) + if (!::sax::Converter::convertDuration(aDuration, aIter.toString())) continue; const sal_Int32 nMS = (aDuration.Hours * 60 + @@ -106,60 +104,60 @@ SdXMLShowsContext::SdXMLShowsContext( SdXMLImport& rImport, const Reference< XFa } case XML_ELEMENT(PRESENTATION, XML_ANIMATIONS): { - aAny <<= IsXMLToken( sValue, XML_ENABLED ); + aAny <<= IsXMLToken( aIter, XML_ENABLED ); mxPresProps->setPropertyValue("AllowAnimations", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_STAY_ON_TOP): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("IsAlwaysOnTop", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_FORCE_MANUAL): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("IsAutomatic", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_ENDLESS): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("IsEndless", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_FULL_SCREEN): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("IsFullScreen", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_MOUSE_VISIBLE): { - bIsMouseVisible = IsXMLToken( sValue, XML_TRUE ); + bIsMouseVisible = IsXMLToken( aIter, XML_TRUE ); break; } case XML_ELEMENT(PRESENTATION, XML_START_WITH_NAVIGATOR): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("StartWithNavigator", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_MOUSE_AS_PEN): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("UsePen", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_TRANSITION_ON_CLICK): { - aAny <<= IsXMLToken( sValue, XML_ENABLED ); + aAny <<= IsXMLToken( aIter, XML_ENABLED ); mxPresProps->setPropertyValue("IsTransitionOnClick", aAny ); break; } case XML_ELEMENT(PRESENTATION, XML_SHOW_LOGO): { - aAny <<= IsXMLToken( sValue, XML_TRUE ); + aAny <<= IsXMLToken( aIter, XML_TRUE ); mxPresProps->setPropertyValue("IsShowLogo", aAny ); break; } diff --git a/xmloff/source/draw/ximpstyl.cxx b/xmloff/source/draw/ximpstyl.cxx index ee2c51add4bb..ac23bc3345a2 100644 --- a/xmloff/source/draw/ximpstyl.cxx +++ b/xmloff/source/draw/ximpstyl.cxx @@ -294,54 +294,53 @@ SdXMLPageMasterStyleContext::SdXMLPageMasterStyleContext( for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(FO, XML_MARGIN_TOP): case XML_ELEMENT(FO_COMPAT, XML_MARGIN_TOP): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnBorderTop, sValue); + mnBorderTop, aIter.toView()); break; } case XML_ELEMENT(FO, XML_MARGIN_BOTTOM): case XML_ELEMENT(FO_COMPAT, XML_MARGIN_BOTTOM): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnBorderBottom, sValue); + mnBorderBottom, aIter.toView()); break; } case XML_ELEMENT(FO, XML_MARGIN_LEFT): case XML_ELEMENT(FO_COMPAT, XML_MARGIN_LEFT): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnBorderLeft, sValue); + mnBorderLeft, aIter.toView()); break; } case XML_ELEMENT(FO, XML_MARGIN_RIGHT): case XML_ELEMENT(FO_COMPAT, XML_MARGIN_RIGHT): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnBorderRight, sValue); + mnBorderRight, aIter.toView()); break; } case XML_ELEMENT(FO, XML_PAGE_WIDTH): case XML_ELEMENT(FO_COMPAT, XML_PAGE_WIDTH): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnWidth, sValue); + mnWidth, aIter.toView()); break; } case XML_ELEMENT(FO, XML_PAGE_HEIGHT): case XML_ELEMENT(FO_COMPAT, XML_PAGE_HEIGHT): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnHeight, sValue); + mnHeight, aIter.toView()); break; } case XML_ELEMENT(STYLE, XML_PRINT_ORIENTATION): { - if( IsXMLToken( sValue, XML_PORTRAIT ) ) + if( IsXMLToken( aIter, XML_PORTRAIT ) ) meOrientation = view::PaperOrientation_PORTRAIT; else meOrientation = view::PaperOrientation_LANDSCAPE; @@ -633,19 +632,18 @@ SdXMLPresentationPlaceholderContext::SdXMLPresentationPlaceholderContext( { for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(PRESENTATION, XML_OBJECT): { - msName = sValue; + msName = aIter.toString(); break; } case XML_ELEMENT(SVG, XML_X): case XML_ELEMENT(SVG_COMPAT, XML_X): { GetSdImport().GetMM100UnitConverter().convertMeasureToCore( - mnX, sValue); + mnX, aIter.toView()); break; } case XML_ELEMENT(SVG, XML_Y): @@ -1373,18 +1371,17 @@ SdXMLHeaderFooterDeclContext::SdXMLHeaderFooterDeclContext(SvXMLImport& rImport, { for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - OUString sValue = aIter.toString(); if( aIter.getToken() == XML_ELEMENT(PRESENTATION, XML_NAME) ) { - maStrName = sValue; + maStrName = aIter.toString(); } else if( aIter.getToken() == XML_ELEMENT(PRESENTATION, XML_SOURCE) ) { - mbFixed = IsXMLToken( sValue, XML_FIXED ); + mbFixed = IsXMLToken( aIter, XML_FIXED ); } else if( aIter.getToken() == XML_ELEMENT(STYLE, XML_DATA_STYLE_NAME) ) { - maStrDateTimeFormat = sValue; + maStrDateTimeFormat = aIter.toString(); } else { diff --git a/xmloff/source/style/DashStyle.cxx b/xmloff/source/style/DashStyle.cxx index 18462d82b4c4..68aeb7770966 100644 --- a/xmloff/source/style/DashStyle.cxx +++ b/xmloff/source/style/DashStyle.cxx @@ -78,66 +78,64 @@ void XMLDashStyleImport::importXML( for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - const OUString aStrValue = aIter.toString(); - switch( aIter.getToken() ) { case XML_ELEMENT(DRAW, XML_NAME): case XML_ELEMENT(DRAW_OOO, XML_NAME): { - rStrName = aStrValue; + rStrName = aIter.toString(); } break; case XML_ELEMENT(DRAW, XML_DISPLAY_NAME): case XML_ELEMENT(DRAW_OOO, XML_DISPLAY_NAME): { - aDisplayName = aStrValue; + aDisplayName = aIter.toString(); } break; case XML_ELEMENT(DRAW, XML_STYLE): case XML_ELEMENT(DRAW_OOO, XML_STYLE): { - SvXMLUnitConverter::convertEnum( aLineDash.Style, aStrValue, pXML_DashStyle_Enum ); + SvXMLUnitConverter::convertEnum( aLineDash.Style, aIter.toString(), pXML_DashStyle_Enum ); } break; case XML_ELEMENT(DRAW, XML_DOTS1): case XML_ELEMENT(DRAW_OOO, XML_DOTS1): - aLineDash.Dots = static_cast(aStrValue.toInt32()); + aLineDash.Dots = static_cast(aIter.toInt32()); break; case XML_ELEMENT(DRAW, XML_DOTS1_LENGTH): case XML_ELEMENT(DRAW_OOO, XML_DOTS1_LENGTH): { - if( aStrValue.indexOf( '%' ) != -1 ) // it's a percentage + if( aIter.toView().find( '%' ) != std::string_view::npos ) // it's a percentage { bIsRel = true; - ::sax::Converter::convertPercent(aLineDash.DotLen, aStrValue); + ::sax::Converter::convertPercent(aLineDash.DotLen, aIter.toView()); } else { rUnitConverter.convertMeasureToCore( aLineDash.DotLen, - aStrValue ); + aIter.toView() ); } } break; case XML_ELEMENT(DRAW, XML_DOTS2): case XML_ELEMENT(DRAW_OOO, XML_DOTS2): - aLineDash.Dashes = static_cast(aStrValue.toInt32()); + aLineDash.Dashes = static_cast(aIter.toInt32()); break; case XML_ELEMENT(DRAW, XML_DOTS2_LENGTH): case XML_ELEMENT(DRAW_OOO, XML_DOTS2_LENGTH): { - if( aStrValue.indexOf( '%' ) != -1 ) // it's a percentage + if( aIter.toView().find( '%' ) != std::string_view::npos ) // it's a percentage { bIsRel = true; - ::sax::Converter::convertPercent(aLineDash.DashLen, aStrValue); + ::sax::Converter::convertPercent(aLineDash.DashLen, aIter.toView()); } else { rUnitConverter.convertMeasureToCore( aLineDash.DashLen, - aStrValue ); + aIter.toView() ); } } break; @@ -145,15 +143,15 @@ void XMLDashStyleImport::importXML( case XML_ELEMENT(DRAW, XML_DISTANCE): case XML_ELEMENT(DRAW_OOO, XML_DISTANCE): { - if( aStrValue.indexOf( '%' ) != -1 ) // it's a percentage + if( aIter.toView().find( '%' ) != std::string_view::npos ) // it's a percentage { bIsRel = true; - ::sax::Converter::convertPercent(aLineDash.Distance, aStrValue); + ::sax::Converter::convertPercent(aLineDash.Distance, aIter.toView()); } else { rUnitConverter.convertMeasureToCore( aLineDash.Distance, - aStrValue ); + aIter.toView() ); } } break; diff --git a/xmloff/source/style/GradientStyle.cxx b/xmloff/source/style/GradientStyle.cxx index 221023d6e072..47070f0ac67c 100644 --- a/xmloff/source/style/GradientStyle.cxx +++ b/xmloff/source/style/GradientStyle.cxx @@ -78,48 +78,46 @@ void XMLGradientStyleImport::importXML( for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - const OUString aStrValue = aIter.toString(); - sal_Int32 nTmpValue; switch( aIter.getToken() ) { case XML_ELEMENT(DRAW, XML_NAME): - rStrName = aStrValue; + rStrName = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_DISPLAY_NAME): - aDisplayName = aStrValue; + aDisplayName = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_STYLE): - SvXMLUnitConverter::convertEnum( aGradient.Style, aStrValue, pXML_GradientStyle_Enum ); + SvXMLUnitConverter::convertEnum( aGradient.Style, aIter.toString(), pXML_GradientStyle_Enum ); break; case XML_ELEMENT(DRAW, XML_CX): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.XOffset = static_cast< sal_Int16 >( nTmpValue ); break; case XML_ELEMENT(DRAW, XML_CY): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.YOffset = static_cast< sal_Int16 >( nTmpValue ); break; case XML_ELEMENT(DRAW, XML_START_COLOR): - ::sax::Converter::convertColor(aGradient.StartColor, aStrValue); + ::sax::Converter::convertColor(aGradient.StartColor, aIter.toString()); break; case XML_ELEMENT(DRAW, XML_END_COLOR): - ::sax::Converter::convertColor(aGradient.EndColor, aStrValue); + ::sax::Converter::convertColor(aGradient.EndColor, aIter.toString()); break; case XML_ELEMENT(DRAW, XML_START_INTENSITY): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.StartIntensity = static_cast< sal_Int16 >( nTmpValue ); break; case XML_ELEMENT(DRAW, XML_END_INTENSITY): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.EndIntensity = static_cast< sal_Int16 >( nTmpValue ); break; case XML_ELEMENT(DRAW, XML_GRADIENT_ANGLE): { auto const cmp12(rImport.GetODFVersion().compareTo(u"" ODFVER_012_TEXT)); bool const bSuccess = - ::sax::Converter::convertAngle(aGradient.Angle, aStrValue, + ::sax::Converter::convertAngle(aGradient.Angle, aIter.toView(), // tdf#89475 try to detect borked OOo angles (cmp12 < 0) || (cmp12 == 0 && (rImport.isGeneratorVersionOlderThan(SvXMLImport::AOO_4x, SvXMLImport::LO_7x) @@ -129,7 +127,7 @@ void XMLGradientStyleImport::importXML( } break; case XML_ELEMENT(DRAW, XML_BORDER): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.Border = static_cast< sal_Int16 >( nTmpValue ); break; diff --git a/xmloff/source/style/HatchStyle.cxx b/xmloff/source/style/HatchStyle.cxx index eeee5edd2423..59d79a838551 100644 --- a/xmloff/source/style/HatchStyle.cxx +++ b/xmloff/source/style/HatchStyle.cxx @@ -75,34 +75,33 @@ void XMLHatchStyleImport::importXML( for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - const OUString aStrValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(DRAW, XML_NAME): case XML_ELEMENT(DRAW_OOO, XML_NAME): - rStrName = aStrValue; + rStrName = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_DISPLAY_NAME): case XML_ELEMENT(DRAW_OOO, XML_DISPLAY_NAME): - aDisplayName = aStrValue; + aDisplayName = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_STYLE): case XML_ELEMENT(DRAW_OOO, XML_STYLE): - SvXMLUnitConverter::convertEnum( aHatch.Style, aStrValue, pXML_HatchStyle_Enum ); + SvXMLUnitConverter::convertEnum( aHatch.Style, aIter.toString(), pXML_HatchStyle_Enum ); break; case XML_ELEMENT(DRAW, XML_COLOR): case XML_ELEMENT(DRAW_OOO, XML_COLOR): - ::sax::Converter::convertColor(aHatch.Color, aStrValue); + ::sax::Converter::convertColor(aHatch.Color, aIter.toString()); break; case XML_ELEMENT(DRAW, XML_DISTANCE): case XML_ELEMENT(DRAW_OOO, XML_DISTANCE): - rUnitConverter.convertMeasureToCore(aHatch.Distance, aStrValue); + rUnitConverter.convertMeasureToCore(aHatch.Distance, aIter.toView()); break; case XML_ELEMENT(DRAW, XML_ROTATION): case XML_ELEMENT(DRAW_OOO, XML_ROTATION): { sal_Int32 nValue; - if (::sax::Converter::convertNumber(nValue, aStrValue, 0, 3600)) + if (::sax::Converter::convertNumber(nValue, aIter.toView(), 0, 3600)) aHatch.Angle = sal_Int16(nValue); break; } diff --git a/xmloff/source/style/TransGradientStyle.cxx b/xmloff/source/style/TransGradientStyle.cxx index 8f7fe41d1444..f7cbb53a753a 100644 --- a/xmloff/source/style/TransGradientStyle.cxx +++ b/xmloff/source/style/TransGradientStyle.cxx @@ -79,39 +79,37 @@ void XMLTransGradientStyleImport::importXML( for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - const OUString aStrValue = aIter.toString(); - sal_Int32 nTmpValue; switch( aIter.getToken() ) { case XML_ELEMENT(DRAW, XML_NAME): { - rStrName = aStrValue; + rStrName = aIter.toString(); } break; case XML_ELEMENT(DRAW, XML_DISPLAY_NAME): { - aDisplayName = aStrValue; + aDisplayName = aIter.toString(); } break; case XML_ELEMENT(DRAW, XML_STYLE): { - SvXMLUnitConverter::convertEnum( aGradient.Style, aStrValue, pXML_GradientStyle_Enum ); + SvXMLUnitConverter::convertEnum( aGradient.Style, aIter.toString(), pXML_GradientStyle_Enum ); } break; case XML_ELEMENT(DRAW, XML_CX): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.XOffset = sal::static_int_cast< sal_Int16 >(nTmpValue); break; case XML_ELEMENT(DRAW, XML_CY): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.YOffset = sal::static_int_cast< sal_Int16 >(nTmpValue); break; case XML_ELEMENT(DRAW, XML_START): { sal_Int32 aStartTransparency; - ::sax::Converter::convertPercent( aStartTransparency, aStrValue ); + ::sax::Converter::convertPercent( aStartTransparency, aIter.toView() ); sal_uInt8 n = sal::static_int_cast< sal_uInt8 >( ( (100 - aStartTransparency) * 255 ) / 100 ); @@ -123,7 +121,7 @@ void XMLTransGradientStyleImport::importXML( case XML_ELEMENT(DRAW, XML_END): { sal_Int32 aEndTransparency; - ::sax::Converter::convertPercent( aEndTransparency, aStrValue ); + ::sax::Converter::convertPercent( aEndTransparency, aIter.toView() ); sal_uInt8 n = sal::static_int_cast< sal_uInt8 >( ( (100 - aEndTransparency) * 255 ) / 100 ); @@ -136,7 +134,7 @@ void XMLTransGradientStyleImport::importXML( { auto const cmp12(rImport.GetODFVersion().compareTo(u"" ODFVER_012_TEXT)); bool const bSuccess = - ::sax::Converter::convertAngle(aGradient.Angle, aStrValue, + ::sax::Converter::convertAngle(aGradient.Angle, aIter.toView(), // tdf#89475 try to detect borked OOo angles (cmp12 < 0) || (cmp12 == 0 && (rImport.isGeneratorVersionOlderThan(SvXMLImport::AOO_4x, SvXMLImport::LO_7x) @@ -146,7 +144,7 @@ void XMLTransGradientStyleImport::importXML( } break; case XML_ELEMENT(DRAW, XML_BORDER): - ::sax::Converter::convertPercent( nTmpValue, aStrValue ); + ::sax::Converter::convertPercent( nTmpValue, aIter.toView() ); aGradient.Border = sal::static_int_cast< sal_Int16 >(nTmpValue); break; diff --git a/xmloff/source/style/XMLFootnoteSeparatorImport.cxx b/xmloff/source/style/XMLFootnoteSeparatorImport.cxx index f2b776510e6c..879f4d3ffa13 100644 --- a/xmloff/source/style/XMLFootnoteSeparatorImport.cxx +++ b/xmloff/source/style/XMLFootnoteSeparatorImport.cxx @@ -89,14 +89,13 @@ void XMLFootnoteSeparatorImport::startFastElement( // iterate over xattribute list and fill values for (auto &aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - OUString sAttrValue = aIter.toString(); sal_Int32 nTmp; switch (aIter.getToken()) { case XML_ELEMENT(STYLE, XML_WIDTH): { if (GetImport().GetMM100UnitConverter().convertMeasureToCore( - nTmp, sAttrValue)) + nTmp, aIter.toView())) { nLineWeight = static_cast(nTmp); } @@ -105,14 +104,14 @@ void XMLFootnoteSeparatorImport::startFastElement( case XML_ELEMENT(STYLE, XML_DISTANCE_BEFORE_SEP): { if (GetImport().GetMM100UnitConverter().convertMeasureToCore( - nTmp, sAttrValue)) + nTmp, aIter.toView())) nLineTextDistance = nTmp; break; } case XML_ELEMENT(STYLE, XML_DISTANCE_AFTER_SEP): { if (GetImport().GetMM100UnitConverter().convertMeasureToCore( - nTmp, sAttrValue)) + nTmp, aIter.toView())) nLineDistance = nTmp; break; } @@ -127,18 +126,18 @@ void XMLFootnoteSeparatorImport::startFastElement( }; SvXMLUnitConverter::convertEnum( - eLineAdjust, sAttrValue, aXML_HorizontalAdjust_Enum); + eLineAdjust, aIter.toString(), aXML_HorizontalAdjust_Enum); break; } case XML_ELEMENT(STYLE, XML_REL_WIDTH ): { - if (::sax::Converter::convertPercent(nTmp, sAttrValue)) + if (::sax::Converter::convertPercent(nTmp, aIter.toView())) nLineRelWidth = static_cast(nTmp); break; } case XML_ELEMENT(STYLE, XML_COLOR): { - if (::sax::Converter::convertColor(nTmp, sAttrValue)) + if (::sax::Converter::convertColor(nTmp, aIter.toView())) { nLineColor = nTmp; } @@ -155,7 +154,7 @@ void XMLFootnoteSeparatorImport::startFastElement( { XML_TOKEN_INVALID, 0 } }; - SvXMLUnitConverter::convertEnum(nLineStyle, sAttrValue, aXML_LineStyle_Enum); + SvXMLUnitConverter::convertEnum(nLineStyle, aIter.toString(), aXML_LineStyle_Enum); break; } default: diff --git a/xmloff/source/style/xmlnumfi.cxx b/xmloff/source/style/xmlnumfi.cxx index 1910950e9662..10c57aaa04f8 100644 --- a/xmloff/source/style/xmlnumfi.cxx +++ b/xmloff/source/style/xmlnumfi.cxx @@ -421,12 +421,11 @@ SvXMLNumFmtPropContext::SvXMLNumFmtPropContext( SvXMLImport& rImport, { for( auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList ) ) { - OUString sValue = aIter.toString(); switch ( aIter.getToken()) { case XML_ELEMENT(FO, XML_COLOR): case XML_ELEMENT(FO_COMPAT, XML_COLOR): - bColSet = ::sax::Converter::convertColor( m_nColor, sValue ); + bColSet = ::sax::Converter::convertColor( m_nColor, aIter.toString() ); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -456,10 +455,9 @@ SvXMLNumFmtEmbeddedTextContext::SvXMLNumFmtEmbeddedTextContext( SvXMLImport& rIm for( auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList ) ) { - OUString sValue = aIter.toString(); if ( aIter.getToken() == XML_ELEMENT(NUMBER, XML_POSITION) ) { - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) nTextPosition = nAttrVal; } else @@ -655,11 +653,10 @@ SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( SvXMLImport& rImport, for( auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList ) ) { - OUString sValue = aIter.toString(); switch (aIter.getToken()) { case XML_ELEMENT(NUMBER, XML_DECIMAL_PLACES): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) { // fdo#58539 & gnome#627420: limit number of digits during import aNumInfo.nDecimals = std::min(nAttrVal, NF_MAX_FORMAT_SYMBOLS); @@ -667,61 +664,61 @@ SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( SvXMLImport& rImport, break; case XML_ELEMENT(LO_EXT, XML_MIN_DECIMAL_PLACES): case XML_ELEMENT(NUMBER, XML_MIN_DECIMAL_PLACES): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nMinDecimalDigits = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_MIN_INTEGER_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nInteger = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_GROUPING): - if (::sax::Converter::convertBool( bAttrBool, sValue )) + if (::sax::Converter::convertBool( bAttrBool, aIter.toView() )) aNumInfo.bGrouping = bAttrBool; break; case XML_ELEMENT(NUMBER, XML_DISPLAY_FACTOR): - if (::sax::Converter::convertDouble( fAttrDouble, sValue )) + if (::sax::Converter::convertDouble( fAttrDouble, aIter.toView() )) aNumInfo.fDisplayFactor = fAttrDouble; break; case XML_ELEMENT(NUMBER, XML_DECIMAL_REPLACEMENT): - if ( sValue == " " ) + if ( aIter.toView() == " " ) { aNumInfo.bDecAlign = true; // space replacement for "?" bVarDecimals = true; } else - if ( sValue.isEmpty() ) + if ( aIter.isEmpty() ) bVarDecimals = true; // empty replacement string: variable decimals else // all other strings aNumInfo.bDecReplace = true; // decimal replacement with dashes break; case XML_ELEMENT(NUMBER, XML_MIN_EXPONENT_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nExpDigits = std::min(nAttrVal, NF_MAX_FORMAT_SYMBOLS); break; case XML_ELEMENT(NUMBER, XML_EXPONENT_INTERVAL): case XML_ELEMENT(LO_EXT, XML_EXPONENT_INTERVAL): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nExpInterval = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_FORCED_EXPONENT_SIGN): case XML_ELEMENT(LO_EXT, XML_FORCED_EXPONENT_SIGN): - if (::sax::Converter::convertBool( bAttrBool, sValue )) + if (::sax::Converter::convertBool( bAttrBool, aIter.toView() )) aNumInfo.bExpSign = bAttrBool; break; case XML_ELEMENT(NUMBER, XML_MIN_NUMERATOR_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nMinNumerDigits = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_MIN_DENOMINATOR_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nMinDenomDigits = nAttrVal; break; case XML_ELEMENT(LO_EXT, XML_MAX_NUMERATOR_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 1 )) // at least one '#' + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 1 )) // at least one '#' aNumInfo.nMaxNumerDigits = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_DENOMINATOR_VALUE): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 1 )) // 0 is not valid + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 1 )) // 0 is not valid { aNumInfo.nFracDenominator = nAttrVal; bIsMaxDenominator = false; @@ -729,7 +726,7 @@ SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( SvXMLImport& rImport, break; case XML_ELEMENT(NUMBER, XML_MAX_DENOMINATOR_VALUE): // part of ODF 1.3 case XML_ELEMENT(LO_EXT, XML_MAX_DENOMINATOR_VALUE): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 1 ) && aNumInfo.nFracDenominator <= 0) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 1 ) && aNumInfo.nFracDenominator <= 0) { // if denominator value not yet defined aNumInfo.nFracDenominator = nAttrVal; bIsMaxDenominator = true; @@ -737,39 +734,39 @@ SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( SvXMLImport& rImport, break; case XML_ELEMENT(LO_EXT, XML_ZEROS_NUMERATOR_DIGITS): case XML_ELEMENT(NUMBER, XML_ZEROS_NUMERATOR_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nZerosNumerDigits = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_ZEROS_DENOMINATOR_DIGITS): case XML_ELEMENT(LO_EXT, XML_ZEROS_DENOMINATOR_DIGITS): - if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 )) + if (::sax::Converter::convertNumber( nAttrVal, aIter.toView(), 0 )) aNumInfo.nZerosDenomDigits = nAttrVal; break; case XML_ELEMENT(NUMBER, XML_INTEGER_FRACTION_DELIMITER): case XML_ELEMENT(LO_EXT, XML_INTEGER_FRACTION_DELIMITER): - aNumInfo.aIntegerFractionDelimiter = sValue; + aNumInfo.aIntegerFractionDelimiter = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_RFC_LANGUAGE_TAG): - aLanguageTagODF.maRfcLanguageTag = sValue; + aLanguageTagODF.maRfcLanguageTag = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_LANGUAGE): - aLanguageTagODF.maLanguage = sValue; + aLanguageTagODF.maLanguage = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_SCRIPT): - aLanguageTagODF.maScript = sValue; + aLanguageTagODF.maScript = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_COUNTRY): - aLanguageTagODF.maCountry = sValue; + aLanguageTagODF.maCountry = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_STYLE): - SvXMLUnitConverter::convertEnum( bLong, sValue, aStyleValueMap ); + SvXMLUnitConverter::convertEnum( bLong, aIter.toString(), aStyleValueMap ); break; case XML_ELEMENT(NUMBER, XML_TEXTUAL): - if (::sax::Converter::convertBool( bAttrBool, sValue )) + if (::sax::Converter::convertBool( bAttrBool, aIter.toView() )) bTextual = bAttrBool; break; case XML_ELEMENT(NUMBER, XML_CALENDAR): - sCalendar = sValue; + sCalendar = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); @@ -1146,59 +1143,58 @@ SvXMLNumFormatContext::SvXMLNumFormatContext( SvXMLImport& rImport, for( auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList ) ) { - OUString sValue = aIter.toString(); switch (aIter.getToken()) { // attributes for a style case XML_ELEMENT(STYLE, XML_NAME): break; case XML_ELEMENT(NUMBER, XML_RFC_LANGUAGE_TAG): - aLanguageTagODF.maRfcLanguageTag = sValue; + aLanguageTagODF.maRfcLanguageTag = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_LANGUAGE): - aLanguageTagODF.maLanguage = sValue; + aLanguageTagODF.maLanguage = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_SCRIPT): - aLanguageTagODF.maScript = sValue; + aLanguageTagODF.maScript = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_COUNTRY): - aLanguageTagODF.maCountry = sValue; + aLanguageTagODF.maCountry = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_TITLE): - sFormatTitle = sValue; + sFormatTitle = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_AUTOMATIC_ORDER): - if (::sax::Converter::convertBool( bAttrBool, sValue )) + if (::sax::Converter::convertBool( bAttrBool, aIter.toView() )) bAutoOrder = bAttrBool; break; case XML_ELEMENT(NUMBER, XML_FORMAT_SOURCE): - SvXMLUnitConverter::convertEnum( bFromSystem, sValue, aFormatSourceMap ); + SvXMLUnitConverter::convertEnum( bFromSystem, aIter.toString(), aFormatSourceMap ); break; case XML_ELEMENT(NUMBER, XML_TRUNCATE_ON_OVERFLOW): - if (::sax::Converter::convertBool( bAttrBool, sValue )) + if (::sax::Converter::convertBool( bAttrBool, aIter.toView() )) bTruncate = bAttrBool; break; case XML_ELEMENT(STYLE, XML_VOLATILE): // volatile formats can be removed after importing // if not used in other styles - if (::sax::Converter::convertBool( bAttrBool, sValue )) + if (::sax::Converter::convertBool( bAttrBool, aIter.toView() )) bRemoveAfterUse = bAttrBool; break; case XML_ELEMENT(NUMBER, XML_TRANSLITERATION_FORMAT): - aNatNumAttr.Format = sValue; + aNatNumAttr.Format = aIter.toString(); break; case XML_ELEMENT(LO_EXT, XML_TRANSLITERATION_SPELLOUT): case XML_ELEMENT(NUMBER, XML_TRANSLITERATION_SPELLOUT): - aSpellout = sValue; + aSpellout = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_TRANSLITERATION_LANGUAGE): - aNatNumAttr.Locale.Language = sValue; + aNatNumAttr.Locale.Language = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_TRANSLITERATION_COUNTRY): - aNatNumAttr.Locale.Country = sValue; + aNatNumAttr.Locale.Country = aIter.toString(); break; case XML_ELEMENT(NUMBER, XML_TRANSLITERATION_STYLE): - aNatNumAttr.Style = sValue; + aNatNumAttr.Style = aIter.toString(); break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); diff --git a/xmloff/source/style/xmlnumi.cxx b/xmloff/source/style/xmlnumi.cxx index dae61e273821..3d2cc5807789 100644 --- a/xmloff/source/style/xmlnumi.cxx +++ b/xmloff/source/style/xmlnumi.cxx @@ -258,29 +258,28 @@ SvxXMLListLevelStyleContext_Impl::SvxXMLListLevelStyleContext_Impl( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - const OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(TEXT, XML_LEVEL): - nLevel = sValue.toInt32(); + nLevel = aIter.toInt32(); if( nLevel >= 1 ) nLevel--; else nLevel = 0; break; case XML_ELEMENT(TEXT, XML_STYLE_NAME): - sTextStyleName = sValue; + sTextStyleName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_BULLET_CHAR): - if (!sValue.isEmpty()) + if (!aIter.isEmpty()) { sal_Int32 nIndexUtf16 = 0; - cBullet = sValue.iterateCodePoints(&nIndexUtf16); + cBullet = aIter.toString().iterateCodePoints(&nIndexUtf16); } break; case XML_ELEMENT(XLINK, XML_HREF): if( bImage ) - sImageURL = sValue; + sImageURL = aIter.toString(); break; case XML_ELEMENT(XLINK, XML_TYPE): case XML_ELEMENT(XLINK, XML_SHOW): @@ -289,22 +288,22 @@ SvxXMLListLevelStyleContext_Impl::SvxXMLListLevelStyleContext_Impl( break; case XML_ELEMENT(STYLE, XML_NUM_FORMAT): if( bNum ) - sNumFormat = sValue; + sNumFormat = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_NUM_PREFIX): - sPrefix = sValue; + sPrefix = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_NUM_SUFFIX): - sSuffix = sValue; + sSuffix = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_NUM_LETTER_SYNC): if( bNum ) - sNumLetterSync = sValue; + sNumLetterSync = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_START_VALUE): if( bNum ) { - sal_Int32 nTmp = sValue.toInt32(); + sal_Int32 nTmp = aIter.toInt32(); nNumStartValue = (nTmp < 0) ? 1 : ( (nTmp>SHRT_MAX) ? SHRT_MAX : static_cast(nTmp) ); @@ -313,7 +312,7 @@ SvxXMLListLevelStyleContext_Impl::SvxXMLListLevelStyleContext_Impl( case XML_ELEMENT(TEXT, XML_DISPLAY_LEVELS): if( bNum ) { - sal_Int32 nTmp = sValue.toInt32(); + sal_Int32 nTmp = aIter.toInt32(); nNumDisplayLevels = (nTmp < 1) ? 1 : ( (nTmp>SHRT_MAX) ? SHRT_MAX : static_cast(nTmp) ); @@ -553,74 +552,73 @@ SvxXMLListLevelStyleAttrContext_Impl::SvxXMLListLevelStyleAttrContext_Impl( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - const OUString sValue = aIter.toString(); sal_Int32 nVal; switch( aIter.getToken() ) { case XML_ELEMENT(TEXT, XML_SPACE_BEFORE): - if (rUnitConv.convertMeasureToCore(nVal, sValue, SHRT_MIN, SHRT_MAX)) + if (rUnitConv.convertMeasureToCore(nVal, aIter.toView(), SHRT_MIN, SHRT_MAX)) rListLevel.SetSpaceBefore( nVal ); break; case XML_ELEMENT(TEXT, XML_MIN_LABEL_WIDTH): - if (rUnitConv.convertMeasureToCore( nVal, sValue, 0, SHRT_MAX )) + if (rUnitConv.convertMeasureToCore( nVal, aIter.toView(), 0, SHRT_MAX )) rListLevel.SetMinLabelWidth( nVal ); break; case XML_ELEMENT(TEXT, XML_MIN_LABEL_DISTANCE): - if (rUnitConv.convertMeasureToCore( nVal, sValue, 0, USHRT_MAX )) + if (rUnitConv.convertMeasureToCore( nVal, aIter.toView(), 0, USHRT_MAX )) rListLevel.SetMinLabelDist( nVal ); break; case XML_ELEMENT(FO, XML_TEXT_ALIGN): case XML_ELEMENT(FO_COMPAT, XML_TEXT_ALIGN): - if( !sValue.isEmpty() ) + if( !aIter.isEmpty() ) { sal_Int16 eAdjust = HoriOrientation::LEFT; - if( IsXMLToken( sValue, XML_CENTER ) ) + if( IsXMLToken( aIter, XML_CENTER ) ) eAdjust = HoriOrientation::CENTER; - else if( IsXMLToken( sValue, XML_END ) ) + else if( IsXMLToken( aIter, XML_END ) ) eAdjust = HoriOrientation::RIGHT; rListLevel.SetAdjust( eAdjust ); } break; case XML_ELEMENT(STYLE, XML_FONT_NAME): - sFontName = sValue; + sFontName = aIter.toString(); break; case XML_ELEMENT(FO, XML_FONT_FAMILY): case XML_ELEMENT(FO_COMPAT, XML_FONT_FAMILY): - sFontFamily = sValue; + sFontFamily = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_FONT_FAMILY_GENERIC): - sFontFamilyGeneric = sValue; + sFontFamilyGeneric = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_FONT_STYLE_NAME): - sFontStyleName = sValue; + sFontStyleName = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_FONT_PITCH): - sFontPitch = sValue; + sFontPitch = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_FONT_CHARSET): - sFontCharset = sValue; + sFontCharset = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_VERTICAL_POS): - sVerticalPos = sValue; + sVerticalPos = aIter.toString(); break; case XML_ELEMENT(STYLE, XML_VERTICAL_REL): - sVerticalRel = sValue; + sVerticalRel = aIter.toString(); break; case XML_ELEMENT(FO, XML_WIDTH): case XML_ELEMENT(FO_COMPAT, XML_WIDTH): - if (rUnitConv.convertMeasureToCore(nVal, sValue)) + if (rUnitConv.convertMeasureToCore(nVal, aIter.toView())) rListLevel.SetImageWidth( nVal ); break; case XML_ELEMENT(FO, XML_HEIGHT): case XML_ELEMENT(FO_COMPAT, XML_HEIGHT): - if (rUnitConv.convertMeasureToCore(nVal, sValue)) + if (rUnitConv.convertMeasureToCore(nVal, aIter.toView())) rListLevel.SetImageHeight( nVal ); break; case XML_ELEMENT(FO, XML_COLOR): case XML_ELEMENT(FO_COMPAT, XML_COLOR): { sal_Int32 nColor(0); - if (::sax::Converter::convertColor( nColor, sValue )) + if (::sax::Converter::convertColor( nColor, aIter.toView() )) { rListLevel.SetColor( Color(nColor) ); } @@ -628,19 +626,19 @@ SvxXMLListLevelStyleAttrContext_Impl::SvxXMLListLevelStyleAttrContext_Impl( break; case XML_ELEMENT(STYLE, XML_USE_WINDOW_FONT_COLOR): { - if( IsXMLToken( sValue, XML_TRUE ) ) + if( IsXMLToken( aIter, XML_TRUE ) ) rListLevel.SetColor(COL_AUTO); } break; case XML_ELEMENT(FO, XML_FONT_SIZE): case XML_ELEMENT(FO_COMPAT, XML_FONT_SIZE): - if (::sax::Converter::convertPercent( nVal, sValue )) + if (::sax::Converter::convertPercent( nVal, aIter.toView() )) rListLevel.SetRelSize( static_cast(nVal) ); break; case XML_ELEMENT(TEXT, XML_LIST_LEVEL_POSITION_AND_SPACE_MODE): { sal_Int16 ePosAndSpaceMode = PositionAndSpaceMode::LABEL_WIDTH_AND_POSITION; - if( IsXMLToken( sValue, XML_LABEL_ALIGNMENT ) ) + if( IsXMLToken( aIter, XML_LABEL_ALIGNMENT ) ) ePosAndSpaceMode = PositionAndSpaceMode::LABEL_ALIGNMENT; rListLevel.SetPosAndSpaceMode( ePosAndSpaceMode ); } @@ -805,8 +803,6 @@ SvxXMLListLevelStyleLabelAlignmentAttrContext_Impl::SvxXMLListLevelStyleLabelAli sal_Int16 eLabelFollowedBy = LabelFollow::LISTTAB; for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - const OUString sValue = aIter.toString(); - sal_Int32 nVal; switch( aIter.getToken() ) { @@ -816,26 +812,26 @@ SvxXMLListLevelStyleLabelAlignmentAttrContext_Impl::SvxXMLListLevelStyleLabelAli if( eLabelFollowedBy == LabelFollow::NEWLINE) //NewLine from LO_EXT has precedence over other values of the Non LO_EXT namespace break; - if( IsXMLToken( sValue, XML_SPACE ) ) + if( IsXMLToken( aIter, XML_SPACE ) ) eLabelFollowedBy = LabelFollow::SPACE; - else if( IsXMLToken( sValue, XML_NOTHING ) ) + else if( IsXMLToken( aIter, XML_NOTHING ) ) eLabelFollowedBy = LabelFollow::NOTHING; - else if( IsXMLToken( sValue, XML_NEWLINE ) ) + else if( IsXMLToken( aIter, XML_NEWLINE ) ) eLabelFollowedBy = LabelFollow::NEWLINE; } break; case XML_ELEMENT(TEXT, XML_LIST_TAB_STOP_POSITION): - if (rUnitConv.convertMeasureToCore(nVal, sValue, 0, SHRT_MAX)) + if (rUnitConv.convertMeasureToCore(nVal, aIter.toView(), 0, SHRT_MAX)) rLLevel.SetListtabStopPosition( nVal ); break; case XML_ELEMENT(FO, XML_TEXT_INDENT): case XML_ELEMENT(FO_COMPAT, XML_TEXT_INDENT): - if (rUnitConv.convertMeasureToCore(nVal, sValue, SHRT_MIN, SHRT_MAX)) + if (rUnitConv.convertMeasureToCore(nVal, aIter.toView(), SHRT_MIN, SHRT_MAX)) rLLevel.SetFirstLineIndent( nVal ); break; case XML_ELEMENT(FO, XML_MARGIN_LEFT): case XML_ELEMENT(FO_COMPAT, XML_MARGIN_LEFT): - if (rUnitConv.convertMeasureToCore(nVal, sValue, SHRT_MIN, SHRT_MAX)) + if (rUnitConv.convertMeasureToCore(nVal, aIter.toView(), SHRT_MIN, SHRT_MAX)) rLLevel.SetIndentAt( nVal ); break; default: diff --git a/xmloff/source/style/xmltabi.cxx b/xmloff/source/style/xmltabi.cxx index 5c8db3fc89c5..f93e7d86c9b1 100644 --- a/xmloff/source/style/xmltabi.cxx +++ b/xmloff/source/style/xmltabi.cxx @@ -59,55 +59,53 @@ SvxXMLTabStopContext_Impl::SvxXMLTabStopContext_Impl( for (auto &aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - const OUString sValue = aIter.toString(); - sal_Int32 nVal; switch( aIter.getToken() ) { case XML_ELEMENT(STYLE, XML_POSITION): if (GetImport().GetMM100UnitConverter().convertMeasureToCore( - nVal, sValue)) + nVal, aIter.toView())) { aTabStop.Position = nVal; } break; case XML_ELEMENT(STYLE, XML_TYPE): - if( IsXMLToken( sValue, XML_LEFT ) ) + if( IsXMLToken( aIter, XML_LEFT ) ) { aTabStop.Alignment = style::TabAlign_LEFT; } - else if( IsXMLToken( sValue, XML_RIGHT ) ) + else if( IsXMLToken( aIter, XML_RIGHT ) ) { aTabStop.Alignment = style::TabAlign_RIGHT; } - else if( IsXMLToken( sValue, XML_CENTER ) ) + else if( IsXMLToken( aIter, XML_CENTER ) ) { aTabStop.Alignment = style::TabAlign_CENTER; } - else if( IsXMLToken( sValue, XML_CHAR ) ) + else if( IsXMLToken( aIter, XML_CHAR ) ) { aTabStop.Alignment = style::TabAlign_DECIMAL; } - else if( IsXMLToken( sValue, XML_DEFAULT ) ) + else if( IsXMLToken( aIter, XML_DEFAULT ) ) { aTabStop.Alignment = style::TabAlign_DEFAULT; } break; case XML_ELEMENT(STYLE, XML_CHAR): - if( !sValue.isEmpty() ) - aTabStop.DecimalChar = sValue[0]; + if( !aIter.isEmpty() ) + aTabStop.DecimalChar = aIter.toString()[0]; break; case XML_ELEMENT(STYLE, XML_LEADER_STYLE): - if( IsXMLToken( sValue, XML_NONE ) ) + if( IsXMLToken( aIter, XML_NONE ) ) aTabStop.FillChar = ' '; - else if( IsXMLToken( sValue, XML_DOTTED ) ) + else if( IsXMLToken( aIter, XML_DOTTED ) ) aTabStop.FillChar = '.'; else aTabStop.FillChar = '_'; break; case XML_ELEMENT(STYLE, XML_LEADER_TEXT): - if( !sValue.isEmpty() ) - cTextFillChar = sValue[0]; + if( !aIter.isEmpty() ) + cTextFillChar = aIter.toString()[0]; break; default: XMLOFF_WARN_UNKNOWN("xmloff", aIter); diff --git a/xmloff/source/text/XMLCalculationSettingsContext.cxx b/xmloff/source/text/XMLCalculationSettingsContext.cxx index 338080a3b643..fa7ff423658a 100644 --- a/xmloff/source/text/XMLCalculationSettingsContext.cxx +++ b/xmloff/source/text/XMLCalculationSettingsContext.cxx @@ -47,9 +47,7 @@ XMLCalculationSettingsContext::XMLCalculationSettingsContext( SvXMLImport& rImpo { if (aIter.getToken() == XML_ELEMENT(TABLE, XML_NULL_YEAR) ) { - sal_Int32 nTemp; - ::sax::Converter::convertNumber(nTemp, aIter.toString()); - nYear= static_cast (nTemp); + nYear = static_cast (aIter.toInt32()); } else XMLOFF_WARN_UNKNOWN("xmloff", aIter); diff --git a/xmloff/source/text/XMLChangedRegionImportContext.cxx b/xmloff/source/text/XMLChangedRegionImportContext.cxx index 400f18a9b44e..b96da6c36fa2 100644 --- a/xmloff/source/text/XMLChangedRegionImportContext.cxx +++ b/xmloff/source/text/XMLChangedRegionImportContext.cxx @@ -58,24 +58,23 @@ void XMLChangedRegionImportContext::startFastElement( bool bHaveXmlId( false ); for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - const OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(XML, XML_ID): { - sID = sValue; + sID = aIter.toString(); bHaveXmlId = true; break; } case XML_ELEMENT(TEXT, XML_ID): { - if (!bHaveXmlId) { sID = sValue; } + if (!bHaveXmlId) { sID = aIter.toString(); } break; } case XML_ELEMENT(TEXT, XML_MERGE_LAST_PARAGRAPH): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, sValue)) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) { bMergeLastPara = bTmp; } diff --git a/xmloff/source/text/XMLFootnoteConfigurationImportContext.cxx b/xmloff/source/text/XMLFootnoteConfigurationImportContext.cxx index 756c85377bab..1ce26817c015 100644 --- a/xmloff/source/text/XMLFootnoteConfigurationImportContext.cxx +++ b/xmloff/source/text/XMLFootnoteConfigurationImportContext.cxx @@ -136,7 +136,7 @@ XMLFootnoteConfigurationImportContext::XMLFootnoteConfigurationImportContext( { if( aIter.getToken() == XML_ELEMENT(TEXT, XML_NOTE_CLASS) ) { - if( IsXMLToken(aIter.toString(), XML_ENDNOTE ) ) + if( IsXMLToken(aIter, XML_ENDNOTE ) ) { bIsEndnote = true; SetFamily( XmlStyleFamily::TEXT_FOOTNOTECONFIG ); diff --git a/xmloff/source/text/XMLFootnoteImportContext.cxx b/xmloff/source/text/XMLFootnoteImportContext.cxx index fea2e95c4b2a..c7d3c599d86b 100644 --- a/xmloff/source/text/XMLFootnoteImportContext.cxx +++ b/xmloff/source/text/XMLFootnoteImportContext.cxx @@ -69,7 +69,7 @@ void XMLFootnoteImportContext::startFastElement( { if( aIter.getToken() == XML_ELEMENT(TEXT, XML_NOTE_CLASS) ) { - if( IsXMLToken( aIter.toString(), XML_ENDNOTE ) ) + if( IsXMLToken( aIter, XML_ENDNOTE ) ) bIsEndnote = true; break; } diff --git a/xmloff/source/text/XMLIndexBibliographyConfigurationContext.cxx b/xmloff/source/text/XMLIndexBibliographyConfigurationContext.cxx index 759c786f8aa8..b38fd4db03d4 100644 --- a/xmloff/source/text/XMLIndexBibliographyConfigurationContext.cxx +++ b/xmloff/source/text/XMLIndexBibliographyConfigurationContext.cxx @@ -140,7 +140,7 @@ css::uno::Reference< css::xml::sax::XFastContextHandler > XMLIndexBibliographyCo case XML_ELEMENT(TEXT, XML_SORT_ASCENDING): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, aIter.toString())) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) bSort = bTmp; break; } diff --git a/xmloff/source/text/XMLIndexChapterInfoEntryContext.cxx b/xmloff/source/text/XMLIndexChapterInfoEntryContext.cxx index 8436d1bb339b..1f5bbc2ea11f 100644 --- a/xmloff/source/text/XMLIndexChapterInfoEntryContext.cxx +++ b/xmloff/source/text/XMLIndexChapterInfoEntryContext.cxx @@ -103,7 +103,7 @@ void XMLIndexChapterInfoEntryContext::startFastElement( case XML_ELEMENT(TEXT, XML_OUTLINE_LEVEL): { sal_Int32 nTmp; - if (::sax::Converter::convertNumber(nTmp, aIter.toString())) + if (::sax::Converter::convertNumber(nTmp, aIter.toView())) { //control on range is carried out in the UNO level nOutlineLevel = static_cast(nTmp); diff --git a/xmloff/source/text/XMLIndexTOCContext.cxx b/xmloff/source/text/XMLIndexTOCContext.cxx index c883edf6cc28..d3a7ea4ef1b1 100644 --- a/xmloff/source/text/XMLIndexTOCContext.cxx +++ b/xmloff/source/text/XMLIndexTOCContext.cxx @@ -145,7 +145,7 @@ void XMLIndexTOCContext::startFastElement( case XML_ELEMENT(TEXT, XML_PROTECTED): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, aIter.toString())) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) { bProtected = bTmp; } diff --git a/xmloff/source/text/XMLIndexTOCStylesContext.cxx b/xmloff/source/text/XMLIndexTOCStylesContext.cxx index 830369240e7c..aee1b9c9f70b 100644 --- a/xmloff/source/text/XMLIndexTOCStylesContext.cxx +++ b/xmloff/source/text/XMLIndexTOCStylesContext.cxx @@ -65,7 +65,7 @@ void XMLIndexTOCStylesContext::startFastElement( { sal_Int32 nTmp; if (::sax::Converter::convertNumber( - nTmp, aIter.toString(), 1, + nTmp, aIter.toView(), 1, GetImport().GetTextImport()->GetChapterNumbering()-> getCount())) { diff --git a/xmloff/source/text/XMLIndexTabStopEntryContext.cxx b/xmloff/source/text/XMLIndexTabStopEntryContext.cxx index 1c785c53cc4d..b2ac30899440 100644 --- a/xmloff/source/text/XMLIndexTabStopEntryContext.cxx +++ b/xmloff/source/text/XMLIndexTabStopEntryContext.cxx @@ -64,7 +64,6 @@ void XMLIndexTabStopEntryContext::startFastElement( // process three attributes: type, position, leader char for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(STYLE, XML_TYPE): @@ -72,14 +71,14 @@ void XMLIndexTabStopEntryContext::startFastElement( // if it's neither left nor right, value is // ignored. Since left is default, we only need to // check for right - bTabRightAligned = IsXMLToken( sValue, XML_RIGHT ); + bTabRightAligned = IsXMLToken( aIter, XML_RIGHT ); break; } case XML_ELEMENT(STYLE, XML_POSITION): { sal_Int32 nTmp; if (GetImport().GetMM100UnitConverter(). - convertMeasureToCore(nTmp, sValue)) + convertMeasureToCore(nTmp, aIter.toView())) { nTabPosition = nTmp; bTabPositionOK = true; @@ -88,16 +87,16 @@ void XMLIndexTabStopEntryContext::startFastElement( } case XML_ELEMENT(STYLE, XML_LEADER_CHAR): { - sLeaderChar = sValue; + sLeaderChar = aIter.toString(); // valid only, if we have a char! - bLeaderCharOK = !sValue.isEmpty(); + bLeaderCharOK = !sLeaderChar.isEmpty(); break; } case XML_ELEMENT(STYLE, XML_WITH_TAB): { // #i21237# bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, sValue)) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) bWithTab = bTmp; break; } diff --git a/xmloff/source/text/XMLLineNumberingSeparatorImportContext.cxx b/xmloff/source/text/XMLLineNumberingSeparatorImportContext.cxx index 8d6269235ed6..a275307275d1 100644 --- a/xmloff/source/text/XMLLineNumberingSeparatorImportContext.cxx +++ b/xmloff/source/text/XMLLineNumberingSeparatorImportContext.cxx @@ -56,7 +56,7 @@ void XMLLineNumberingSeparatorImportContext::startFastElement( if ( aIter.getToken() == XML_ELEMENT(TEXT, XML_INCREMENT) ) { sal_Int32 nTmp; - if (::sax::Converter::convertNumber(nTmp, aIter.toString(), 0)) + if (::sax::Converter::convertNumber(nTmp, aIter.toView(), 0)) { rLineNumberingContext.SetSeparatorIncrement(static_cast(nTmp)); } diff --git a/xmloff/source/text/XMLSectionFootnoteConfigImport.cxx b/xmloff/source/text/XMLSectionFootnoteConfigImport.cxx index 26e8af646d37..fcdbf2221b5a 100644 --- a/xmloff/source/text/XMLSectionFootnoteConfigImport.cxx +++ b/xmloff/source/text/XMLSectionFootnoteConfigImport.cxx @@ -77,14 +77,12 @@ void XMLSectionFootnoteConfigImport::startFastElement( // iterate over xattribute list and fill values for (auto &aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - OUString sAttrValue = aIter.toString(); - switch(aIter.getToken()) { case XML_ELEMENT(TEXT, XML_START_VALUE): { sal_Int32 nTmp; - if (::sax::Converter::convertNumber(nTmp, sAttrValue)) + if (::sax::Converter::convertNumber(nTmp, aIter.toView())) { nNumRestartAt = static_cast< sal_Int16 >( nTmp ) - 1; bNumRestart = true; @@ -93,31 +91,31 @@ void XMLSectionFootnoteConfigImport::startFastElement( } case XML_ELEMENT(TEXT, XML_NOTE_CLASS): { - if( IsXMLToken( sAttrValue, XML_ENDNOTE ) ) + if( IsXMLToken( aIter, XML_ENDNOTE ) ) bEndnote = true; break; } case XML_ELEMENT(STYLE, XML_NUM_PREFIX): { - sNumPrefix = sAttrValue; + sNumPrefix = aIter.toString(); bNumOwn = true; break; } case XML_ELEMENT(TEXT, XML_NUM_SUFFIX): { - sNumSuffix = sAttrValue; + sNumSuffix = aIter.toString(); bNumOwn = true; break; } case XML_ELEMENT(TEXT, XML_NUM_FORMAT): { - sNumFormat = sAttrValue; + sNumFormat = aIter.toString(); bNumOwn = true; break; } case XML_ELEMENT(TEXT, XML_NUM_LETTER_SYNC): { - sNumLetterSync = sAttrValue; + sNumLetterSync = aIter.toString(); bNumOwn = true; break; } diff --git a/xmloff/source/text/XMLSectionImportContext.cxx b/xmloff/source/text/XMLSectionImportContext.cxx index 701fc76af839..54620ddfe5a9 100644 --- a/xmloff/source/text/XMLSectionImportContext.cxx +++ b/xmloff/source/text/XMLSectionImportContext.cxx @@ -198,22 +198,21 @@ void XMLSectionImportContext::ProcessAttributes( { for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); - switch (aIter.getToken()) { case XML_ELEMENT(XML, XML_ID): - sXmlId = sValue; + sXmlId = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_STYLE_NAME): - sStyleName = sValue; + sStyleName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_NAME): - sName = sValue; + sName = aIter.toString(); bValid = true; break; case XML_ELEMENT(TEXT, XML_CONDITION): { + OUString sValue = aIter.toString(); OUString sTmp; sal_uInt16 nPrefix = GetImport().GetNamespaceMap(). GetKeyByAttrValueQName(sValue, &sTmp); @@ -227,12 +226,12 @@ void XMLSectionImportContext::ProcessAttributes( } break; case XML_ELEMENT(TEXT, XML_DISPLAY): - if (IsXMLToken(sValue, XML_TRUE)) + if (IsXMLToken(aIter, XML_TRUE)) { bIsVisible = true; } - else if ( IsXMLToken(sValue, XML_NONE) || - IsXMLToken(sValue, XML_CONDITION) ) + else if ( IsXMLToken(aIter, XML_NONE) || + IsXMLToken(aIter, XML_CONDITION) ) { bIsVisible = false; } @@ -241,7 +240,7 @@ void XMLSectionImportContext::ProcessAttributes( case XML_ELEMENT(TEXT, XML_IS_HIDDEN): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, sValue)) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) { bIsCurrentlyVisible = !bTmp; bIsCurrentlyVisibleOK = true; @@ -249,7 +248,7 @@ void XMLSectionImportContext::ProcessAttributes( } break; case XML_ELEMENT(TEXT, XML_PROTECTION_KEY): - ::comphelper::Base64::decode(aSequence, sValue); + ::comphelper::Base64::decode(aSequence, aIter.toString()); bSequenceOK = true; break; case XML_ELEMENT(TEXT, XML_PROTECTED): @@ -257,7 +256,7 @@ void XMLSectionImportContext::ProcessAttributes( case XML_ELEMENT(TEXT, XML_PROTECT): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, sValue)) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) { bProtect = bTmp; } diff --git a/xmloff/source/text/XMLSectionSourceDDEImportContext.cxx b/xmloff/source/text/XMLSectionSourceDDEImportContext.cxx index 08f6ab51b53e..3e98f5114100 100644 --- a/xmloff/source/text/XMLSectionSourceDDEImportContext.cxx +++ b/xmloff/source/text/XMLSectionSourceDDEImportContext.cxx @@ -75,7 +75,7 @@ void XMLSectionSourceDDEImportContext::startFastElement(sal_Int32 /*nElement*/, case XML_ELEMENT(OFFICE, XML_AUTOMATIC_UPDATE): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, aIter.toString())) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) { bAutomaticUpdate = bTmp; } diff --git a/xmloff/source/text/XMLTextColumnsContext.cxx b/xmloff/source/text/XMLTextColumnsContext.cxx index acf7ad7a64cc..5cf2d03f60cd 100644 --- a/xmloff/source/text/XMLTextColumnsContext.cxx +++ b/xmloff/source/text/XMLTextColumnsContext.cxx @@ -87,18 +87,17 @@ XMLTextColumnContext_Impl::XMLTextColumnContext_Impl( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - const OUString sValue = aIter.toString(); sal_Int32 nVal; switch( aIter.getToken() ) { case XML_ELEMENT(STYLE, XML_REL_WIDTH): { - sal_Int32 nPos = sValue.indexOf( '*' ); - if( nPos != -1 && nPos+1 == sValue.getLength() ) + size_t nPos = aIter.toView().find( '*' ); + if( nPos != std::string_view::npos && static_cast(nPos+1) == aIter.getLength() ) { if (::sax::Converter::convertNumber( nVal, - std::u16string_view(sValue).substr(0, nPos), + aIter.toView().substr(0, nPos), 0, USHRT_MAX)) aColumn.Width = nVal; } @@ -107,13 +106,13 @@ XMLTextColumnContext_Impl::XMLTextColumnContext_Impl( case XML_ELEMENT(FO, XML_START_INDENT): case XML_ELEMENT(FO_COMPAT, XML_START_INDENT): if( GetImport().GetMM100UnitConverter(). - convertMeasureToCore( nVal, sValue ) ) + convertMeasureToCore( nVal, aIter.toView() ) ) aColumn.LeftMargin = nVal; break; case XML_ELEMENT(FO, XML_END_INDENT): case XML_ELEMENT(FO_COMPAT, XML_END_INDENT): if( GetImport().GetMM100UnitConverter(). - convertMeasureToCore( nVal, sValue ) ) + convertMeasureToCore( nVal, aIter.toView() ) ) aColumn.RightMargin = nVal; break; default: @@ -158,30 +157,28 @@ XMLTextColumnSepContext_Impl::XMLTextColumnSepContext_Impl( { for (auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList )) { - const OUString sValue = aIter.toString(); - sal_Int32 nVal; switch( aIter.getToken() ) { case XML_ELEMENT(STYLE, XML_WIDTH): if( GetImport().GetMM100UnitConverter(). - convertMeasureToCore( nVal, sValue ) ) + convertMeasureToCore( nVal, aIter.toView() ) ) nWidth = nVal; break; case XML_ELEMENT(STYLE, XML_HEIGHT): - if (::sax::Converter::convertPercent( nVal, sValue ) && + if (::sax::Converter::convertPercent( nVal, aIter.toView() ) && nVal >=1 && nVal <= 100 ) nHeight = static_cast(nVal); break; case XML_ELEMENT(STYLE, XML_COLOR): - ::sax::Converter::convertColor( nColor, sValue ); + ::sax::Converter::convertColor( nColor, aIter.toView() ); break; case XML_ELEMENT(STYLE, XML_VERTICAL_ALIGN): - SvXMLUnitConverter::convertEnum( eVertAlign, sValue, + SvXMLUnitConverter::convertEnum( eVertAlign, aIter.toString(), pXML_Sep_Align_Enum ); break; case XML_ELEMENT(STYLE, XML_STYLE): - SvXMLUnitConverter::convertEnum( nStyle, sValue, + SvXMLUnitConverter::convertEnum( nStyle, aIter.toString(), pXML_Sep_Style_Enum ); break; default: @@ -211,19 +208,18 @@ XMLTextColumnsContext::XMLTextColumnsContext( sal_Int32 nVal; for (auto &aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - const OUString sValue = aIter.toString(); switch(aIter.getToken()) { case XML_ELEMENT(FO, XML_COLUMN_COUNT): case XML_ELEMENT(FO_COMPAT, XML_COLUMN_COUNT): - if(::sax::Converter::convertNumber( nVal, sValue, 0, SHRT_MAX )) + if(::sax::Converter::convertNumber( nVal, aIter.toView(), 0, SHRT_MAX )) nCount = static_cast(nVal); break; case XML_ELEMENT(FO, XML_COLUMN_GAP): case XML_ELEMENT(FO_COMPAT, XML_COLUMN_GAP): { bAutomatic = GetImport().GetMM100UnitConverter(). - convertMeasureToCore( nAutomaticDistance, sValue ); + convertMeasureToCore( nAutomaticDistance, aIter.toView() ); break; } default: diff --git a/xmloff/source/text/XMLTextFrameContext.cxx b/xmloff/source/text/XMLTextFrameContext.cxx index 066b5d124a99..2855e64dd45f 100644 --- a/xmloff/source/text/XMLTextFrameContext.cxx +++ b/xmloff/source/text/XMLTextFrameContext.cxx @@ -220,40 +220,39 @@ XMLTextFrameContourContext_Impl::XMLTextFrameContourContext_Impl( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(SVG, XML_VIEWBOX): case XML_ELEMENT(SVG_COMPAT, XML_VIEWBOX): - sViewBox = sValue; + sViewBox = aIter.toString(); break; case XML_ELEMENT(SVG, XML_D): case XML_ELEMENT(SVG_COMPAT, XML_D): if( bPath ) - sD = sValue; + sD = aIter.toString(); break; case XML_ELEMENT(DRAW,XML_POINTS): if( !bPath ) - sPoints = sValue; + sPoints = aIter.toString(); break; case XML_ELEMENT(SVG, XML_WIDTH): case XML_ELEMENT(SVG_COMPAT, XML_WIDTH): - if (::sax::Converter::convertMeasurePx(nWidth, sValue)) + if (::sax::Converter::convertMeasurePx(nWidth, aIter.toView())) bPixelWidth = true; else GetImport().GetMM100UnitConverter().convertMeasureToCore( - nWidth, sValue); + nWidth, aIter.toView()); break; case XML_ELEMENT(SVG, XML_HEIGHT): case XML_ELEMENT(SVG_COMPAT, XML_HEIGHT): - if (::sax::Converter::convertMeasurePx(nHeight, sValue)) + if (::sax::Converter::convertMeasurePx(nHeight, aIter.toView())) bPixelHeight = true; else GetImport().GetMM100UnitConverter().convertMeasureToCore( - nHeight, sValue); + nHeight, aIter.toView()); break; case XML_ELEMENT(DRAW, XML_RECREATE_ON_EDIT): - bAuto = IsXMLToken(sValue, XML_TRUE); + bAuto = IsXMLToken(aIter, XML_TRUE); break; } } @@ -836,22 +835,22 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl( bOwnBase64Stream = false; mbMultipleContent = bMultipleContent; - auto processAttr = [&](sal_Int32 nElement, OUString rValue) -> void + auto processAttr = [&](sal_Int32 nElement, const sax_fastparser::FastAttributeList::FastAttributeIter& aIter) -> void { switch( nElement ) { case XML_ELEMENT(DRAW, XML_STYLE_NAME): - sStyleName = rValue; + sStyleName = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_NAME): - m_sOrigName = rValue; - sName = rValue; + m_sOrigName = aIter.toString(); + sName = m_sOrigName; break; case XML_ELEMENT(DRAW, XML_FRAME_NAME): - sFrameName = rValue; + sFrameName = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_APPLET_NAME): - sAppletName = rValue; + sAppletName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_ANCHOR_TYPE): if( TextContentAnchorType_AT_PARAGRAPH == eAnchorType || @@ -860,7 +859,7 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl( { TextContentAnchorType eNew; - if( XMLAnchorTypePropHdl::convert( rValue, eNew ) && + if( XMLAnchorTypePropHdl::convert( aIter.toString(), eNew ) && ( TextContentAnchorType_AT_PARAGRAPH == eNew || TextContentAnchorType_AT_CHARACTER == eNew || TextContentAnchorType_AS_CHARACTER == eNew || @@ -871,83 +870,83 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl( case XML_ELEMENT(TEXT, XML_ANCHOR_PAGE_NUMBER): { sal_Int32 nTmp; - if (::sax::Converter::convertNumber(nTmp, rValue, 1, SHRT_MAX)) + if (::sax::Converter::convertNumber(nTmp, aIter.toView(), 1, SHRT_MAX)) nPage = static_cast(nTmp); } break; case XML_ELEMENT(SVG, XML_X): case XML_ELEMENT(SVG_COMPAT, XML_X): GetImport().GetMM100UnitConverter().convertMeasureToCore( - nX, rValue); + nX, aIter.toView()); break; case XML_ELEMENT(SVG, XML_Y): case XML_ELEMENT(SVG_COMPAT, XML_Y): GetImport().GetMM100UnitConverter().convertMeasureToCore( - nY, rValue ); + nY, aIter.toView() ); break; case XML_ELEMENT(SVG, XML_WIDTH): case XML_ELEMENT(SVG_COMPAT, XML_WIDTH): // relative widths are obsolete since SRC617. Remove them some day! - if( rValue.indexOf( '%' ) != -1 ) + if( aIter.toView().find( '%' ) != std::string_view::npos ) { sal_Int32 nTmp; - ::sax::Converter::convertPercent( nTmp, rValue ); + ::sax::Converter::convertPercent( nTmp, aIter.toView() ); nRelWidth = static_cast(nTmp); } else { GetImport().GetMM100UnitConverter().convertMeasureToCore( - nWidth, rValue, 0 ); + nWidth, aIter.toView(), 0 ); } break; case XML_ELEMENT(STYLE, XML_REL_WIDTH): - if( IsXMLToken(rValue, XML_SCALE) ) + if( IsXMLToken(aIter, XML_SCALE) ) { bSyncWidth = true; } else { sal_Int32 nTmp; - if (::sax::Converter::convertPercent( nTmp, rValue )) + if (::sax::Converter::convertPercent( nTmp, aIter.toView() )) nRelWidth = static_cast(nTmp); } break; case XML_ELEMENT(FO, XML_MIN_WIDTH): case XML_ELEMENT(FO_COMPAT, XML_MIN_WIDTH): - if( rValue.indexOf( '%' ) != -1 ) + if( aIter.toView().find( '%' ) != std::string_view::npos ) { sal_Int32 nTmp; - ::sax::Converter::convertPercent( nTmp, rValue ); + ::sax::Converter::convertPercent( nTmp, aIter.toView() ); nRelWidth = static_cast(nTmp); } else { GetImport().GetMM100UnitConverter().convertMeasureToCore( - nWidth, rValue, 0 ); + nWidth, aIter.toView(), 0 ); } bMinWidth = true; break; case XML_ELEMENT(SVG, XML_HEIGHT): case XML_ELEMENT(SVG_COMPAT, XML_HEIGHT): // relative heights are obsolete since SRC617. Remove them some day! - if( rValue.indexOf( '%' ) != -1 ) + if( aIter.toView().find( '%' ) != std::string_view::npos ) { sal_Int32 nTmp; - ::sax::Converter::convertPercent( nTmp, rValue ); + ::sax::Converter::convertPercent( nTmp, aIter.toView() ); nRelHeight = static_cast(nTmp); } else { GetImport().GetMM100UnitConverter().convertMeasureToCore( - nHeight, rValue, 0 ); + nHeight, aIter.toView(), 0 ); } break; case XML_ELEMENT(STYLE, XML_REL_HEIGHT): - if( IsXMLToken( rValue, XML_SCALE ) ) + if( IsXMLToken( aIter, XML_SCALE ) ) { bSyncHeight = true; } - else if( IsXMLToken( rValue, XML_SCALE_MIN ) ) + else if( IsXMLToken( aIter, XML_SCALE_MIN ) ) { bSyncHeight = true; bMinHeight = true; @@ -955,33 +954,33 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl( else { sal_Int32 nTmp; - if (::sax::Converter::convertPercent( nTmp, rValue )) + if (::sax::Converter::convertPercent( nTmp, aIter.toView() )) nRelHeight = static_cast(nTmp); } break; case XML_ELEMENT(FO, XML_MIN_HEIGHT): case XML_ELEMENT(FO_COMPAT, XML_MIN_HEIGHT): - if( rValue.indexOf( '%' ) != -1 ) + if( aIter.toView().find( '%' ) != std::string_view::npos ) { sal_Int32 nTmp; - ::sax::Converter::convertPercent( nTmp, rValue ); + ::sax::Converter::convertPercent( nTmp, aIter.toView() ); nRelHeight = static_cast(nTmp); } else { GetImport().GetMM100UnitConverter().convertMeasureToCore( - nHeight, rValue, 0 ); + nHeight, aIter.toView(), 0 ); } bMinHeight = true; break; case XML_ELEMENT(DRAW, XML_ZINDEX): - ::sax::Converter::convertNumber( nZIndex, rValue, -1 ); + ::sax::Converter::convertNumber( nZIndex, aIter.toView(), -1 ); break; case XML_ELEMENT(DRAW, XML_CHAIN_NEXT_NAME): - sNextName = rValue; + sNextName = aIter.toString(); break; case XML_ELEMENT(XLINK, XML_HREF): - sHRef = rValue; + sHRef = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_TRANSFORM): { @@ -997,7 +996,7 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl( // but is not generally available (as it should be, a 'current' UnitConverter should // be available at GetExport() - and maybe was once). May have to be addressed as soon // as translate transformations are used here. - aSdXMLImExTransform2D.SetString(rValue, GetImport().GetMM100UnitConverter()); + aSdXMLImExTransform2D.SetString(aIter.toString(), GetImport().GetMM100UnitConverter()); aSdXMLImExTransform2D.GetFullTransform(aFullTransform); if(!aFullTransform.isIdentity()) @@ -1045,32 +1044,32 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl( } break; case XML_ELEMENT(DRAW, XML_CODE): - sCode = rValue; + sCode = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_OBJECT): break; case XML_ELEMENT(DRAW, XML_ARCHIVE): break; case XML_ELEMENT(DRAW, XML_MAY_SCRIPT): - bMayScript = IsXMLToken( rValue, XML_TRUE ); + bMayScript = IsXMLToken( aIter, XML_TRUE ); break; case XML_ELEMENT(DRAW, XML_MIME_TYPE): case XML_ELEMENT(LO_EXT, XML_MIME_TYPE): - sMimeType = rValue; + sMimeType = aIter.toString(); break; case XML_ELEMENT(DRAW, XML_NOTIFY_ON_UPDATE_OF_RANGES): case XML_ELEMENT(DRAW, XML_NOTIFY_ON_UPDATE_OF_TABLE): - sTblName = rValue; + sTblName = aIter.toString(); break; default: - XMLOFF_WARN_UNKNOWN_ATTR("xmloff", nElement, rValue); + XMLOFF_WARN_UNKNOWN("xmloff", aIter); } }; for( auto& aIter : sax_fastparser::castToFastAttributeList(rAttrList) ) - processAttr(aIter.getToken(), aIter.toString()); + processAttr(aIter.getToken(), aIter); for( auto& aIter : sax_fastparser::castToFastAttributeList(rFrameAttrList) ) - processAttr(aIter.getToken(), aIter.toString()); + processAttr(aIter.getToken(), aIter); if( ( (XML_TEXT_FRAME_GRAPHIC == nType || XML_TEXT_FRAME_OBJECT == nType || @@ -1354,8 +1353,7 @@ XMLTextFrameContext::XMLTextFrameContext( case XML_ELEMENT(TEXT, XML_ANCHOR_TYPE): { TextContentAnchorType eNew; - if( XMLAnchorTypePropHdl::convert( aIter.toString(), - eNew ) && + if( XMLAnchorTypePropHdl::convert( aIter.toString(), eNew ) && ( TextContentAnchorType_AT_PARAGRAPH == eNew || TextContentAnchorType_AT_CHARACTER == eNew || TextContentAnchorType_AS_CHARACTER == eNew || diff --git a/xmloff/source/text/XMLTextFrameHyperlinkContext.cxx b/xmloff/source/text/XMLTextFrameHyperlinkContext.cxx index a6e717f2a879..240d9776b519 100644 --- a/xmloff/source/text/XMLTextFrameHyperlinkContext.cxx +++ b/xmloff/source/text/XMLTextFrameHyperlinkContext.cxx @@ -49,25 +49,24 @@ XMLTextFrameHyperlinkContext::XMLTextFrameHyperlinkContext( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(XLINK, XML_HREF): - sHRef = GetImport().GetAbsoluteReference( sValue ); + sHRef = GetImport().GetAbsoluteReference( aIter.toString() ); break; case XML_ELEMENT(OFFICE, XML_NAME): - sName = sValue; + sName = aIter.toString(); break; case XML_ELEMENT(OFFICE, XML_TARGET_FRAME_NAME): - sTargetFrameName = sValue; + sTargetFrameName = aIter.toString(); break; case XML_ELEMENT(XLINK, XML_SHOW): - sShow = sValue; + sShow = aIter.toString(); break; case XML_ELEMENT(OFFICE, XML_SERVER_MAP): { bool bTmp(false); - if (::sax::Converter::convertBool( bTmp, sValue )) + if (::sax::Converter::convertBool( bTmp, aIter.toView() )) { bMap = bTmp; } diff --git a/xmloff/source/text/XMLTextListBlockContext.cxx b/xmloff/source/text/XMLTextListBlockContext.cxx index 28e86eeac159..9240413a004b 100644 --- a/xmloff/source/text/XMLTextListBlockContext.cxx +++ b/xmloff/source/text/XMLTextListBlockContext.cxx @@ -84,7 +84,6 @@ XMLTextListBlockContext::XMLTextListBlockContext( bool bIsContinueNumberingAttributePresent( false ); for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(XML, XML_ID): @@ -92,20 +91,20 @@ XMLTextListBlockContext::XMLTextListBlockContext( // xml:id is also the list ID (#i92221#) if ( mnLevel == 0 ) // root element { - msListId = sValue; + msListId = aIter.toString(); } break; case XML_ELEMENT(TEXT, XML_CONTINUE_NUMBERING): - mbRestartNumbering = !IsXMLToken(sValue, XML_TRUE); + mbRestartNumbering = !IsXMLToken(aIter, XML_TRUE); bIsContinueNumberingAttributePresent = true; break; case XML_ELEMENT(TEXT, XML_STYLE_NAME): - msListStyleName = sValue; + msListStyleName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_CONTINUE_LIST): if ( mnLevel == 0 ) // root element { - msContinueListId = sValue; + msContinueListId = aIter.toString(); } break; default: diff --git a/xmloff/source/text/XMLTextShapeImportHelper.cxx b/xmloff/source/text/XMLTextShapeImportHelper.cxx index 3c64ffd58e0a..74f8a7a49b26 100644 --- a/xmloff/source/text/XMLTextShapeImportHelper.cxx +++ b/xmloff/source/text/XMLTextShapeImportHelper.cxx @@ -88,14 +88,13 @@ void XMLTextShapeImportHelper::addShape( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - const OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(TEXT, XML_ANCHOR_TYPE): { TextContentAnchorType eNew; // OD 2004-06-01 #i26791# - allow all anchor types - if ( XMLAnchorTypePropHdl::convert( sValue, eNew ) ) + if ( XMLAnchorTypePropHdl::convert( aIter.toString(), eNew ) ) { eAnchorType = eNew; } @@ -104,13 +103,13 @@ void XMLTextShapeImportHelper::addShape( case XML_ELEMENT(TEXT, XML_ANCHOR_PAGE_NUMBER): { sal_Int32 nTmp; - if (::sax::Converter::convertNumber(nTmp, sValue, 1, SHRT_MAX)) + if (::sax::Converter::convertNumber(nTmp, aIter.toView(), 1, SHRT_MAX)) nPage = static_cast(nTmp); } break; case XML_ELEMENT(SVG, XML_Y): case XML_ELEMENT(SVG_COMPAT, XML_Y): - rImport.GetMM100UnitConverter().convertMeasureToCore( nY, sValue ); + rImport.GetMM100UnitConverter().convertMeasureToCore( nY, aIter.toString() ); break; } } diff --git a/xmloff/source/text/XMLTrackedChangesImportContext.cxx b/xmloff/source/text/XMLTrackedChangesImportContext.cxx index 630feb6a1db0..63e0d1497764 100644 --- a/xmloff/source/text/XMLTrackedChangesImportContext.cxx +++ b/xmloff/source/text/XMLTrackedChangesImportContext.cxx @@ -54,7 +54,7 @@ void XMLTrackedChangesImportContext::startFastElement( sal_Int32 /*nElement*/, if (aIter.getToken() == XML_ELEMENT(TEXT, XML_TRACK_CHANGES) ) { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, aIter.toString())) + if (::sax::Converter::convertBool(bTmp, aIter.toView())) { bTrackChanges = bTmp; } diff --git a/xmloff/source/text/txtdropi.cxx b/xmloff/source/text/txtdropi.cxx index b188f39b5bfe..66b219884b44 100644 --- a/xmloff/source/text/txtdropi.cxx +++ b/xmloff/source/text/txtdropi.cxx @@ -47,23 +47,21 @@ void XMLTextDropCapImportContext::ProcessAttrs( sal_Int32 nTmp; for (auto &aIter : sax_fastparser::castToFastAttributeList(xAttrList)) { - const OUString sValue = aIter.toString(); - switch( aIter.getToken() ) { case XML_ELEMENT(STYLE, XML_LINES): - if (::sax::Converter::convertNumber( nTmp, sValue, 0, 255 )) + if (::sax::Converter::convertNumber( nTmp, aIter.toView(), 0, 255 )) { aFormat.Lines = nTmp < 2 ? 0 : static_cast(nTmp); } break; case XML_ELEMENT(STYLE, XML_LENGTH): - if( IsXMLToken( sValue, XML_WORD ) ) + if( IsXMLToken( aIter, XML_WORD ) ) { bWholeWord = true; } - else if (::sax::Converter::convertNumber( nTmp, sValue, 1, 255 )) + else if (::sax::Converter::convertNumber( nTmp, aIter.toView(), 1, 255 )) { bWholeWord = false; aFormat.Count = static_cast(nTmp); @@ -72,14 +70,14 @@ void XMLTextDropCapImportContext::ProcessAttrs( case XML_ELEMENT(STYLE, XML_DISTANCE): if (GetImport().GetMM100UnitConverter().convertMeasureToCore( - nTmp, sValue, 0 )) + nTmp, aIter.toView(), 0 )) { aFormat.Distance = static_cast(nTmp); } break; case XML_ELEMENT(STYLE, XML_STYLE_NAME): - sStyleName = sValue; + sStyleName = aIter.toString(); break; default: diff --git a/xmloff/source/text/txtfldi.cxx b/xmloff/source/text/txtfldi.cxx index 53794de8a9f7..f0cc32b3a33c 100644 --- a/xmloff/source/text/txtfldi.cxx +++ b/xmloff/source/text/txtfldi.cxx @@ -2665,7 +2665,7 @@ void XMLDdeFieldDeclImportContext::startFastElement( case XML_ELEMENT(OFFICE, XML_AUTOMATIC_UPDATE): { bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, aIter.toString()) ) + if (::sax::Converter::convertBool(bTmp, aIter.toView()) ) { bUpdate = bTmp; } @@ -3445,19 +3445,18 @@ static bool lcl_ProcessLabel( bool bValid = false; for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch (aIter.getToken()) { case XML_ELEMENT(TEXT, XML_VALUE): { - rLabel = sValue; + rLabel = aIter.toString(); bValid = true; break; } case XML_ELEMENT(TEXT, XML_CURRENT_SELECTED): { bool bTmp(false); - if (::sax::Converter::convertBool( bTmp, sValue )) + if (::sax::Converter::convertBool( bTmp, aIter.toView() )) rIsSelected = bTmp; break; } diff --git a/xmloff/source/text/txtparai.cxx b/xmloff/source/text/txtparai.cxx index 733946a98b40..2319af663736 100644 --- a/xmloff/source/text/txtparai.cxx +++ b/xmloff/source/text/txtparai.cxx @@ -1637,38 +1637,37 @@ XMLParaContext::XMLParaContext( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(XML, XML_ID): - m_sXmlId = sValue; + m_sXmlId = aIter.toString(); bHaveXmlId = true; break; case XML_ELEMENT(XHTML, XML_ABOUT): - m_sAbout = sValue; + m_sAbout = aIter.toString(); m_bHaveAbout = true; break; case XML_ELEMENT(XHTML, XML_PROPERTY): - m_sProperty = sValue; + m_sProperty = aIter.toString(); break; case XML_ELEMENT(XHTML, XML_CONTENT): - m_sContent = sValue; + m_sContent = aIter.toString(); break; case XML_ELEMENT(XHTML, XML_DATATYPE): - m_sDatatype = sValue; + m_sDatatype = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_ID): - if (!bHaveXmlId) { m_sXmlId = sValue; } + if (!bHaveXmlId) { m_sXmlId = aIter.toString(); } break; case XML_ELEMENT(TEXT, XML_STYLE_NAME): - sStyleName = sValue; + sStyleName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_COND_STYLE_NAME): - aCondStyleName = sValue; + aCondStyleName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_OUTLINE_LEVEL): { - sal_Int32 nTmp = sValue.toInt32(); + sal_Int32 nTmp = aIter.toInt32(); if( nTmp > 0 ) { if( nTmp > 127 ) @@ -1682,28 +1681,27 @@ XMLParaContext::XMLParaContext( case XML_ELEMENT(LO_EXT, XML_OUTLINE_CONTENT_VISIBLE): { bool bBool(false); - if (::sax::Converter::convertBool(bBool, sValue)) + if (::sax::Converter::convertBool(bBool, aIter.toView())) mbOutlineContentVisible = bBool; } break; case XML_ELEMENT(TEXT, XML_IS_LIST_HEADER): { bool bBool(false); - if (::sax::Converter::convertBool(bBool, sValue)) + if (::sax::Converter::convertBool(bBool, aIter.toView())) bIsListHeader = bBool; } break; case XML_ELEMENT(TEXT, XML_RESTART_NUMBERING): { bool bBool(false); - if (::sax::Converter::convertBool(bBool, sValue)) + if (::sax::Converter::convertBool(bBool, aIter.toView())) bIsRestart = bBool; } break; case XML_ELEMENT(TEXT, XML_START_VALUE): { - nStartValue = sal::static_int_cast< sal_Int16 >( - sValue.toInt32()); + nStartValue = sal::static_int_cast< sal_Int16 >(aIter.toInt32()); } break; default: @@ -2031,25 +2029,24 @@ XMLNumberedParaContext::XMLNumberedParaContext( for( auto& aIter : sax_fastparser::castToFastAttributeList(xAttrList) ) { - OUString sValue = aIter.toString(); switch( aIter.getToken() ) { case XML_ELEMENT(XML, XML_ID): //FIXME: there is no UNO API for lists break; case XML_ELEMENT(TEXT, XML_LIST_ID): - m_ListId = sValue; + m_ListId = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_LEVEL): { - sal_Int32 nTmp = sValue.toInt32(); + sal_Int32 nTmp = aIter.toInt32(); if ( nTmp >= 1 && nTmp <= SHRT_MAX ) { m_Level = static_cast(nTmp) - 1; } } break; case XML_ELEMENT(TEXT, XML_STYLE_NAME): - StyleName = sValue; + StyleName = aIter.toString(); break; case XML_ELEMENT(TEXT, XML_CONTINUE_NUMBERING): // this attribute is deprecated @@ -2057,7 +2054,7 @@ XMLNumberedParaContext::XMLNumberedParaContext( break; case XML_ELEMENT(TEXT, XML_START_VALUE): { - sal_Int32 nTmp = sValue.toInt32(); + sal_Int32 nTmp = aIter.toInt32(); if ( nTmp >= 0 && nTmp <= SHRT_MAX ) { m_StartValue = static_cast(nTmp); } diff --git a/xmloff/source/text/txtvfldi.cxx b/xmloff/source/text/txtvfldi.cxx index f112fc88f11c..f9e9a8a2fdba 100644 --- a/xmloff/source/text/txtvfldi.cxx +++ b/xmloff/source/text/txtvfldi.cxx @@ -715,7 +715,7 @@ XMLVariableDeclImportContext::XMLVariableDeclImportContext( { sal_Int32 nLevel; bool const bRet = ::sax::Converter::convertNumber( - nLevel, aIter.toString(), 0, + nLevel, aIter.toView(), 0, GetImport().GetTextImport()->GetChapterNumbering()-> getCount()); if (bRet) -- cgit v1.2.3