[PATCH] String=>OUString svtools grfmgr + fmtfield
Norbert Thiebaud (via Code Review)
gerrit at gerrit.libreoffice.org
Thu Jun 13 06:20:18 PDT 2013
Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/4261
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/61/4261/1
String=>OUString svtools grfmgr + fmtfield
Change-Id: Ie3d49014cb5edfbb709bea97b6f1332351fc6f3c
---
M include/svtools/fmtfield.hxx
M include/svtools/grfmgr.hxx
M svtools/source/control/fmtfield.cxx
M svtools/source/graphic/grfmgr.cxx
4 files changed, 27 insertions(+), 27 deletions(-)
diff --git a/include/svtools/fmtfield.hxx b/include/svtools/fmtfield.hxx
index 6fab7e4..df7b741 100644
--- a/include/svtools/fmtfield.hxx
+++ b/include/svtools/fmtfield.hxx
@@ -52,7 +52,7 @@
};
protected:
- String m_sLastValidText;
+ OUString m_sLastValidText;
// hat nichts mit dem current value zu tun, ist der letzte Text, der waehrend einer Eingabe als gueltig erkannt
// wurde (also durch CheckText geprueft, nicht durch den Formatter gejagt)
Selection m_aLastSelection;
@@ -84,8 +84,8 @@
// ausgibt
sal_Bool m_bTreatAsNumber;
// und mit den folgenden Members koennen wir das Ganze hier auch zur formatierten Text-Ausgabe benutzen ...
- String m_sCurrentTextValue;
- String m_sDefaultText;
+ OUString m_sCurrentTextValue;
+ OUString m_sDefaultText;
// die bei der letzten Ausgabe-Operation vom Formatter gelieferte Farbe (nicht dass wir sie beachten wuerden, aber
// man kann sie von aussen abfragen)
@@ -188,10 +188,10 @@
(der Text wird einfach nur durch den Formatter gejagt und dann gesetzt)
*/
void SetTextFormatted(const OUString& rText);
- String GetTextValue() const;
+ OUString GetTextValue() const;
void SetDefaultText(const OUString& rDefault) { m_sDefaultText = rDefault; }
- String GetDefaultText() const { return m_sDefaultText; }
+ OUString GetDefaultText() const { return m_sDefaultText; }
// die bei der letzten Ausgabe-Operation vom Formatter gelieferte Farbe (Ausgabe-Operationen werden getriggert durch
// SetValue, SetTextValue, SetTextFormatted, also indirekt eventuell auch durch SetMin-/-MaxValue)
diff --git a/include/svtools/grfmgr.hxx b/include/svtools/grfmgr.hxx
index 9a3bd8c..c479fa0 100644
--- a/include/svtools/grfmgr.hxx
+++ b/include/svtools/grfmgr.hxx
@@ -169,9 +169,9 @@
sal_uLong mnSizeBytes;
GraphicType meType;
GraphicManager* mpMgr;
- String* mpLink;
+ OUString* mpLink;
Link* mpSwapStreamHdl;
- String* mpUserData;
+ OUString* mpUserData;
Timer* mpSwapOutTimer;
GrfSimpleCacheObj* mpSimpleCache;
sal_uLong mnAnimationLoopCount;
@@ -343,7 +343,7 @@
const Graphic& GetGraphic() const;
void SetGraphic( const Graphic& rGraphic, const GraphicObject* pCopyObj = 0);
- void SetGraphic( const Graphic& rGraphic, const String& rLink );
+ void SetGraphic( const Graphic& rGraphic, const OUString& rLink );
/** Get graphic transformed according to given attributes
@@ -380,15 +380,15 @@
void SetAttr( const GraphicAttr& rAttr );
const GraphicAttr& GetAttr() const { return maAttr; }
- sal_Bool HasLink() const { return( mpLink != NULL && mpLink->Len() > 0 ); }
+ sal_Bool HasLink() const { return( mpLink != NULL && !mpLink->isEmpty()); }
void SetLink();
- void SetLink( const String& rLink );
- String GetLink() const;
+ void SetLink( const OUString& rLink );
+ OUString GetLink() const;
- sal_Bool HasUserData() const { return( mpUserData != NULL && mpUserData->Len() > 0 ); }
+ sal_Bool HasUserData() const { return( mpUserData != NULL && !mpUserData->isEmpty() ); }
void SetUserData();
- void SetUserData( const String& rUserData );
- String GetUserData() const;
+ void SetUserData( const OUString& rUserData );
+ OUString GetUserData() const;
OString GetUniqueID() const;
diff --git a/svtools/source/control/fmtfield.cxx b/svtools/source/control/fmtfield.cxx
index 7e04e8c..807ca54 100644
--- a/svtools/source/control/fmtfield.cxx
+++ b/svtools/source/control/fmtfield.cxx
@@ -451,7 +451,7 @@
}
//------------------------------------------------------------------------------
-String FormattedField::GetTextValue() const
+OUString FormattedField::GetTextValue() const
{
if (m_bValueDirty)
{
diff --git a/svtools/source/graphic/grfmgr.cxx b/svtools/source/graphic/grfmgr.cxx
index 136aa57..1a25822 100644
--- a/svtools/source/graphic/grfmgr.cxx
+++ b/svtools/source/graphic/grfmgr.cxx
@@ -81,8 +81,8 @@
SvDataCopyStream(),
maGraphic ( rGraphicObj.GetGraphic() ),
maAttr ( rGraphicObj.maAttr ),
- mpLink ( rGraphicObj.mpLink ? ( new String( *rGraphicObj.mpLink ) ) : NULL ),
- mpUserData ( rGraphicObj.mpUserData ? ( new String( *rGraphicObj.mpUserData ) ) : NULL )
+ mpLink ( rGraphicObj.mpLink ? ( new OUString( *rGraphicObj.mpLink ) ) : NULL ),
+ mpUserData ( rGraphicObj.mpUserData ? ( new OUString( *rGraphicObj.mpUserData ) ) : NULL )
{
ImplConstruct();
ImplAssignGraphicData();
@@ -330,8 +330,8 @@
maGraphic = rGraphicObj.GetGraphic();
maAttr = rGraphicObj.maAttr;
- mpLink = rGraphicObj.mpLink ? new String( *rGraphicObj.mpLink ) : NULL;
- mpUserData = rGraphicObj.mpUserData ? new String( *rGraphicObj.mpUserData ) : NULL;
+ mpLink = rGraphicObj.mpLink ? new OUString( *rGraphicObj.mpLink ) : NULL;
+ mpUserData = rGraphicObj.mpUserData ? new OUString( *rGraphicObj.mpUserData ) : NULL;
ImplAssignGraphicData();
mbAutoSwapped = sal_False;
mpMgr = rGraphicObj.mpMgr;
@@ -396,12 +396,12 @@
delete mpLink, mpLink = NULL;
}
-void GraphicObject::SetLink( const String& rLink )
+void GraphicObject::SetLink( const OUString& rLink )
{
- delete mpLink, mpLink = new String( rLink );
+ delete mpLink, mpLink = new OUString( rLink );
}
-String GraphicObject::GetLink() const
+OUString GraphicObject::GetLink() const
{
if( mpLink )
return *mpLink;
@@ -415,12 +415,12 @@
delete mpUserData, mpUserData = NULL;
}
-void GraphicObject::SetUserData( const String& rUserData )
+void GraphicObject::SetUserData( const OUString& rUserData )
{
- delete mpUserData, mpUserData = new String( rUserData );
+ delete mpUserData, mpUserData = new OUString( rUserData );
}
-String GraphicObject::GetUserData() const
+OUString GraphicObject::GetUserData() const
{
if( mpUserData )
return *mpUserData;
@@ -753,10 +753,10 @@
mpSwapOutTimer->Start();
}
-void GraphicObject::SetGraphic( const Graphic& rGraphic, const String& rLink )
+void GraphicObject::SetGraphic( const Graphic& rGraphic, const OUString& rLink )
{
SetGraphic( rGraphic );
- mpLink = new String( rLink );
+ mpLink = new OUString( rLink );
}
Graphic GraphicObject::GetTransformedGraphic( const Size& rDestSize, const MapMode& rDestMap, const GraphicAttr& rAttr ) const
--
To view, visit https://gerrit.libreoffice.org/4261
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie3d49014cb5edfbb709bea97b6f1332351fc6f3c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud <nthiebaud at gmail.com>
More information about the LibreOffice
mailing list