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

Christian Franke Christian.Franke@t-online.de
Fri Nov 16 19:35:00 GMT 2007


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



More information about the Cygwin-patches mailing list