diff options
author | Jon Skeet <jonskeet@google.com> | 2015-07-15 22:03:38 +0100 |
---|---|---|
committer | Jon Skeet <jonskeet@google.com> | 2015-07-15 22:04:45 +0100 |
commit | db52c9dd585da7d5c9063409117b8a5680e384db (patch) | |
tree | 7a1b6c5d7047946443c6ad9b531e717552413c62 /src/google | |
parent | 541b442b99e1e9a1c514e2aacfe1e83561a9ab68 (diff) | |
download | protobuf-db52c9dd585da7d5c9063409117b8a5680e384db.tar.gz protobuf-db52c9dd585da7d5c9063409117b8a5680e384db.tar.bz2 protobuf-db52c9dd585da7d5c9063409117b8a5680e384db.zip |
Address requested change from code review.
Diffstat (limited to 'src/google')
-rw-r--r-- | src/google/protobuf/compiler/csharp/csharp_message.cc | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/google/protobuf/compiler/csharp/csharp_message.cc b/src/google/protobuf/compiler/csharp/csharp_message.cc index 5a72b2e7..79b762d1 100644 --- a/src/google/protobuf/compiler/csharp/csharp_message.cc +++ b/src/google/protobuf/compiler/csharp/csharp_message.cc @@ -122,10 +122,10 @@ void MessageGenerator::GenerateStaticVariables(io::Printer* printer) { for (int i = 0; i < descriptor_->nested_type_count(); i++) { // Don't generate accessor table fields for maps... - if (IsMapEntryMessage(descriptor_->nested_type(i))) continue; - - MessageGenerator messageGenerator(descriptor_->nested_type(i)); - messageGenerator.GenerateStaticVariables(printer); + if (!IsMapEntryMessage(descriptor_->nested_type(i))) { + MessageGenerator messageGenerator(descriptor_->nested_type(i)); + messageGenerator.GenerateStaticVariables(printer); + } } } @@ -164,10 +164,10 @@ void MessageGenerator::GenerateStaticVariableInitializers(io::Printer* printer) // Generate static member initializers for all non-map-entry nested types. for (int i = 0; i < descriptor_->nested_type_count(); i++) { - if (IsMapEntryMessage(descriptor_->nested_type(i))) continue; - - MessageGenerator messageGenerator(descriptor_->nested_type(i)); - messageGenerator.GenerateStaticVariableInitializers(printer); + if (!IsMapEntryMessage(descriptor_->nested_type(i))) { + MessageGenerator messageGenerator(descriptor_->nested_type(i)); + messageGenerator.GenerateStaticVariableInitializers(printer); + } } } @@ -302,10 +302,10 @@ void MessageGenerator::Generate(io::Printer* printer) { } for (int i = 0; i < descriptor_->nested_type_count(); i++) { // Don't generate nested types for maps... - if (IsMapEntryMessage(descriptor_->nested_type(i))) continue; - - MessageGenerator messageGenerator(descriptor_->nested_type(i)); - messageGenerator.Generate(printer); + if (!IsMapEntryMessage(descriptor_->nested_type(i))) { + MessageGenerator messageGenerator(descriptor_->nested_type(i)); + messageGenerator.Generate(printer); + } } printer->Outdent(); printer->Print("}\n" |