Hi, I have submitted a patch for review:
https://gerrit.libreoffice.org/3142 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/42/3142/1 fdo#38838 : Replacement of String with OUString in core/chart2 Change-Id: I7abf9649a0c5997cebfe3463ce9b633ab59bf3b0 --- M chart2/source/controller/dialogs/DataBrowser.cxx M chart2/source/controller/dialogs/DataBrowser.hxx M chart2/source/controller/dialogs/res_ErrorBar.cxx M chart2/source/controller/dialogs/tp_DataSource.cxx M chart2/source/controller/main/ChartController_Insert.cxx M chart2/source/controller/main/ChartController_Tools.cxx 6 files changed, 89 insertions(+), 89 deletions(-) diff --git a/chart2/source/controller/dialogs/DataBrowser.cxx b/chart2/source/controller/dialogs/DataBrowser.cxx index a291c55..ca424c8 100644 --- a/chart2/source/controller/dialogs/DataBrowser.cxx +++ b/chart2/source/controller/dialogs/DataBrowser.cxx @@ -160,7 +160,7 @@ void SetWidth( sal_Int32 nWidth ); void SetChartType( const Reference< chart2::XChartType > & xChartType, bool bSwapXAndYAxis ); - void SetSeriesName( const String & rName ); + void SetSeriesName( const OUString & rName ); void SetRange( sal_Int32 nStartCol, sal_Int32 nEndCol ); void SetPixelPosX( sal_Int32 nPos ); @@ -309,7 +309,7 @@ m_spSymbol->SetImage( GetChartTypeImage( xChartType, bSwapXAndYAxis ) ); } -void SeriesHeader::SetSeriesName( const String & rName ) +void SeriesHeader::SetSeriesName( const OUString & rName ) { m_spSeriesName->SetText( rName ); } @@ -616,7 +616,7 @@ spHeader->SetColor( Color( nColor )); spHeader->SetChartType( aIt->m_xChartType, aIt->m_bSwapXAndYAxis ); spHeader->SetSeriesName( - String( DataSeriesHelper::getDataSeriesLabel( + OUString( DataSeriesHelper::getDataSeriesLabel( aIt->m_xDataSeries, (aIt->m_xChartType.is() ? aIt->m_xChartType->getRoleOfSequenceForSeriesLabel() : @@ -635,22 +635,22 @@ Invalidate(); } -String DataBrowser::GetColString( sal_Int32 nColumnId ) const +OUString DataBrowser::GetColString( sal_Int32 nColumnId ) const { OSL_ASSERT( m_apDataBrowserModel.get()); if( nColumnId > 0 ) - return String( m_apDataBrowserModel->getRoleOfColumn( static_cast< sal_Int32 >( nColumnId ) - 1 )); - return String(); + return OUString( m_apDataBrowserModel->getRoleOfColumn( static_cast< sal_Int32 >( nColumnId ) - 1 )); + return OUString(); } -String DataBrowser::GetRowString( sal_Int32 nRow ) const +OUString DataBrowser::GetRowString( sal_Int32 nRow ) const { return OUString::valueOf(nRow + 1); } String DataBrowser::GetCellText( long nRow, sal_uInt16 nColumnId ) const { - String aResult; + OUString aResult; if( nColumnId == 0 ) { @@ -669,7 +669,7 @@ if( ! ::rtl::math::isNan( fData ) && m_spNumberFormatterWrapper.get() ) - aResult = String( m_spNumberFormatterWrapper->getFormattedString( + aResult = OUString( m_spNumberFormatterWrapper->getFormattedString( GetNumberFormatKey( nRow, nColumnId ), fData, nLabelColor, bColorChanged )); } @@ -686,14 +686,14 @@ bool bColorChanged = false; sal_Int32 nDateNumberFormat = DiagramHelper::getDateNumberFormat( Reference< util::XNumberFormatsSupplier >( m_xChartDoc, uno::UNO_QUERY) ); if( ! ::rtl::math::isNan( fDouble ) && m_spNumberFormatterWrapper.get() ) - aResult = String( m_spNumberFormatterWrapper->getFormattedString( + aResult = OUString( m_spNumberFormatterWrapper->getFormattedString( nDateNumberFormat, fDouble, nLabelColor, bColorChanged )); } } else { OSL_ASSERT( m_apDataBrowserModel->getCellType( nColIndex, nRow ) == DataBrowserModel::TEXT ); - aResult = m_apDataBrowserModel->getCellText( nColIndex, nRow ); + aResult = (OUString)m_apDataBrowserModel->getCellText( nColIndex, nRow ); } } @@ -800,9 +800,9 @@ { sal_uInt32 nDummy = 0; double fDummy = 0.0; - String aText( m_aNumberEditField.GetText()); + OUString aText( m_aNumberEditField.GetText()); - if( aText.Len() > 0 && + if( aText.getLength() > 0 && m_spNumberFormatterWrapper.get() && m_spNumberFormatterWrapper->getSvNumberFormatter() && ! m_spNumberFormatterWrapper->getSvNumberFormatter()->IsNumberFormat( @@ -1082,9 +1082,9 @@ { if( rController == m_rTextEditController ) { - String aText( GetCellText( nRow, nCol ) ); + OUString aText( GetCellText( nRow, nCol ) ); m_aTextEditField.SetText( aText ); - m_aTextEditField.SetSelection( Selection( 0, aText.Len() )); + m_aTextEditField.SetSelection( Selection( 0, aText.getLength() )); } else if( rController == m_rNumberEditController ) { @@ -1094,8 +1094,8 @@ m_aNumberEditField.SetTextValue( String()); else m_aNumberEditField.SetValue( GetCellNumber( nRow, nCol ) ); - String aText( m_aNumberEditField.GetText()); - m_aNumberEditField.SetSelection( Selection( 0, aText.Len())); + OUString aText( m_aNumberEditField.GetText()); + m_aNumberEditField.SetSelection( Selection( 0, aText.getLength())); } else { @@ -1153,10 +1153,10 @@ { sal_uInt32 nDummy = 0; double fDummy = 0.0; - String aText( m_aNumberEditField.GetText()); + OUString aText( m_aNumberEditField.GetText()); // an empty string is valid, if no numberformatter exists, all // values are treated as valid - if( aText.Len() > 0 && pSvNumberFormatter && + if( aText.getLength() > 0 && pSvNumberFormatter && ! pSvNumberFormatter->IsNumberFormat( aText, nDummy, fDummy ) ) { bChangeValid = sal_False; @@ -1262,7 +1262,7 @@ spHeader->SetColor( Color( nColor )); spHeader->SetChartType( aIt->m_xChartType, aIt->m_bSwapXAndYAxis ); spHeader->SetSeriesName( - String( DataSeriesHelper::getDataSeriesLabel( + OUString( DataSeriesHelper::getDataSeriesLabel( aIt->m_xDataSeries, (aIt->m_xChartType.is() ? aIt->m_xChartType->getRoleOfSequenceForSeriesLabel() : diff --git a/chart2/source/controller/dialogs/DataBrowser.hxx b/chart2/source/controller/dialogs/DataBrowser.hxx index f155ab3..3d5a5b8 100644 --- a/chart2/source/controller/dialogs/DataBrowser.hxx +++ b/chart2/source/controller/dialogs/DataBrowser.hxx @@ -181,8 +181,8 @@ void RenewTable(); void ImplAdjustHeaderControls(); - String GetColString( sal_Int32 nColumnId ) const; - String GetRowString( sal_Int32 nRow ) const; + OUString GetColString( sal_Int32 nColumnId ) const; + OUString GetRowString( sal_Int32 nRow ) const; DECL_LINK( SeriesHeaderGotFocus, impl::SeriesHeaderEdit* ); DECL_LINK( SeriesHeaderChanged, impl::SeriesHeaderEdit* ); diff --git a/chart2/source/controller/dialogs/res_ErrorBar.cxx b/chart2/source/controller/dialogs/res_ErrorBar.cxx index 31f945a..8c1b2f4 100644 --- a/chart2/source/controller/dialogs/res_ErrorBar.cxx +++ b/chart2/source/controller/dialogs/res_ErrorBar.cxx @@ -268,7 +268,7 @@ ( m_aRbFunction.IsChecked()) && ( m_aLbFunction.GetSelectEntryPos() == CHART_LB_FUNCTION_ERROR_MARGIN )); bool bIsPercentage( m_aRbPercent.IsChecked() || bIsErrorMargin ); - String aCustomUnit; + OUString aCustomUnit; if( bIsPercentage ) { @@ -458,12 +458,12 @@ if( pButton == &m_aIbRangePositive ) { m_pCurrentRangeChoosingField = &m_aEdRangePositive; - aUIString = String( SchResId( STR_DATA_SELECT_RANGE_FOR_POSITIVE_ERRORBARS )); + aUIString = OUString( SchResId( STR_DATA_SELECT_RANGE_FOR_POSITIVE_ERRORBARS )); } else { m_pCurrentRangeChoosingField = &m_aEdRangeNegative; - aUIString = String( SchResId( STR_DATA_SELECT_RANGE_FOR_NEGATIVE_ERRORBARS )); + aUIString = OUString( SchResId( STR_DATA_SELECT_RANGE_FOR_NEGATIVE_ERRORBARS )); } assert( m_pParentDialog ); @@ -602,7 +602,7 @@ m_bRangePosUnique = ( aState != SFX_ITEM_DONTCARE ); if( aState == SFX_ITEM_SET ) { - String sRangePositive = (static_cast< const SfxStringItem * >( pPoolItem ))->GetValue(); + OUString sRangePositive = (static_cast< const SfxStringItem * >( pPoolItem ))->GetValue(); m_aEdRangePositive.SetText( sRangePositive ); } @@ -610,11 +610,11 @@ m_bRangeNegUnique = ( aState != SFX_ITEM_DONTCARE ); if( aState == SFX_ITEM_SET ) { - String sRangeNegative = (static_cast< const SfxStringItem * >( pPoolItem ))->GetValue(); + OUString sRangeNegative = (static_cast< const SfxStringItem * >( pPoolItem ))->GetValue(); m_aEdRangeNegative.SetText( sRangeNegative ); if( m_eErrorKind == CHERROR_RANGE && - sRangeNegative.Len() > 0 && - sRangeNegative.Equals( m_aEdRangePositive.GetText())) + sRangeNegative.getLength() > 0 && + sRangeNegative == m_aEdRangePositive.GetText() ) m_aCbSyncPosNeg.Check(); } @@ -632,13 +632,13 @@ { if( m_eErrorKind == CHERROR_RANGE ) { - String aPosRange; - String aNegRange; + OUString aPosRange; + OUString aNegRange; if( m_bHasInternalDataProvider ) { // the strings aPosRange/aNegRange have to be set to a non-empty // arbitrary string to generate error-bar sequences - aPosRange.Assign( sal_Unicode('x')); + aPosRange = OUString("x"); aNegRange = aPosRange; } else diff --git a/chart2/source/controller/dialogs/tp_DataSource.cxx b/chart2/source/controller/dialogs/tp_DataSource.cxx index 35fe350..11f1d7a 100644 --- a/chart2/source/controller/dialogs/tp_DataSource.cxx +++ b/chart2/source/controller/dialogs/tp_DataSource.cxx @@ -60,15 +60,15 @@ const OUString lcl_aLabelRole( "label" ); -String lcl_GetRoleLBEntry( +OUString lcl_GetRoleLBEntry( const OUString & rRole, const OUString & rRange ) { - String aEntry( rRole ); - aEntry += '\t'; - aEntry += String( + OUString aEntry( rRole ); + aEntry += "\t"; + aEntry += OUString( ::chart::DialogModel::ConvertRoleFromInternalToUI( rRole )); - aEntry += '\t'; - aEntry += String( rRange ); + aEntry += "\t"; + aEntry += OUString( rRange ); return aEntry; } @@ -91,10 +91,10 @@ pEntry->m_xDataSeries.is() && pEntry->m_xChartType.is()) { - String aLabel( ::chart::DataSeriesHelper::getDataSeriesLabel( + OUString aLabel( ::chart::DataSeriesHelper::getDataSeriesLabel( pEntry->m_xDataSeries, pEntry->m_xChartType->getRoleOfSequenceForSeriesLabel())); - if( aLabel.Len()) + if( aLabel.getLength()) { rOutListBox.SetEntryText( pEntry, aLabel ); bResult = true; @@ -469,23 +469,23 @@ for( ::std::vector< DialogModel::tSeriesWithChartTypeByName >::const_iterator aIt = aSeries.begin(); aIt != aSeries.end(); ++aIt ) { - String aLabel( (*aIt).first ); - if( !aLabel.Len()) + OUString aLabel( (*aIt).first ); + if( aLabel.isEmpty()) { if( nUnnamedSeriesIndex > 1 ) { - OUString aResString( String( ::chart::SchResId( STR_DATA_UNNAMED_SERIES_WITH_INDEX ))); + OUString aResString( OUString( ::chart::SchResId( STR_DATA_UNNAMED_SERIES_WITH_INDEX ))); // replace index of unnamed series const OUString aReplacementStr( "%NUMBER" ); sal_Int32 nIndex = aResString.indexOf( aReplacementStr ); if( nIndex != -1 ) - aLabel = String( aResString.replaceAt( + aLabel = OUString( aResString.replaceAt( nIndex, aReplacementStr.getLength(), OUString::valueOf(nUnnamedSeriesIndex))); } - if( aLabel.Len() == 0 ) - aLabel = String( ::chart::SchResId( STR_DATA_UNNAMED_SERIES )); + if( aLabel.isEmpty() ) + aLabel = OUString( ::chart::SchResId( STR_DATA_UNNAMED_SERIES )); ++nUnnamedSeriesIndex; } diff --git a/chart2/source/controller/main/ChartController_Insert.cxx b/chart2/source/controller/main/ChartController_Insert.cxx index 8c7c733..329d2ff 100644 --- a/chart2/source/controller/main/ChartController_Insert.cxx +++ b/chart2/source/controller/main/ChartController_Insert.cxx @@ -111,7 +111,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_AXES ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_AXES ))), m_xUndoManager ); try @@ -149,7 +149,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_GRIDS ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_GRIDS ))), m_xUndoManager ); try @@ -186,7 +186,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_TITLES ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_TITLES ))), m_xUndoManager ); try @@ -217,7 +217,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_LEGEND ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_LEGEND ))), m_xUndoManager ); LegendHelper::hideLegend( getModel() ); @@ -228,7 +228,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_LEGEND ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_LEGEND ))), m_xUndoManager ); Reference< chart2::XLegend > xLegend = LegendHelper::showLegend( getModel(), m_xCC ); @@ -239,7 +239,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_LEGEND ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_LEGEND ))), m_xUndoManager ); try @@ -270,7 +270,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_DATALABELS ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_DATALABELS ))), m_xUndoManager ); //if a series is selected insert labels for that series only: @@ -332,7 +332,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_AVERAGE_LINE ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_AVERAGE_LINE ))), m_xUndoManager ); lcl_InsertMeanValueLine( m_xCC ).operator()( ObjectIdentifier::getDataSeriesForCID( m_aSelection.getSelectedCID(), getModel() )); @@ -343,7 +343,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_AVERAGE_LINE ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_AVERAGE_LINE ))), m_xUndoManager ); uno::Reference< chart2::XDataSeries > xSeries( @@ -416,7 +416,7 @@ { UndoLiveUpdateGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_CURVE ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_CURVE ))), m_xUndoManager ); // add a linear curve @@ -475,7 +475,7 @@ UndoLiveUpdateGuard aUndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::INSERT, - String( SchResId( bYError ? STR_OBJECT_ERROR_BARS_Y : STR_OBJECT_ERROR_BARS_X ))), + OUString( SchResId( bYError ? STR_OBJECT_ERROR_BARS_Y : STR_OBJECT_ERROR_BARS_X ))), m_xUndoManager ); // add error bars with standard deviation @@ -582,7 +582,7 @@ // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_CURVE_EQUATION ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_CURVE_EQUATION ))), m_xUndoManager ); xEqProp->setPropertyValue( "ShowEquation", uno::makeAny( true )); xEqProp->setPropertyValue( "ShowCorrelationCoefficient", uno::makeAny( bInsertR2 )); @@ -599,7 +599,7 @@ { UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_CURVE_EQUATION ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_CURVE_EQUATION ))), m_xUndoManager ); xEqProp->setPropertyValue( "ShowCorrelationCoefficient", uno::makeAny( true )); aUndoGuard.commit(); @@ -614,7 +614,7 @@ { UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_CURVE_EQUATION ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_CURVE_EQUATION ))), m_xUndoManager ); xEqProp->setPropertyValue( "ShowCorrelationCoefficient", uno::makeAny( false )); aUndoGuard.commit(); @@ -629,7 +629,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_AVERAGE_LINE ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_AVERAGE_LINE ))), m_xUndoManager ); RegressionCurveHelper::removeMeanValueLine( xRegCurveCnt ); aUndoGuard.commit(); @@ -644,7 +644,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_CURVE ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_CURVE ))), m_xUndoManager ); RegressionCurveHelper::removeAllExceptMeanValueLine( xRegCurveCnt ); aUndoGuard.commit(); @@ -659,7 +659,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_CURVE_EQUATION ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_CURVE_EQUATION ))), m_xUndoManager ); RegressionCurveHelper::removeEquations( xRegCurveCnt ); aUndoGuard.commit(); @@ -674,7 +674,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_CURVE ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_CURVE ))), m_xUndoManager ); StatisticsHelper::removeErrorBars( xDataSeries, bYError ); aUndoGuard.commit(); @@ -688,7 +688,7 @@ if( xSeries.is() ) { UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::INSERT, - String( SchResId( STR_OBJECT_DATALABELS ))), + OUString( SchResId( STR_OBJECT_DATALABELS ))), m_xUndoManager ); DataSeriesHelper::insertDataLabelsToSeriesAndAllPoints( xSeries ); aUndoGuard.commit(); @@ -698,7 +698,7 @@ void ChartController::executeDispatch_InsertDataLabel() { UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::INSERT, - String( SchResId( STR_OBJECT_LABEL ))), + OUString( SchResId( STR_OBJECT_LABEL ))), m_xUndoManager ); DataSeriesHelper::insertDataLabelToPoint( ObjectIdentifier::getObjectPropertySet( m_aSelection.getSelectedCID(), getModel() ) ); aUndoGuard.commit(); @@ -711,7 +711,7 @@ if( xSeries.is() ) { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::DELETE, - String( SchResId( STR_OBJECT_DATALABELS ))), + OUString( SchResId( STR_OBJECT_DATALABELS ))), m_xUndoManager ); DataSeriesHelper::deleteDataLabelsFromSeriesAndAllPoints( xSeries ); aUndoGuard.commit(); @@ -721,7 +721,7 @@ void ChartController::executeDispatch_DeleteDataLabel() { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::DELETE, - String( SchResId( STR_OBJECT_LABEL ))), + OUString( SchResId( STR_OBJECT_LABEL ))), m_xUndoManager ); DataSeriesHelper::deleteDataLabelsFromPoint( ObjectIdentifier::getObjectPropertySet( m_aSelection.getSelectedCID(), getModel() ) ); aUndoGuard.commit(); @@ -730,7 +730,7 @@ void ChartController::executeDispatch_ResetAllDataPoints() { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::FORMAT, - String( SchResId( STR_OBJECT_DATAPOINTS ))), + OUString( SchResId( STR_OBJECT_DATAPOINTS ))), m_xUndoManager ); uno::Reference< chart2::XDataSeries > xSeries( ObjectIdentifier::getDataSeriesForCID( m_aSelection.getSelectedCID(), getModel() ), uno::UNO_QUERY ); if( xSeries.is() ) @@ -740,7 +740,7 @@ void ChartController::executeDispatch_ResetDataPoint() { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( ActionDescriptionProvider::FORMAT, - String( SchResId( STR_OBJECT_DATAPOINT ))), + OUString( SchResId( STR_OBJECT_DATAPOINT ))), m_xUndoManager ); uno::Reference< chart2::XDataSeries > xSeries( ObjectIdentifier::getDataSeriesForCID( m_aSelection.getSelectedCID(), getModel() ), uno::UNO_QUERY ); if( xSeries.is() ) @@ -759,7 +759,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_TITLE ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_TITLE ))), m_xUndoManager ); Reference< XAxis > xAxis = ObjectIdentifier::getAxisForCID( m_aSelection.getSelectedCID(), getModel() ); @@ -791,7 +791,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_AXIS ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_AXIS ))), m_xUndoManager ); try @@ -813,7 +813,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_AXIS ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_AXIS ))), m_xUndoManager ); try @@ -835,7 +835,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_GRID ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_GRID ))), m_xUndoManager ); try @@ -857,7 +857,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_GRID ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_GRID ))), m_xUndoManager ); try @@ -879,7 +879,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::INSERT, String( SchResId( STR_OBJECT_GRID ))), + ActionDescriptionProvider::INSERT, OUString( SchResId( STR_OBJECT_GRID ))), m_xUndoManager ); try @@ -903,7 +903,7 @@ { UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_GRID ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_GRID ))), m_xUndoManager ); try diff --git a/chart2/source/controller/main/ChartController_Tools.cxx b/chart2/source/controller/main/ChartController_Tools.cxx index 17581e3..bc16fc4 100644 --- a/chart2/source/controller/main/ChartController_Tools.cxx +++ b/chart2/source/controller/main/ChartController_Tools.cxx @@ -105,7 +105,7 @@ { ::chart::UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( ::chart::SchResId( STR_OBJECT_DATASERIES ))), + ActionDescriptionProvider::DELETE, OUString( ::chart::SchResId( STR_OBJECT_DATASERIES ))), xUndoManager ); Reference< chart2::XDiagram > xDiagram( ::chart::ChartModelHelper::findDiagram( xModel ) ); @@ -135,7 +135,7 @@ { ::chart::UndoGuard aUndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( ::chart::SchResId( STR_OBJECT_CURVE ))), + ActionDescriptionProvider::DELETE, OUString( ::chart::SchResId( STR_OBJECT_CURVE ))), xUndoManager ); ::chart::RegressionCurveHelper::removeAllExceptMeanValueLine( xRegCurveCnt ); bResult = true; @@ -184,7 +184,7 @@ { // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( - String( SchResId( STR_ACTION_REARRANGE_CHART )), + OUString( SchResId( STR_ACTION_REARRANGE_CHART )), m_xUndoManager ); ControllerLockGuard aCtlLockGuard( xModel ); @@ -238,7 +238,7 @@ SolarMutexGuard aSolarGuard; // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( - String( SchResId( STR_ACTION_SCALE_TEXT )), + OUString( SchResId( STR_ACTION_SCALE_TEXT )), m_xUndoManager ); ControllerLockGuard aCtlLockGuard( getModel() ); SAL_WNODEPRECATED_DECLARATIONS_PUSH @@ -620,7 +620,7 @@ // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_TITLE ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_TITLE ))), m_xUndoManager ); TitleHelper::removeTitle( ObjectIdentifier::getTitleTypeForCID( aCID ), getModel() ); @@ -639,7 +639,7 @@ // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_LEGEND ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_LEGEND ))), m_xUndoManager ); xLegendProp->setPropertyValue( "Show", uno::makeAny( false )); bReturn = true; @@ -679,7 +679,7 @@ // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_AVERAGE_LINE ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_AVERAGE_LINE ))), m_xUndoManager ); RegressionCurveHelper::removeMeanValueLine( xRegCurveCnt ); bReturn = true; @@ -702,7 +702,7 @@ // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( STR_OBJECT_CURVE_EQUATION ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( STR_OBJECT_CURVE_EQUATION ))), m_xUndoManager ); { ControllerLockGuard aCtlLockGuard( xModel ); @@ -736,7 +736,7 @@ // using assignment for broken gcc 3.3 UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, String( SchResId( nId ))), + ActionDescriptionProvider::DELETE, OUString( SchResId( nId ))), m_xUndoManager ); { ControllerLockGuard aCtlLockGuard( xModel ); @@ -759,7 +759,7 @@ { UndoGuard aUndoGuard = UndoGuard( ActionDescriptionProvider::createDescription( - ActionDescriptionProvider::DELETE, ::rtl::OUString( String( + ActionDescriptionProvider::DELETE, ::rtl::OUString( OUString( SchResId( aObjectType == OBJECTTYPE_DATA_LABEL ? STR_OBJECT_LABEL : STR_OBJECT_DATALABELS )))), m_xUndoManager ); chart2::DataPointLabel aLabel; @@ -825,7 +825,7 @@ { Reference< frame::XModel > xModel( getModel() ); UndoGuard aUndoGuard = UndoGuard( - String( SchResId( STR_ACTION_TOGGLE_LEGEND )), m_xUndoManager ); + OUString( SchResId( STR_ACTION_TOGGLE_LEGEND )), m_xUndoManager ); Reference< beans::XPropertySet > xLegendProp( LegendHelper::getLegend( xModel ), uno::UNO_QUERY ); bool bChanged = false; if( xLegendProp.is()) @@ -859,7 +859,7 @@ { Reference< frame::XModel > xModel( getModel() ); UndoGuard aUndoGuard = UndoGuard( - String( SchResId( STR_ACTION_TOGGLE_GRID_HORZ )), m_xUndoManager ); + OUString( SchResId( STR_ACTION_TOGGLE_GRID_HORZ )), m_xUndoManager ); Reference< chart2::XDiagram > xDiagram( ChartModelHelper::findDiagram( getModel() )); if( xDiagram.is()) { -- To view, visit https://gerrit.libreoffice.org/3142 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7abf9649a0c5997cebfe3463ce9b633ab59bf3b0 Gerrit-PatchSet: 1 Gerrit-Project: core Gerrit-Branch: master Gerrit-Owner: Prashant Pandey <prashant3.yi...@gmail.com> _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice