Merge branch 'master' into add-generated-string

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "prost"
-version = "0.12.1"
+version = "0.12.3"
 authors = [
     "Dan Burkert <dan@danburkert.com>",
     "Lucio Franco <luciofranco14@gmail.com",
@@ -48,7 +48,7 @@ std = []
 
 [dependencies]
 bytes = { version = "1", default-features = false }
-prost-derive = { version = "0.12.1", path = "prost-derive", optional = true }
+prost-derive = { version = "0.12.3", path = "prost-derive", optional = true }
 
 [dev-dependencies]
 criterion = { version = "0.4", default-features = false }

README.md

@@ -44,7 +44,7 @@ start-to-finish example.
 
 ### MSRV
 
-`prost` follows the `tokio-rs` projects MSRV model and supports 1.60. For more
+`prost` follows the `tokio-rs` project's MSRV model and supports 1.60. For more
 information on the tokio msrv policy you can check it out [here][tokio msrv]
 
 [tokio msrv]: https://github.com/tokio-rs/tokio/#supported-rust-versions

prost-build/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "prost-build"
-version = "0.12.1"
+version = "0.12.3"
 authors = [
     "Dan Burkert <dan@danburkert.com>",
     "Lucio Franco <luciofranco14@gmail.com>",
@@ -27,8 +27,8 @@ itertools = { version = ">=0.10, <0.12", default-features = false, features = ["
 log = "0.4"
 multimap = { version = "0.8", default-features = false }
 petgraph = { version = "0.6", default-features = false }
-prost = { version = "0.12.1", path = "..", default-features = false }
-prost-types = { version = "0.12.1", path = "../prost-types", default-features = false }
+prost = { version = "0.12.3", path = "..", default-features = false }
+prost-types = { version = "0.12.3", path = "../prost-types", default-features = false }
 tempfile = "3"
 once_cell = "1.17.1"
 regex = { version = "1.8.1", default-features = false, features = ["std", "unicode-bool"] }

prost-build/src/code_generator.rs

@@ -29,6 +29,7 @@ enum Syntax {
 pub struct CodeGenerator<'a> {
     config: &'a mut Config,
     package: String,
+    type_path: Vec<String>,
     source_info: Option<SourceCodeInfo>,
     syntax: Syntax,
     message_graph: &'a MessageGraph,
@@ -69,6 +70,7 @@ impl<'a> CodeGenerator<'a> {
         let mut code_gen = CodeGenerator {
             config,
             package: file.package.unwrap_or_default(),
+            type_path: Vec::new(),
             source_info,
             syntax,
             message_graph,
@@ -84,13 +86,6 @@ impl<'a> CodeGenerator<'a> {
             code_gen.package
         );
 
-        if code_gen.config.enable_type_names {
-            code_gen.buf.push_str(&format!(
-                "const PACKAGE: &str = \"{}\";\n",
-                code_gen.package,
-            ));
-        }
-
         code_gen.path.push(4);
         for (idx, message) in file.message_type.into_iter().enumerate() {
             code_gen.path.push(idx as i32);
@@ -128,10 +123,16 @@ impl<'a> CodeGenerator<'a> {
 
         let message_name = message.name().to_string();
         let fq_message_name = format!(
-            "{}{}.{}",
-            if self.package.is_empty() { "" } else { "." },
-            self.package,
-            message.name()
+            "{}{}{}{}.{}",
+            if self.package.is_empty() && self.type_path.is_empty() {
+                ""
+            } else {
+                "."
+            },
+            self.package.trim_matches('.'),
+            if self.type_path.is_empty() { "" } else { "." },
+            self.type_path.join("."),
+            message_name,
         );
 
         // Skip external types.
@@ -282,19 +283,34 @@ impl<'a> CodeGenerator<'a> {
         ));
         self.depth += 1;
 
-        self.buf
-            .push_str("const PACKAGE: &'static str = PACKAGE;\n");
         self.buf.push_str(&format!(
             "const NAME: &'static str = \"{}\";\n",
-            message_name
+            message_name,
+        ));
+        self.buf.push_str(&format!(
+            "const PACKAGE: &'static str = \"{}\";\n",
+            self.package,
+        ));
+
+        let prost_path = self.config.prost_path.as_deref().unwrap_or("::prost");
+        let string_path = format!("{}::alloc::string::String", prost_path);
+        let format_path = format!("{}::alloc::format", prost_path);
+
+        self.buf.push_str(&format!(
+            r#"fn full_name() -> {string_path} {{
+                {format_path}!("{}{}{}{}{{}}", Self::NAME)
+            }}"#,
+            self.package.trim_matches('.'),
+            if self.package.is_empty() { "" } else { "." },
+            self.type_path.join("."),
+            if self.type_path.is_empty() { "" } else { "." },
         ));
 
         if let Some(domain_name) = self.config.type_name_domains.get_first(fq_message_name) {
             self.buf.push_str(&format!(
-                r#"fn type_url() -> String {{
-                    format!("{}/{{}}", Self::full_name())
+                r#"fn type_url() -> {string_path} {{
+                    {format_path}!("{domain_name}/{{}}", Self::full_name())
                 }}"#,
-                domain_name
             ));
         }
 
@@ -684,11 +700,18 @@ impl<'a> CodeGenerator<'a> {
 
         let enum_values = &desc.value;
         let fq_proto_enum_name = format!(
-            "{}{}.{}",
-            if self.package.is_empty() { "" } else { "." },
-            self.package,
-            proto_enum_name
+            "{}{}{}{}.{}",
+            if self.package.is_empty() && self.type_path.is_empty() {
+                ""
+            } else {
+                "."
+            },
+            self.package.trim_matches('.'),
+            if self.type_path.is_empty() { "" } else { "." },
+            self.type_path.join("."),
+            proto_enum_name,
         );
+
         if self
             .extern_paths
             .resolve_ident(&fq_proto_enum_name)
@@ -906,17 +929,15 @@ impl<'a> CodeGenerator<'a> {
         self.buf.push_str(&to_snake(module));
         self.buf.push_str(" {\n");
 
-        self.package.push('.');
-        self.package.push_str(module);
+        self.type_path.push(module.into());
 
         self.depth += 1;
     }
 
     fn pop_mod(&mut self) {
         self.depth -= 1;
 
-        let idx = self.package.rfind('.').unwrap();
-        self.package.truncate(idx);
+        self.type_path.pop();
 
         self.push_indent();
         self.buf.push_str("}\n");
@@ -954,7 +975,11 @@ impl<'a> CodeGenerator<'a> {
             return proto_ident;
         }
 
-        let mut local_path = self.package.split('.').peekable();
+        let mut local_path = self
+            .package
+            .split('.')
+            .chain(self.type_path.iter().map(String::as_str))
+            .peekable();
 
         // If no package is specified the start of the package name will be '.'
         // and split will return an empty string ("") which breaks resolution

prost-build/src/lib.rs

@@ -1,4 +1,4 @@
-#![doc(html_root_url = "https://docs.rs/prost-build/0.12.1")]
+#![doc(html_root_url = "https://docs.rs/prost-build/0.12.2")]
 #![allow(clippy::option_as_ref_deref, clippy::format_push_string)]
 
 //! `prost-build` compiles `.proto` files into Rust.

prost-derive/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "prost-derive"
-version = "0.12.1"
+version = "0.12.3"
 authors = [
     "Dan Burkert <dan@danburkert.com>",
     "Lucio Franco <luciofranco14@gmail.com>",

prost-derive/src/lib.rs

@@ -1,4 +1,4 @@
-#![doc(html_root_url = "https://docs.rs/prost-derive/0.12.1")]
+#![doc(html_root_url = "https://docs.rs/prost-derive/0.12.2")]
 // The `quote!` macro requires deep recursion.
 #![recursion_limit = "4096"]
 

prost-types/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "prost-types"
-version = "0.12.1"
+version = "0.12.3"
 authors = [
     "Dan Burkert <dan@danburkert.com>",
     "Lucio Franco <luciofranco14@gmail.com",
@@ -22,7 +22,7 @@ default = ["std"]
 std = ["prost/std"]
 
 [dependencies]
-prost = { version = "0.12.1", path = "..", default-features = false, features = ["prost-derive"] }
+prost = { version = "0.12.3", path = "..", default-features = false, features = ["prost-derive"] }
 
 [dev-dependencies]
 proptest = "1"

prost-types/src/lib.rs

@@ -1,4 +1,4 @@
-#![doc(html_root_url = "https://docs.rs/prost-types/0.12.1")]
+#![doc(html_root_url = "https://docs.rs/prost-types/0.12.2")]
 
 //! Protocol Buffers well-known types.
 //!

src/lib.rs

@@ -1,4 +1,4 @@
-#![doc(html_root_url = "https://docs.rs/prost/0.12.1")]
+#![doc(html_root_url = "https://docs.rs/prost/0.12.2")]
 #![cfg_attr(not(feature = "std"), no_std)]
 #![doc = include_str!("../README.md")]
 

src/name.rs

@@ -5,23 +5,27 @@ use alloc::{format, string::String};
 
 /// Associate a type name with a [`Message`] type.
 pub trait Name: Message {
-    /// Type name for this [`Message`]. This is the camel case name,
-    /// e.g. `TypeName`.
+    /// Simple name for this [`Message`].
+    /// This name is the same as it appears in the source .proto file, e.g. `FooBar`.
     const NAME: &'static str;
 
     /// Package name this message type is contained in. They are domain-like
     /// and delimited by `.`, e.g. `google.protobuf`.
     const PACKAGE: &'static str;
 
-    /// Full name of this message type containing both the package name and
-    /// type name, e.g. `google.protobuf.TypeName`.
+    /// Fully-qualified unique name for this [`Message`].
+    /// It's prefixed with the package name and names of any parent messages,
+    /// e.g. `google.rpc.BadRequest.FieldViolation`.
+    /// By default, this is the package name followed by the message name.
+    /// Fully-qualified names must be unique within a domain of Type URLs.
     fn full_name() -> String {
         format!("{}.{}", Self::PACKAGE, Self::NAME)
     }
 
-    /// Type URL for this message, which by default is the full name with a
+    /// Type URL for this [`Message`], which by default is the full name with a
     /// leading slash, but may also include a leading domain name, e.g.
     /// `type.googleapis.com/google.profile.Person`.
+    /// This can be used when serializing with the [`Any`] type.
     fn type_url() -> String {
         format!("/{}", Self::full_name())
     }

tests/src/type_names.proto

@@ -3,7 +3,9 @@ syntax = "proto3";
 package type_names;
 
 message Foo {
+    message Bar {
+    }
 }
 
-message Bar {
-}
+message Baz {
+}

tests/src/type_names.rs

@@ -1,4 +1,3 @@
-use prost::alloc::{format, string::String};
 use prost::Name;
 
 include!(concat!(env!("OUT_DIR"), "/type_names.rs"));
@@ -7,9 +6,16 @@ include!(concat!(env!("OUT_DIR"), "/type_names.rs"));
 fn valid_type_names() {
     assert_eq!("Foo", Foo::NAME);
     assert_eq!("type_names", Foo::PACKAGE);
+    assert_eq!("type_names.Foo", Foo::full_name());
     assert_eq!("tests/type_names.Foo", Foo::type_url());
 
-    assert_eq!("Bar", Bar::NAME);
-    assert_eq!("type_names", Bar::PACKAGE);
-    assert_eq!("/type_names.Bar", Bar::type_url());
+    assert_eq!("Bar", foo::Bar::NAME);
+    assert_eq!("type_names", foo::Bar::PACKAGE);
+    assert_eq!("type_names.Foo.Bar", foo::Bar::full_name());
+    assert_eq!("tests/type_names.Foo.Bar", foo::Bar::type_url());
+
+    assert_eq!("Baz", Baz::NAME);
+    assert_eq!("type_names", Baz::PACKAGE);
+    assert_eq!("type_names.Baz", Baz::full_name());
+    assert_eq!("/type_names.Baz", Baz::type_url());
 }