Replace Version.moduleName with SwiftProtobufInfo.name.

The Namer already had to know the module and used the other
constant so generated module references would be correct.

Author: thomasvl

Sources/protoc-gen-swift/Descriptor+Extensions.swift

@@ -211,7 +211,7 @@ extension FieldDescriptor {
     switch type {
     case .bool: return "false"
     case .string: return "String()"
-    case .bytes: return "\(Version.moduleName).Internal.emptyData"
+    case .bytes: return "\(SwiftProtobufInfo.name).Internal.emptyData"
     case .group, .message:
       return namer.fullName(message: messageType) + "()"
     case .enum:
@@ -229,31 +229,31 @@ extension FieldDescriptor {
       let valueTraits = valueField.traitsType(namer: namer)
       switch valueField.type {
       case .message:  // Map's can't have a group as the value
-        return "\(Version.moduleName)._ProtobufMessageMap<\(keyTraits),\(valueTraits)>"
+        return "\(SwiftProtobufInfo.name)._ProtobufMessageMap<\(keyTraits),\(valueTraits)>"
       case .enum:
-        return "\(Version.moduleName)._ProtobufEnumMap<\(keyTraits),\(valueTraits)>"
+        return "\(SwiftProtobufInfo.name)._ProtobufEnumMap<\(keyTraits),\(valueTraits)>"
       default:
-        return "\(Version.moduleName)._ProtobufMap<\(keyTraits),\(valueTraits)>"
+        return "\(SwiftProtobufInfo.name)._ProtobufMap<\(keyTraits),\(valueTraits)>"
       }
     }
     switch type {
-    case .double: return "\(Version.moduleName).ProtobufDouble"
-    case .float: return "\(Version.moduleName).ProtobufFloat"
-    case .int64: return "\(Version.moduleName).ProtobufInt64"
-    case .uint64: return "\(Version.moduleName).ProtobufUInt64"
-    case .int32: return "\(Version.moduleName).ProtobufInt32"
-    case .fixed64: return "\(Version.moduleName).ProtobufFixed64"
-    case .fixed32: return "\(Version.moduleName).ProtobufFixed32"
-    case .bool: return "\(Version.moduleName).ProtobufBool"
-    case .string: return "\(Version.moduleName).ProtobufString"
+    case .double: return "\(SwiftProtobufInfo.name).ProtobufDouble"
+    case .float: return "\(SwiftProtobufInfo.name).ProtobufFloat"
+    case .int64: return "\(SwiftProtobufInfo.name).ProtobufInt64"
+    case .uint64: return "\(SwiftProtobufInfo.name).ProtobufUInt64"
+    case .int32: return "\(SwiftProtobufInfo.name).ProtobufInt32"
+    case .fixed64: return "\(SwiftProtobufInfo.name).ProtobufFixed64"
+    case .fixed32: return "\(SwiftProtobufInfo.name).ProtobufFixed32"
+    case .bool: return "\(SwiftProtobufInfo.name).ProtobufBool"
+    case .string: return "\(SwiftProtobufInfo.name).ProtobufString"
     case .group, .message: return namer.fullName(message: messageType)
-    case .bytes: return "\(Version.moduleName).ProtobufBytes"
-    case .uint32: return "\(Version.moduleName).ProtobufUInt32"
+    case .bytes: return "\(SwiftProtobufInfo.name).ProtobufBytes"
+    case .uint32: return "\(SwiftProtobufInfo.name).ProtobufUInt32"
     case .enum: return namer.fullName(enum: enumType)
-    case .sfixed32: return "\(Version.moduleName).ProtobufSFixed32"
-    case .sfixed64: return "\(Version.moduleName).ProtobufSFixed64"
-    case .sint32: return "\(Version.moduleName).ProtobufSInt32"
-    case .sint64: return "\(Version.moduleName).ProtobufSInt64"
+    case .sfixed32: return "\(SwiftProtobufInfo.name).ProtobufSFixed32"
+    case .sfixed64: return "\(SwiftProtobufInfo.name).ProtobufSFixed64"
+    case .sint32: return "\(SwiftProtobufInfo.name).ProtobufSInt32"
+    case .sint64: return "\(SwiftProtobufInfo.name).ProtobufSInt64"
     }
   }
 }

Sources/protoc-gen-swift/EnumGenerator.swift

@@ -58,7 +58,7 @@ class EnumGenerator {
 
     p.print("\n")
     p.print(enumDescriptor.protoSourceComments())
-    p.print("\(visibility)enum \(swiftRelativeName): \(Version.moduleName).Enum {\n")
+    p.print("\(visibility)enum \(swiftRelativeName): \(SwiftProtobufInfo.name).Enum {\n")
     p.indent()
     p.print("\(visibility)typealias RawValue = Int\n")
 
@@ -120,7 +120,7 @@ class EnumGenerator {
 
   func generateRuntimeSupport(printer p: inout CodePrinter) {
     p.print("\n")
-    p.print("extension \(swiftFullName): \(Version.moduleName)._ProtoNameProviding {\n")
+    p.print("extension \(swiftFullName): \(SwiftProtobufInfo.name)._ProtoNameProviding {\n")
     p.indent()
     generateProtoNameProviding(printer: &p)
     p.outdent()
@@ -156,7 +156,7 @@ class EnumGenerator {
   private func generateProtoNameProviding(printer p: inout CodePrinter) {
     let visibility = generatorOptions.visibilitySourceSnippet
 
-    p.print("\(visibility)static let _protobuf_nameMap: \(Version.moduleName)._NameMap = [\n")
+    p.print("\(visibility)static let _protobuf_nameMap: \(SwiftProtobufInfo.name)._NameMap = [\n")
     p.indent()
     for v in mainEnumValueDescriptorsSorted {
       if v.aliases.isEmpty {

Sources/protoc-gen-swift/ExtensionSetGenerator.swift

@@ -47,7 +47,7 @@ class ExtensionSetGenerator {
             default: modifier = ""
             }
 
-            return "\(Version.moduleName).\(label)\(modifier)ExtensionField"
+            return "\(SwiftProtobufInfo.name).\(label)\(modifier)ExtensionField"
         }
 
         init(descriptor: FieldDescriptor, generatorOptions: GeneratorOptions, namer: SwiftProtobufNamer) {
@@ -81,7 +81,7 @@ class ExtensionSetGenerator {
 
             p.print(
               comments,
-              "\(visibility)\(scope)let \(swiftRelativeExtensionName) = \(Version.moduleName).MessageExtension<\(extensionFieldType)<\(traitsType)>, \(containingTypeSwiftFullName)>(\n")
+              "\(visibility)\(scope)let \(swiftRelativeExtensionName) = \(SwiftProtobufInfo.name).MessageExtension<\(extensionFieldType)<\(traitsType)>, \(containingTypeSwiftFullName)>(\n")
             p.indent()
             p.print(
               "_protobuf_fieldNumber: \(fieldDescriptor.number),\n",
@@ -223,7 +223,7 @@ class ExtensionSetGenerator {
           "/// this .proto file. It can be used any place an `SwiftProtobuf.ExtensionMap` is needed\n",
           "/// in parsing, or it can be combined with other `SwiftProtobuf.SimpleExtensionMap`s to create\n",
           "/// a larger `SwiftProtobuf.SimpleExtensionMap`.\n",
-          "\(generatorOptions.visibilitySourceSnippet)let \(filePrefix)\(filenameAsIdentifer)_Extensions: \(Version.moduleName).SimpleExtensionMap = [\n")
+          "\(generatorOptions.visibilitySourceSnippet)let \(filePrefix)\(filenameAsIdentifer)_Extensions: \(SwiftProtobufInfo.name).SimpleExtensionMap = [\n")
         p.indent()
         var separator = ""
         for e in extensions {

Sources/protoc-gen-swift/FileGenerator.swift

@@ -93,7 +93,7 @@ class FileGenerator {
         if !fileDescriptor.isBundledProto {
             // The well known types ship with the runtime, everything else needs
             // to import the runtime.
-            p.print("import \(Version.moduleName)\n")
+            p.print("import \(SwiftProtobufInfo.name)\n")
         }
         if let neededImports = generatorOptions.protoToModuleMappings.neededModules(forFile: fileDescriptor) {
             p.print("\n")
@@ -190,10 +190,10 @@ class FileGenerator {
             "// incompatible with the version of SwiftProtobuf to which you are linking.\n",
             "// Please ensure that you are building against the same version of the API\n",
             "// that was used to generate this file.\n",
-            "fileprivate struct _GeneratedWithProtocGenSwiftVersion: \(Version.moduleName).ProtobufAPIVersionCheck {\n")
+            "fileprivate struct _GeneratedWithProtocGenSwiftVersion: \(SwiftProtobufInfo.name).ProtobufAPIVersionCheck {\n")
         p.indent()
         p.print(
-            "struct _\(v): \(Version.moduleName).ProtobufAPIVersion_\(v) {}\n",
+            "struct _\(v): \(SwiftProtobufInfo.name).ProtobufAPIVersion_\(v) {}\n",
             "typealias Version = _\(v)\n")
         p.outdent()
         p.print("}\n")

Sources/protoc-gen-swift/MessageFieldGenerator.swift

@@ -161,7 +161,7 @@ class MessageFieldGenerator: FieldGeneratorBase, FieldGenerator {
         guard isGroupOrMessage && fieldDescriptor.messageType.hasRequiredFields() else { return }
 
         if isRepeated {  // Map or Array
-            p.print("if !\(Version.moduleName).Internal.areAllInitialized(\(storedProperty)) {return false}\n")
+            p.print("if !\(SwiftProtobufInfo.name).Internal.areAllInitialized(\(storedProperty)) {return false}\n")
         } else {
             p.print("if let v = \(storedProperty), !v.isInitialized {return false}\n")
         }

Sources/protoc-gen-swift/MessageGenerator.swift

@@ -118,7 +118,7 @@ class MessageGenerator {
 
     let conformances: String
     if isExtensible {
-      conformances = ": \(Version.moduleName).ExtensibleMessage"
+      conformances = ": \(SwiftProtobufInfo.name).ExtensibleMessage"
     } else {
       conformances = ""
     }
@@ -127,7 +127,7 @@ class MessageGenerator {
         descriptor.protoSourceComments(),
         "\(visibility)struct \(swiftRelativeName)\(conformances) {\n")
     p.indent()
-    p.print("// \(Version.moduleName).Message conformance is added in an extension below. See the\n",
+    p.print("// \(SwiftProtobufInfo.name).Message conformance is added in an extension below. See the\n",
             "// `Message` and `Message+*Additions` files in the SwiftProtobuf library for\n",
             "// methods supported on all messages.\n")
 
@@ -137,7 +137,7 @@ class MessageGenerator {
 
     p.print(
         "\n",
-        "\(visibility)var unknownFields = \(Version.moduleName).UnknownStorage()\n")
+        "\(visibility)var unknownFields = \(SwiftProtobufInfo.name).UnknownStorage()\n")
 
     for o in oneofs {
       o.generateMainEnum(printer: &p)
@@ -164,7 +164,7 @@ class MessageGenerator {
     if isExtensible {
       p.print(
           "\n",
-          "\(visibility)var _protobuf_extensionFieldValues = \(Version.moduleName).ExtensionFieldValueSet()\n")
+          "\(visibility)var _protobuf_extensionFieldValues = \(SwiftProtobufInfo.name).ExtensionFieldValueSet()\n")
     }
     if let storage = storage {
       if !isExtensible {
@@ -200,7 +200,7 @@ class MessageGenerator {
   func generateRuntimeSupport(printer p: inout CodePrinter, file: FileGenerator, parent: MessageGenerator?) {
     p.print(
         "\n",
-        "extension \(swiftFullName): \(Version.moduleName).Message, \(Version.moduleName)._MessageImplementationBase, \(Version.moduleName)._ProtoNameProviding {\n")
+        "extension \(swiftFullName): \(SwiftProtobufInfo.name).Message, \(SwiftProtobufInfo.name)._MessageImplementationBase, \(SwiftProtobufInfo.name)._ProtoNameProviding {\n")
     p.indent()
 
     if let parent = parent {
@@ -239,9 +239,9 @@ class MessageGenerator {
 
   private func generateProtoNameProviding(printer p: inout CodePrinter) {
     if fields.isEmpty {
-      p.print("\(visibility)static let _protobuf_nameMap = \(Version.moduleName)._NameMap()\n")
+      p.print("\(visibility)static let _protobuf_nameMap = \(SwiftProtobufInfo.name)._NameMap()\n")
     } else {
-      p.print("\(visibility)static let _protobuf_nameMap: \(Version.moduleName)._NameMap = [\n")
+      p.print("\(visibility)static let _protobuf_nameMap: \(SwiftProtobufInfo.name)._NameMap = [\n")
       p.indent()
       for f in fields {
         p.print("\(f.number): \(f.fieldMapNames),\n")
@@ -256,7 +256,7 @@ class MessageGenerator {
   ///
   /// - Parameter p: The code printer.
   private func generateDecodeMessage(printer p: inout CodePrinter) {
-    p.print("\(visibility)mutating func decodeMessage<D: \(Version.moduleName).Decoder>(decoder: inout D) throws {\n")
+    p.print("\(visibility)mutating func decodeMessage<D: \(SwiftProtobufInfo.name).Decoder>(decoder: inout D) throws {\n")
     p.indent()
     if storage != nil {
       p.print("_ = _uniqueStorage()\n")
@@ -319,7 +319,7 @@ class MessageGenerator {
   ///
   /// - Parameter p: The code printer.
   private func generateTraverse(printer p: inout CodePrinter) {
-    p.print("\(visibility)func traverse<V: \(Version.moduleName).Visitor>(visitor: inout V) throws {\n")
+    p.print("\(visibility)func traverse<V: \(SwiftProtobufInfo.name).Visitor>(visitor: inout V) throws {\n")
     p.indent()
     generateWithLifetimeExtension(printer: &p, throws: true) { p in
       if let storage = storage {

Sources/protoc-gen-swift/StringUtils.swift

@@ -61,7 +61,7 @@ func trimWhitespace(_ s: String) -> String {
 ///  \n\r\t\\\'\" and three-digit octal escapes but nothing else.
 func escapedToDataLiteral(_ s: String) -> String {
   if s.isEmpty {
-    return "\(Version.moduleName).Internal.emptyData"
+    return "\(SwiftProtobufInfo.name).Internal.emptyData"
   }
   var out = "Data(["
   var separator = ""

Sources/protoc-gen-swift/Version.swift

@@ -27,8 +27,5 @@ struct Version {
     // or API-changing).
     static let compatibilityVersion = 2
 
-    // Name of the SwiftProtobuf module
-    static let moduleName = "SwiftProtobuf"
-
     static let copyright = "Copyright (C) 2014-2017 Apple Inc. and the project authors"
 }

Sources/protoc-gen-swift/main.swift

@@ -101,7 +101,7 @@ struct GeneratorPlugin {
         + "In particular, if you have renamed this program, you will need to\n"
         + "adjust the protoc command-line option accordingly.\n"
         + "\n"
-        + "The generated Swift output requires the \(Version.moduleName) \(SwiftProtobuf.Version.versionString)\n"
+        + "The generated Swift output requires the \(SwiftProtobufInfo.name) \(SwiftProtobuf.Version.versionString)\n"
         + "library be included in your project.\n"
         + "\n"
         + "If you use `swift build` to compile your project, add this to\n"