Christopher Faylor wrote:
..
Patch is tested with 1.5.24-2. Merge with HEAD looks good, but was not
actually tested. Therefore, no changelog provided yet.
Thanks for this patch. Apart from the missing ChangeLog I'm inclined
to apply it to the upcoming 1.5.25 release, but I don't like to have it
in HEAD as is.
I'm not so sure it's appropriate for either yet.
Isn't it possible to use at least some of the managed mode functions
which deal with munging characters to do some of encoding? It seems
like the patch duplicates some of the functionality from path.cc.
I realize that the registry is sort of the opposite of a managed mount
but it seems like the encoding functions might be potentially used in
reverse for this.
I actually consulted path.cc before starting the patch but did not find
any function which provides the required functionality OOTB. Therefore,
I solved the tradeoff between "reuse" and "do not change working code if
you don't have time for thorough regression testing" by the latter :-)
Christian