protogen: use _protoFile suffix for file descriptor variable
A "_ProtoFile" suffix can potentially conflict with a sub-message named
"ProtoFile" nested within a message that matches the camel-cased
form of the basename of the .proto source file.
Avoid unlikely conflicts and rename this to use a "_protoFile" suffix,
which can never conflict except with an enum value that is also named
"protoFile" (which is a violation of the style guide).
Change-Id: Ie9d22f9f741a63021b8f76906b20c6c2f599885b
Reviewed-on: https://go-review.googlesource.com/c/157218
Reviewed-by: Damien Neil <dneil@google.com>
diff --git a/protogen/protogen.go b/protogen/protogen.go
index eb89cfa..3f2a92e 100644
--- a/protogen/protogen.go
+++ b/protogen/protogen.go
@@ -435,7 +435,7 @@
}
}
f.GoDescriptorIdent = GoIdent{
- GoName: camelCase(cleanGoName(path.Base(prefix), true)) + "_ProtoFile",
+ GoName: camelCase(cleanGoName(path.Base(prefix), true)) + "_protoFile",
GoImportPath: f.GoImportPath,
}
f.GeneratedFilenamePrefix = prefix