From 153a226a2aa4c98cce932350ff74b8b3eb089e28 Mon Sep 17 00:00:00 2001
From: Daniel Martin <fizbin@gmail.com>
Date: Wed, 8 Apr 2015 09:53:26 -0400
Subject: [PATCH] CR feedback: spacing, document return values

---
 src/google/protobuf/compiler/java/java_extension.h | 4 ++++
 src/google/protobuf/compiler/java/java_file.cc     | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/google/protobuf/compiler/java/java_extension.h b/src/google/protobuf/compiler/java/java_extension.h
index fac5ba00c9..bdd4226379 100644
--- a/src/google/protobuf/compiler/java/java_extension.h
+++ b/src/google/protobuf/compiler/java/java_extension.h
@@ -67,7 +67,11 @@ class ExtensionGenerator {
   virtual ~ExtensionGenerator() {}
 
   virtual void Generate(io::Printer* printer) = 0;
+
+  // Returns an estimate of the number of bytes the printed code will compile to
   virtual int GenerateNonNestedInitializationCode(io::Printer* printer) = 0;
+
+  // Returns an estimate of the number of bytes the printed code will compile to
   virtual int GenerateRegistrationCode(io::Printer* printer) = 0;
 
  protected:
diff --git a/src/google/protobuf/compiler/java/java_file.cc b/src/google/protobuf/compiler/java/java_file.cc
index 9f285774d0..4a1f452988 100644
--- a/src/google/protobuf/compiler/java/java_file.cc
+++ b/src/google/protobuf/compiler/java/java_file.cc
@@ -65,7 +65,7 @@ namespace {
 
 struct FieldDescriptorCompare {
   bool operator ()(const FieldDescriptor* f1, const FieldDescriptor* f2) {
-    if(f1== NULL) {
+    if(f1 == NULL) {
       return false;
     }
     if(f2 == NULL) {