[protobuf] Re: Issue 541 in protobuf: Double decode in google.protobuf.text_format._CUnescape
Comment #2 on issue 541 by matt.k...@undue.org: Double decode in google.protobuf.text_format._CUnescape http://code.google.com/p/protobuf/issues/detail?id=541 Don't be confused by the input escaping, there is only one leading backslash in '\\x5c'. I've just checked the trunk and I can see that it is fixed but for a different reason -- the regex only matches single hex sequences (e.g. \x5), thus avoiding the double unescaping. My problem now is that I don't have a fixed protobuf library available in my distro. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups Protocol Buffers group. To unsubscribe from this group and stop receiving emails from it, send an email to protobuf+unsubscr...@googlegroups.com. To post to this group, send email to protobuf@googlegroups.com. Visit this group at http://groups.google.com/group/protobuf. For more options, visit https://groups.google.com/groups/opt_out.
[protobuf] Re: Issue 541 in protobuf: Double decode in google.protobuf.text_format._CUnescape
Comment #3 on issue 541 by xiaof...@google.com: Double decode in google.protobuf.text_format._CUnescape http://code.google.com/p/protobuf/issues/detail?id=541 You are right. I was misunderstanding '\\x5c' as having 2 backslashes. The fixed code is already in 2.5.0 release. Maybe it won't be long before it's available in your distro. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups Protocol Buffers group. To unsubscribe from this group and stop receiving emails from it, send an email to protobuf+unsubscr...@googlegroups.com. To post to this group, send email to protobuf@googlegroups.com. Visit this group at http://groups.google.com/group/protobuf. For more options, visit https://groups.google.com/groups/opt_out.
[protobuf] Re: Issue 541 in protobuf: Double decode in google.protobuf.text_format._CUnescape
Updates: Status: Invalid Comment #1 on issue 541 by xiaof...@google.com: Double decode in google.protobuf.text_format._CUnescape http://code.google.com/p/protobuf/issues/detail?id=541 I think this is fixed in trunk. The current implementation of re.sub() will only do the unescaping when there are an odd number of backslashes. In the case you give, re.sub() will do nothing. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups Protocol Buffers group. To unsubscribe from this group and stop receiving emails from it, send an email to protobuf+unsubscr...@googlegroups.com. To post to this group, send email to protobuf@googlegroups.com. Visit this group at http://groups.google.com/group/protobuf. For more options, visit https://groups.google.com/groups/opt_out.
