[Libreoffice-commits] core.git: framework/source

[Stephan Bergmann]

 framework/source/services/substitutepathvars.cxx |    7 +++++++
 1 file changed, 7 insertions(+)

New commits:
commit bae1e2aa636f144fe01cb8d9721e54525fed8b2c
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Sep 27 09:52:59 2016 +0200

    Add note
    
    Change-Id: I68af0bcfd6700ee40f2d0879e57de9d66c62c983

diff --git a/framework/source/services/substitutepathvars.cxx b/framework/source/services/substitutepathvars.cxx
index de2b9ca..dd1747a 100644
--- a/framework/source/services/substitutepathvars.cxx
+++ b/framework/source/services/substitutepathvars.cxx
@@ -1102,6 +1102,13 @@ throw ( RuntimeException )
                 {
                     // Special path variables as they can occur in the middle of a path. Only match if they
                     // describe a whole directory and not only a substring of a directory!
+                    // (Ideally, all substitutions should stick to syntactical
+                    // boundaries within the given URL, like not covering only
+                    // part of a URL path segment; however, at least when saving
+                    // an Impress document, one URL that is passed in is of the
+                    // form <file:///.../share/palette%3Bfile:///.../user/
+                    // config/standard.sob>, re-substituted to
+                    // <$(inst)/share/palette%3B$(user)/config/standard.sob>.)
                     const sal_Unicode* pStr = aURL.getStr();
 
                     if ( nPos > 0 )