Addressed comments (see below) and synced, new patch set uploaded.
http://codereview.appspot.com/53053/diff/1/5 File google/protobuf/compiler/cpp/cpp_file.cc (right): http://codereview.appspot.com/53053/diff/1/5#newcode401 Line 401: On 2009/05/06 01:03:14, jasonh wrote: > Indent() for prettier generated code? Done. http://codereview.appspot.com/53053/diff/1/6 File google/protobuf/compiler/cpp/cpp_message.cc (right): http://codereview.appspot.com/53053/diff/1/6#newcode608 Line 608: "delete $classname$::default_instance_;\n" On 2009/05/06 01:03:14, jasonh wrote: > Set these to NULL for consistency with other parts of the library? I figured I'd skip it and save some bytes of code size. http://codereview.appspot.com/53053/diff/1/13 File google/protobuf/extension_set.cc (right): http://codereview.appspot.com/53053/diff/1/13#newcode84 Line 84: delete registry_; On 2009/05/06 01:03:14, jasonh wrote: > registry_ = NULL;? Done. http://codereview.appspot.com/53053/diff/1/9 File google/protobuf/stubs/common.cc (right): http://codereview.appspot.com/53053/diff/1/9#newcode323 Line 323: // We don't need to lock shutdown_fucntions_mutex because it's up to the On 2009/05/06 01:03:14, jasonh wrote: > s/fucntions/functions/ Done. http://codereview.appspot.com/53053 --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---