List:Backup« Previous MessageNext Message »
From:Charles Bell Date:February 22 2010 2:00pm
Subject:Re: Please review fix for backup_accented
View as plain text  

Patch approved. I will include it in my next push.


> Hi Hema,
> as far as I can imagine, the problem is that the test uses character
> strings, which do not contain valid SJIS characters. Some change in the
> server seems to behave different on invalid characters than before.
> IMHO this is acceptable. Only valid input needs to produce predictable
> output.
> I propose the following patch, which uses valid SJIS characters (I
> hope):
> Please approve.
> Regards
> Ingo

Please review fix for backup_accentedIngo Strüwing20 Feb
  • Re: Please review fix for backup_accentedCharles Bell22 Feb
  • Re: Please review fix for backup_accentedHema Sridharan22 Feb