[protobuf] Re: Issue 180 in protobuf: Compiler warning for negating a uint64 in compiler/parser.cc:723

2010-04-23 Thread protobuf
Comment #4 on issue 180 by ken...@google.com: Compiler warning for negating a uint64 in compiler/parser.cc:723 http://code.google.com/p/protobuf/issues/detail?id=180 I haven't seen this warning when testing on VS2008, but sure, why not? Revision 327. Note that negating an unsigned value

[protobuf] Re: Issue 180 in protobuf: Compiler warning for negating a uint64 in compiler/parser.cc:723

2010-04-23 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.3.1 Comment #5 on issue 180 by ken...@google.com: Compiler warning for negating a uint64 in compiler/parser.cc:723 http://code.google.com/p/protobuf/issues/detail?id=180 (No comment was entered for this change.) -- You received this

[protobuf] Re: Issue 180 in protobuf: Compiler warning for negating a uint64 in compiler/parser.cc:723

2010-04-22 Thread protobuf
Comment #1 on issue 180 by jam...@chromium.org: Compiler warning for negating a uint64 in compiler/parser.cc:723 http://code.google.com/p/protobuf/issues/detail?id=180 Simple patch (cast to int64 before negating). Passes 'make check' Attachments: patch.diff 652 bytes -- You

[protobuf] Re: Issue 180 in protobuf: Compiler warning for negating a uint64 in compiler/parser.cc:723

2010-04-22 Thread protobuf
Comment #2 on issue 180 by jam...@chromium.org: Compiler warning for negating a uint64 in compiler/parser.cc:723 http://code.google.com/p/protobuf/issues/detail?id=180 Patch in rietveld form (I accidentally uploaded it to the Chromium rietveld instance instead of the normal one, but it