On Sat, Jan 13, 2007 at 07:25:51PM +0100, Michael Gerz wrote:

> this patch fixes the encoding problem in the same way as Georg's patch. 
> However, I also fixed a potential problem that may occur if no backup 
> can be made successfully and I used proper variable names.
> 
> Folks, could someone please check this on Linux? A second Windows expert 
> would be fine, too (maybe someone who uses French accents in path names?).

I test your patch on cygwin and it works. I don't see any reason why
it should not work on linux. Only a suggestion: maybe you should rename
the bool variable successfulBackup as madeBackup or similar, as a backup
may not be attempted and the name could be misleading.

-- 
Enrico

Reply via email to