Fair enough. I've attached a patch that adds a new command line option. Oleg.
On Thu, Mar 26, 2009 at 9:14 AM, Kenton Varda <ken...@google.com> wrote: > The existing format mimics GCC. Unfortunately I think if we changed it, it > would break other tools. Maybe we could enable this format based on a > command-line flag? > > On Wed, Mar 25, 2009 at 8:59 PM, os_ <oleg.smol...@gmail.com> wrote: >> >> Hey there, below is a tiny patch that lets VS200x go to the correct >> line in the .proto file when generation fails. IE it's to allow the >> interactive F4/NextError where VS relies on a certain format of error >> messages. >> >> Regards, >> Oleg. >> >> diff -aur ../protobuf-2.0.3-orig/src/google/protobuf/compiler/ >> command_line_interface.cc src/google/protobuf/compiler/ >> command_line_interface.cc >> --- ../protobuf-2.0.3-orig/src/google/protobuf/compiler/ >> command_line_interface.cc 2008-12-05 10:07:18.000000000 -0800 >> +++ src/google/protobuf/compiler/command_line_interface.cc 2009-03-25 >> 20:51:12.703125000 -0700 >> @@ -142,7 +142,7 @@ >> // to each here. >> cerr << filename; >> if (line != -1) { >> - cerr << ":" << (line + 1) << ":" << (column + 1); >> + cerr << "(" << (line + 1) << ") : error in column=" << >> (column + >> 1); >> } >> cerr << ": " << message << endl; >> } >> >> >> > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Protocol Buffers" group. To post to this group, send email to protobuf@googlegroups.com To unsubscribe from this group, send email to protobuf+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/protobuf?hl=en -~----------~----~----~----~------~----~------~--~---
vs2008a.patch
Description: Binary data