Skip to content
Snippets Groups Projects
Commit 727b3431 authored by Quanah Gibson-Mount's avatar Quanah Gibson-Mount
Browse files

getURLString() is const now to avoid uneeded copying when using UrlLists

parent 8a1e7a4b
No related branches found
No related tags found
No related merge requests found
...@@ -70,7 +70,7 @@ void LDAPUrl::setScope( const std::string &scope ) ...@@ -70,7 +70,7 @@ void LDAPUrl::setScope( const std::string &scope )
regenerate = true; regenerate = true;
} }
const string& LDAPUrl::getURLString() const string& LDAPUrl::getURLString() const
{ {
if (regenerate){ if (regenerate){
this->components2Url(); this->components2Url();
...@@ -323,7 +323,7 @@ void LDAPUrl::string2list(const std::string &src, StringList& sl, ...@@ -323,7 +323,7 @@ void LDAPUrl::string2list(const std::string &src, StringList& sl,
} }
void LDAPUrl::components2Url() void LDAPUrl::components2Url() const
{ {
std::ostringstream url; std::ostringstream url;
std::string encoded = ""; std::string encoded = "";
...@@ -395,7 +395,7 @@ void LDAPUrl::components2Url() ...@@ -395,7 +395,7 @@ void LDAPUrl::components2Url()
void LDAPUrl::percentEncode( const std::string &src, void LDAPUrl::percentEncode( const std::string &src,
std::string &dest, std::string &dest,
int flags) int flags) const
{ {
std::ostringstream o; std::ostringstream o;
o.setf(std::ios::hex, std::ios::basefield); o.setf(std::ios::hex, std::ios::basefield);
......
...@@ -58,7 +58,7 @@ class LDAPUrl{ ...@@ -58,7 +58,7 @@ class LDAPUrl{
/** /**
* @return The complete URL as a string * @return The complete URL as a string
*/ */
const std::string& getURLString(); const std::string& getURLString() const;
/** /**
* Set the URL member attribute * Set the URL member attribute
...@@ -140,7 +140,7 @@ class LDAPUrl{ ...@@ -140,7 +140,7 @@ class LDAPUrl{
*/ */
void percentEncode( const std::string& src, void percentEncode( const std::string& src,
std::string& dest, std::string& dest,
int flags=0 ); int flags=0 ) const;
protected : protected :
/** /**
...@@ -158,13 +158,13 @@ class LDAPUrl{ ...@@ -158,13 +158,13 @@ class LDAPUrl{
* (this function is mostly for internal use and gets called * (this function is mostly for internal use and gets called
* automatically whenever necessary) * automatically whenever necessary)
*/ */
void components2Url(); void components2Url() const;
void string2list(const std::string &src, StringList& sl, void string2list(const std::string &src, StringList& sl,
bool percentDecode=false); bool percentDecode=false);
protected : protected :
bool regenerate; mutable bool regenerate;
int m_Port; int m_Port;
int m_Scope; int m_Scope;
std::string m_Host; std::string m_Host;
...@@ -172,7 +172,7 @@ class LDAPUrl{ ...@@ -172,7 +172,7 @@ class LDAPUrl{
std::string m_Filter; std::string m_Filter;
StringList m_Attrs; StringList m_Attrs;
StringList m_Extensions; StringList m_Extensions;
std::string m_urlString; mutable std::string m_urlString;
std::string m_Scheme; std::string m_Scheme;
enum mode { base, attrs, scope, filter, extensions }; enum mode { base, attrs, scope, filter, extensions };
}; };
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment