Fixes #779. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1803
-- Commit Summary --
* keyfile: Output unescaped filename if session file cannot be opened
-- File Changes --
M src/keyfile.c (2)
-- Patch Links --
https://github.com/geany/geany/pull/1803.patch https://github.com/geany/geany/pull/1803.diff
LGBI
Closed in 7fd73a06a7aab8582af36891f7b2a19dbe08188b
Closed #1803.
Does this address the secondary issue I raise here? https://github.com/geany/geany/issues/779#issuecomment-161904655
to which elextra replied:
"Technically all other filenames in config files probably should be encoded, but thats another issue."
Well, except that I actually meant to say "escaped" not "encoded" on #779 no it only addresses the output to the screen/terminal, as I said, the stored filenames are __intended__ to be escaped so they do not contain characters that cause problems for keyfiles when they are part of a list of items. In the [project] section paths are by themselves in an entry and so in practice they probably don't actually need escaping.
Which is why I only said technically all filenames should be escaped, it allows them to contain any Unicode character without caring what the encoding is. But since nobody seems to be doing that, its not broke, so don't fix it :)
@J-Dunn: no, this PR only fixes the issue which #779 directly addresses. As @elextr wrote the path encoding in the config file is correct. It only would be a bug if you see such enocidng in an output.
@elextr: no, only purely changed what PR #779 addresses and won't do anything else in this PR.
github-comments@lists.geany.org