summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCaolán McNamara <caolan.mcnamara@collabora.com>2024-01-06 19:37:42 +0000
committerCaolán McNamara <caolan.mcnamara@collabora.com>2024-01-06 20:42:38 +0100
commit2b72f2b3ef94452ae649fc6a44bec049f1acb173 (patch)
tree143d7fb15a10ca35790d010de4a78de8f4e4d30d
parentd0bf35d30f0662189f646a5c67d89715b725f12c (diff)
cid#1546421 COPY_INSTEAD_OF_MOVEHEADmaster
Change-Id: I33787047d5125252204aac78ed876bc102a1ec11 Reviewed-on: https://gerrit.libreoffice.org/c/cppunit/+/161722 Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com> Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
-rw-r--r--include/cppunit/tools/XmlElement.h8
-rw-r--r--src/cppunit/XmlElement.cpp8
2 files changed, 8 insertions, 8 deletions
diff --git a/include/cppunit/tools/XmlElement.h b/include/cppunit/tools/XmlElement.h
index 3478a35..d033aa2 100644
--- a/include/cppunit/tools/XmlElement.h
+++ b/include/cppunit/tools/XmlElement.h
@@ -79,14 +79,14 @@ public:
* \param attributeName Name of the attribute. Must not be an empty.
* \param value Value of the attribute.
*/
- void addAttribute( std::string attributeName,
- std::string value );
+ void addAttribute( const std::string& attributeName,
+ const std::string& value );
/*! \brief Adds an attribute with the specified numeric value.
* \param attributeName Name of the attribute. Must not be empty.
* \param numericValue Numeric value of the attribute.
*/
- void addAttribute( std::string attributeName,
+ void addAttribute( const std::string& attributeName,
int numericValue );
/*! \brief Adds a child element to the element.
@@ -125,7 +125,7 @@ private:
typedef std::pair<std::string,std::string> Attribute;
std::string attributesAsString() const;
- std::string escape( std::string value ) const;
+ std::string escape( const std::string& value ) const;
private:
std::string m_name;
diff --git a/src/cppunit/XmlElement.cpp b/src/cppunit/XmlElement.cpp
index be02385..3a59814 100644
--- a/src/cppunit/XmlElement.cpp
+++ b/src/cppunit/XmlElement.cpp
@@ -74,15 +74,15 @@ XmlElement::setContent( int numericContent )
void
-XmlElement::addAttribute( std::string attributeName,
- std::string value )
+XmlElement::addAttribute( const std::string& attributeName,
+ const std::string& value )
{
m_attributes.push_back( Attribute( attributeName, value ) );
}
void
-XmlElement::addAttribute( std::string attributeName,
+XmlElement::addAttribute( const std::string& attributeName,
int numericValue )
{
addAttribute( attributeName, StringTools::toString( numericValue ) );
@@ -194,7 +194,7 @@ XmlElement::attributesAsString() const
std::string
-XmlElement::escape( std::string value ) const
+XmlElement::escape( const std::string& value ) const
{
std::string escaped;
for ( unsigned int index =0; index < value.length(); ++index )