Re: [Patch] Encode invalid chars in /proc/registry entries

[Christian Franke <Christian dot Franke at t-online dot de>]

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