On Sat, 2003-01-25 at 16:19, Stuart Donaldson wrote:
> In essence, Fazal Majid's patch which he included in the bug report is:
>               #self.chars = string.replace(self.chars,'\t','\\\\t')
>               #self.chars = string.replace(self.chars, "'",
> "\\'")
>               self.chars = string.replace(self.chars,'"',r'\"')
> +             self.chars = string.replace(self.chars,r'\n',r'\\n')
> +             self.chars = string.replace(self.chars,r'\t',r'\\t')
> +             self.chars = string.replace(self.chars,r'\r',r'\\r')

This is weird.  The proper string.replace statements were commented out
(except for \r, which was probably just an oversight).  The proper patch
would be to uncomment these lines.

But that they were commented out surprises me.

In fact, what should really happen is
    string.replace(r'\', r'\\')
    string.replace(r'"', r'\"')

  Ian



-------------------------------------------------------
This SF.NET email is sponsored by:
SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See!
http://www.vasoftware.com
_______________________________________________
Webware-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/webware-devel

Reply via email to