include/formula/IControlReferenceHandler.hxx | 2 - reportdesign/source/ui/dlg/Formula.cxx | 2 - reportdesign/source/ui/inc/Formula.hxx | 2 - sc/source/ui/formdlg/formula.cxx | 2 - sc/source/ui/inc/anyrefdg.hxx | 12 ++++----- sc/source/ui/inc/formula.hxx | 2 - sc/source/ui/miscdlgs/anyrefdg.cxx | 30 ++++++++++++------------ sc/source/ui/sidebar/CellLineStyleControl.cxx | 31 +++++++++---------------- sc/source/ui/sidebar/CellLineStyleControl.hxx | 3 -- sc/source/ui/sidebar/CellLineStyleValueSet.cxx | 10 +++----- sc/source/ui/sidebar/CellLineStyleValueSet.hxx | 6 +++- svx/source/svdraw/svdundo.cxx | 4 +-- 12 files changed, 49 insertions(+), 57 deletions(-)
New commits: commit eace5e0a1c4bab37699429447b1a17e3c9410503 Author: Caolán McNamara <caol...@redhat.com> Date: Fri Aug 2 16:49:24 2013 +0200 remove last uses of XubString in sc Change-Id: I44c42f0268b0d7c2e77ab7e807610d39b7b7ea56 diff --git a/include/formula/IControlReferenceHandler.hxx b/include/formula/IControlReferenceHandler.hxx index bee4bac..cc22e45 100644 --- a/include/formula/IControlReferenceHandler.hxx +++ b/include/formula/IControlReferenceHandler.hxx @@ -28,7 +28,7 @@ namespace formula class FORMULA_DLLPUBLIC SAL_NO_VTABLE IControlReferenceHandler { public: - virtual void ShowReference(const String& _sRef) = 0; + virtual void ShowReference(const OUString& _sRef) = 0; virtual void HideReference( sal_Bool bDoneRefMode = sal_True ) = 0; virtual void ReleaseFocus( RefEdit* pEdit, RefButton* pButton = NULL ) = 0; virtual void ToggleCollapsed( RefEdit* pEdit, RefButton* pButton = NULL ) = 0; diff --git a/reportdesign/source/ui/dlg/Formula.cxx b/reportdesign/source/ui/dlg/Formula.cxx index 84330da..6abf2f1 100644 --- a/reportdesign/source/ui/dlg/Formula.cxx +++ b/reportdesign/source/ui/dlg/Formula.cxx @@ -173,7 +173,7 @@ IFunctionManager* FormulaDialog::getFunctionManager() return m_aFunctionManager.get(); } // ----------------------------------------------------------------------------- -void FormulaDialog::ShowReference(const String& /*_sRef*/) +void FormulaDialog::ShowReference(const OUString& /*_sRef*/) { } // ----------------------------------------------------------------------------- diff --git a/reportdesign/source/ui/inc/Formula.hxx b/reportdesign/source/ui/inc/Formula.hxx index fcfbc6c..c551bbd 100644 --- a/reportdesign/source/ui/inc/Formula.hxx +++ b/reportdesign/source/ui/inc/Formula.hxx @@ -87,7 +87,7 @@ public: virtual ::std::auto_ptr<formula::FormulaTokenArray> convertToTokenArray(const ::com::sun::star::uno::Sequence< ::com::sun::star::sheet::FormulaToken >& _aTokenList); // IControlReferenceHandler - virtual void ShowReference(const String& _sRef); + virtual void ShowReference(const OUString& _sRef); virtual void HideReference( sal_Bool bDoneRefMode = sal_True ); virtual void ReleaseFocus( formula::RefEdit* pEdit, formula::RefButton* pButton = NULL ); virtual void ToggleCollapsed( formula::RefEdit* pEdit, formula::RefButton* pButton = NULL ); diff --git a/sc/source/ui/formdlg/formula.cxx b/sc/source/ui/formdlg/formula.cxx index 37ee0e2..25ae7d0 100644 --- a/sc/source/ui/formdlg/formula.cxx +++ b/sc/source/ui/formdlg/formula.cxx @@ -497,7 +497,7 @@ void ScFormulaDlg::showReference(const String& _sFormula) { ShowReference(_sFormula); } -void ScFormulaDlg::ShowReference(const String& _sFormula) +void ScFormulaDlg::ShowReference(const OUString& _sFormula) { m_aHelper.ShowReference(_sFormula); } diff --git a/sc/source/ui/inc/anyrefdg.hxx b/sc/source/ui/inc/anyrefdg.hxx index bba97a7..660de08 100644 --- a/sc/source/ui/inc/anyrefdg.hxx +++ b/sc/source/ui/inc/anyrefdg.hxx @@ -75,12 +75,12 @@ public: ScFormulaReferenceHelper(IAnyRefDialog* _pDlg,SfxBindings* _pBindings); ~ScFormulaReferenceHelper(); - void ShowSimpleReference( const XubString& rStr ); - void ShowFormulaReference( const XubString& rStr ); + void ShowSimpleReference(const OUString& rStr); + void ShowFormulaReference(const OUString& rStr); bool ParseWithNames( ScRangeList& rRanges, const String& rStr, ScDocument* pDoc ); void Init(); - void ShowReference( const XubString& rStr ); + void ShowReference(const OUString& rStr); void ReleaseFocus( formula::RefEdit* pEdit, formula::RefButton* pButton = NULL ); void HideReference( bool bDoneRefMode = true ); void RefInputStart( formula::RefEdit* pEdit, formula::RefButton* pButton = NULL ); @@ -136,8 +136,8 @@ protected: virtual void RefInputStart( formula::RefEdit* pEdit, formula::RefButton* pButton = NULL ); virtual void RefInputDone( sal_Bool bForced = false ); - void ShowSimpleReference( const XubString& rStr ); - void ShowFormulaReference( const XubString& rStr ); + void ShowSimpleReference(const OUString& rStr); + void ShowFormulaReference(const OUString& rStr); bool ParseWithNames( ScRangeList& rRanges, const String& rStr, ScDocument* pDoc ); @@ -155,7 +155,7 @@ public: virtual sal_Bool IsTableLocked() const; virtual sal_Bool IsDocAllowed( SfxObjectShell* pDocSh ) const; - virtual void ShowReference( const XubString& rStr ); + virtual void ShowReference(const OUString& rStr); virtual void HideReference( sal_Bool bDoneRefMode = sal_True ); virtual void ToggleCollapsed( formula::RefEdit* pEdit, formula::RefButton* pButton = NULL ); diff --git a/sc/source/ui/inc/formula.hxx b/sc/source/ui/inc/formula.hxx index 2082bc9..0a662e1 100644 --- a/sc/source/ui/inc/formula.hxx +++ b/sc/source/ui/inc/formula.hxx @@ -86,7 +86,7 @@ public: virtual sal_Bool Close(); // sc::IAnyRefDialog - virtual void ShowReference(const String& _sRef); + virtual void ShowReference(const OUString& _sRef); virtual void HideReference( sal_Bool bDoneRefMode = sal_True ); virtual void SetReference( const ScRange& rRef, ScDocument* pD ); diff --git a/sc/source/ui/miscdlgs/anyrefdg.cxx b/sc/source/ui/miscdlgs/anyrefdg.cxx index 69f6f53..1cd114e 100644 --- a/sc/source/ui/miscdlgs/anyrefdg.cxx +++ b/sc/source/ui/miscdlgs/anyrefdg.cxx @@ -112,7 +112,7 @@ void ScFormulaReferenceHelper::enableInput( bool bEnable ) } } // ----------------------------------------------------------------------------- -void ScFormulaReferenceHelper::ShowSimpleReference( const XubString& rStr ) +void ScFormulaReferenceHelper::ShowSimpleReference(const OUString& rStr) { if( bEnableColorRef ) { @@ -172,7 +172,7 @@ bool ScFormulaReferenceHelper::ParseWithNames( ScRangeList& rRanges, const Strin return !bError; } // ----------------------------------------------------------------------------- -void ScFormulaReferenceHelper::ShowFormulaReference( const XubString& rStr ) +void ScFormulaReferenceHelper::ShowFormulaReference(const OUString& rStr) { if( bEnableColorRef) { @@ -251,20 +251,20 @@ void ScFormulaReferenceHelper::HideReference( bool bDoneRefMode ) } } // ----------------------------------------------------------------------------- -void ScFormulaReferenceHelper::ShowReference( const XubString& rStr ) +void ScFormulaReferenceHelper::ShowReference(const OUString& rStr) { if( bEnableColorRef ) { - if( rStr.Search('(')!=STRING_NOTFOUND || - rStr.Search('+')!=STRING_NOTFOUND || - rStr.Search('*')!=STRING_NOTFOUND || - rStr.Search('-')!=STRING_NOTFOUND || - rStr.Search('/')!=STRING_NOTFOUND || - rStr.Search('&')!=STRING_NOTFOUND || - rStr.Search('<')!=STRING_NOTFOUND || - rStr.Search('>')!=STRING_NOTFOUND || - rStr.Search('=')!=STRING_NOTFOUND || - rStr.Search('^')!=STRING_NOTFOUND) + if( rStr.indexOf('(') != -1 || + rStr.indexOf('+') != -1 || + rStr.indexOf('*') != -1 || + rStr.indexOf('-') != -1 || + rStr.indexOf('/') != -1 || + rStr.indexOf('&') != -1 || + rStr.indexOf('<') != -1 || + rStr.indexOf('>') != -1 || + rStr.indexOf('=') != -1 || + rStr.indexOf('^') != -1 ) { ShowFormulaReference(rStr); } @@ -1014,9 +1014,9 @@ void ScRefHandler::HideReference( sal_Bool bDoneRefMode ) m_aHelper.HideReference( bDoneRefMode ); } // ----------------------------------------------------------------------------- -void ScRefHandler::ShowReference( const XubString& rStr ) +void ScRefHandler::ShowReference(const OUString& rStr) { - m_aHelper.ShowReference( rStr ); + m_aHelper.ShowReference(rStr); } // ----------------------------------------------------------------------------- void ScRefHandler::ReleaseFocus( formula::RefEdit* pEdit, formula::RefButton* pButton ) commit e9d8831d18d1120593137632a31b2dad9f8d20f7 Author: Caolán McNamara <caol...@redhat.com> Date: Fri Aug 2 16:37:18 2013 +0200 get rid of last use of UniString in LibreOffice of raw "UniString" :-), the rest use the #define UniString String alias Change-Id: I6c4ca3a6c2b425b9214691e03deda5dd1a8a11bf diff --git a/svx/source/svdraw/svdundo.cxx b/svx/source/svdraw/svdundo.cxx index 2d478e0..f2f3b37 100644 --- a/svx/source/svdraw/svdundo.cxx +++ b/svx/source/svdraw/svdundo.cxx @@ -255,9 +255,9 @@ void SdrUndoAttrObj::ensureStyleSheetInStyleSheetPool(SfxStyleSheetBasePool& rSt { // re-insert remembered style which was removed in the meantime. To do this // without assertion, do it without parent and set parent after insertion - const UniString aParent(rSheet.GetParent()); + const OUString aParent(rSheet.GetParent()); - rSheet.SetParent(UniString()); + rSheet.SetParent(OUString()); rStyleSheetPool.Insert(&rSheet); rSheet.SetParent(aParent); } commit fca99395ad3c3310aa04d5ff725b4a82fc17826e Author: Caolán McNamara <caol...@redhat.com> Date: Fri Aug 2 16:57:30 2013 +0200 no need for dynamic array of XubStrings Change-Id: I7a25917a2bec3ecd394444f057c892b3b02a4c3d diff --git a/sc/source/ui/sidebar/CellLineStyleControl.cxx b/sc/source/ui/sidebar/CellLineStyleControl.cxx index 339eff3..39e140a 100644 --- a/sc/source/ui/sidebar/CellLineStyleControl.cxx +++ b/sc/source/ui/sidebar/CellLineStyleControl.cxx @@ -36,18 +36,12 @@ CellLineStyleControl::CellLineStyleControl(Window* pParent, CellAppearanceProper mrCellAppearancePropertyPanel(rPanel), maPushButtonMoreOptions(this, ScResId(PB_OPTIONS)), maCellLineStyleValueSet(this, ScResId(VS_STYLE)), - mpStr(0), mbVSfocus(true) { Initialize(); FreeResource(); } -CellLineStyleControl::~CellLineStyleControl(void) -{ - delete[] mpStr; -} - void CellLineStyleControl::Initialize() { //maPushButtonMoreOptions.SetIcoPosX(2); @@ -63,21 +57,20 @@ void CellLineStyleControl::Initialize() maCellLineStyleValueSet.InsertItem(i); } - mpStr = new XubString[9]; - mpStr[0] = GetSettings().GetLocaleI18nHelper().GetNum( 5, 2 ).AppendAscii("pt"); - mpStr[1] = GetSettings().GetLocaleI18nHelper().GetNum( 250, 2 ).AppendAscii("pt"); - mpStr[2] = GetSettings().GetLocaleI18nHelper().GetNum( 400, 2 ).AppendAscii("pt"); - mpStr[3] = GetSettings().GetLocaleI18nHelper().GetNum( 500, 2 ).AppendAscii("pt"); - mpStr[4] = GetSettings().GetLocaleI18nHelper().GetNum( 110, 2 ).AppendAscii("pt"); - mpStr[5] = GetSettings().GetLocaleI18nHelper().GetNum( 260, 2 ).AppendAscii("pt"); - mpStr[6] = GetSettings().GetLocaleI18nHelper().GetNum( 450, 2 ).AppendAscii("pt"); - mpStr[7] = GetSettings().GetLocaleI18nHelper().GetNum( 505, 2 ).AppendAscii("pt"); - mpStr[8] = GetSettings().GetLocaleI18nHelper().GetNum( 750, 2 ).AppendAscii("pt"); - maCellLineStyleValueSet.SetUnit(mpStr); + maStr[0] = GetSettings().GetLocaleI18nHelper().GetNum( 5, 2 ).AppendAscii("pt"); + maStr[1] = GetSettings().GetLocaleI18nHelper().GetNum( 250, 2 ).AppendAscii("pt"); + maStr[2] = GetSettings().GetLocaleI18nHelper().GetNum( 400, 2 ).AppendAscii("pt"); + maStr[3] = GetSettings().GetLocaleI18nHelper().GetNum( 500, 2 ).AppendAscii("pt"); + maStr[4] = GetSettings().GetLocaleI18nHelper().GetNum( 110, 2 ).AppendAscii("pt"); + maStr[5] = GetSettings().GetLocaleI18nHelper().GetNum( 260, 2 ).AppendAscii("pt"); + maStr[6] = GetSettings().GetLocaleI18nHelper().GetNum( 450, 2 ).AppendAscii("pt"); + maStr[7] = GetSettings().GetLocaleI18nHelper().GetNum( 505, 2 ).AppendAscii("pt"); + maStr[8] = GetSettings().GetLocaleI18nHelper().GetNum( 750, 2 ).AppendAscii("pt"); + maCellLineStyleValueSet.SetUnit(&maStr[0]); - for(sal_uInt16 i = 1 ; i <= 9 ; i++) + for (sal_uInt16 i = 1; i <= CELL_LINE_STYLE_ENTRIES; ++i) { - maCellLineStyleValueSet.SetItemText(i, mpStr[i-1]); + maCellLineStyleValueSet.SetItemText(i, maStr[i-1]); } SetAllNoSel(); diff --git a/sc/source/ui/sidebar/CellLineStyleControl.hxx b/sc/source/ui/sidebar/CellLineStyleControl.hxx index d7d47da..d2da90c 100644 --- a/sc/source/ui/sidebar/CellLineStyleControl.hxx +++ b/sc/source/ui/sidebar/CellLineStyleControl.hxx @@ -32,7 +32,7 @@ private: CellAppearancePropertyPanel& mrCellAppearancePropertyPanel; PushButton maPushButtonMoreOptions; CellLineStyleValueSet maCellLineStyleValueSet; - XubString* mpStr; + OUString maStr[CELL_LINE_STYLE_ENTRIES]; /// bitfield bool mbVSfocus : 1; @@ -45,7 +45,6 @@ private: public: CellLineStyleControl(Window* pParent, CellAppearancePropertyPanel& rPanel); - virtual ~CellLineStyleControl(void); void GetFocus(); void SetLineStyleSelect(sal_uInt16 out, sal_uInt16 in, sal_uInt16 dis); diff --git a/sc/source/ui/sidebar/CellLineStyleValueSet.cxx b/sc/source/ui/sidebar/CellLineStyleValueSet.cxx index fc8e5e5..b093503 100644 --- a/sc/source/ui/sidebar/CellLineStyleValueSet.cxx +++ b/sc/source/ui/sidebar/CellLineStyleValueSet.cxx @@ -27,7 +27,6 @@ CellLineStyleValueSet::CellLineStyleValueSet( Window* pParent, const ResId& rRes nSelItem(0), bCusEnable(false) { - strUnit = new XubString[9]; SetColCount( 1 ); SetLineCount( 9); } @@ -35,14 +34,13 @@ CellLineStyleValueSet::CellLineStyleValueSet( Window* pParent, const ResId& rRes CellLineStyleValueSet::~CellLineStyleValueSet() { delete pVDev; - delete[] strUnit; } -void CellLineStyleValueSet::SetUnit(XubString* str) +void CellLineStyleValueSet::SetUnit(const OUString* str) { - for(int i = 0; i < 9; i++) + for (int i = 0; i < CELL_LINE_STYLE_ENTRIES; ++i) { - strUnit[i] = str[i]; + maStrUnit[i] = str[i]; } } @@ -121,7 +119,7 @@ void CellLineStyleValueSet::UserDraw( const UserDrawEvent& rUDEvt ) pDev->SetFont(aFont); Point aStart(aBLPos.X() + nRectWidth * 7 / 9 - 5 , aBLPos.Y() + nRectHeight/6); - pDev->DrawText(aStart, strUnit[ nItemId - 1 ]); //can't set TEXT_DRAW_ENDELLIPSIS here, or the text will disapear + pDev->DrawText(aStart, maStrUnit[nItemId - 1]); //can't set TEXT_DRAW_ENDELLIPSIS here, or the text will disapear //draw line if( nSelItem == nItemId ) diff --git a/sc/source/ui/sidebar/CellLineStyleValueSet.hxx b/sc/source/ui/sidebar/CellLineStyleValueSet.hxx index ebdfa53..6044b99 100644 --- a/sc/source/ui/sidebar/CellLineStyleValueSet.hxx +++ b/sc/source/ui/sidebar/CellLineStyleValueSet.hxx @@ -21,6 +21,8 @@ #include <svtools/valueset.hxx> #include <vcl/image.hxx> +#define CELL_LINE_STYLE_ENTRIES 9 + namespace sc { namespace sidebar { class CellLineStyleValueSet : public ValueSet @@ -28,14 +30,14 @@ class CellLineStyleValueSet : public ValueSet private: VirtualDevice* pVDev; sal_uInt16 nSelItem; - XubString* strUnit; + OUString maStrUnit[CELL_LINE_STYLE_ENTRIES]; Image imgCus; bool bCusEnable; public: CellLineStyleValueSet( Window* pParent, const ResId& rResId); virtual ~CellLineStyleValueSet(); - void SetUnit(XubString* str); + void SetUnit(const OUString* str); void SetSelItem(sal_uInt16 nSel); sal_uInt16 GetSelItem(); void SetImage(Image img);
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits