[Libreoffice-commits] core.git: include/tools
Stephan Bergmann (via logerrit)
logerrit at kemper.freedesktop.org
Fri Jun 21 10:20:46 UTC 2019
include/tools/urlobj.hxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit f270bd15fed87a9cb0f8dfa73b34c617d66f59c2
Author: Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Fri Jun 21 10:01:06 2019 +0200
Commit: Stephan Bergmann <sbergman at redhat.com>
CommitDate: Fri Jun 21 12:19:27 2019 +0200
Fix INetURLObject::setName description
...after 359e0b47a0f96ffa595a0c38a5e5318d797812fe "loplugin:unuseddefaultparams"
had removed its nIndex and bIgnoreFinalSlash parameters denoting the segment
whose name to set.
(One would assume that a hierarchical URL always has a last segment, so the
"[if] a last segment does not exist, false is returned" part be removed from the
description. But for a hierarchical URL with an empty (i.e., just "/" in the
canonic form) path, setName("foo") actually returns false.)
Change-Id: I35b96f4687784648c141e16c0806fe49d80c940a
Reviewed-on: https://gerrit.libreoffice.org/74487
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman at redhat.com>
diff --git a/include/tools/urlobj.hxx b/include/tools/urlobj.hxx
index a1244bd7eba6..eb3b961c204b 100644
--- a/include/tools/urlobj.hxx
+++ b/include/tools/urlobj.hxx
@@ -569,7 +569,7 @@ public:
rtl_TextEncoding eCharset = RTL_TEXTENCODING_UTF8)
const;
- /** Set the name of a segment (preserving any parameters and any query or
+ /** Set the name of the last segment (preserving any parameters and any query or
fragment part).
@param rTheName The new name.
@@ -580,7 +580,7 @@ public:
@return True if the name has successfully been modified (and the
resulting URI is still valid). If the path is not hierarchical, or
- the specified segment does not exist, false is returned. If false is
+ a last segment does not exist, false is returned. If false is
returned, the object is not modified.
*/
bool setName(OUString const& rTheName,
More information about the Libreoffice-commits
mailing list