Add const to generated enum values()

Enum values() functions return static arrays of static strings and could be of compile time use. Make callable in const contexts.
smithy-lang · Nov 20, 2022 · a5f775a · a5f775a
1 parent 5eeb5d7
commit a5f775a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.next.toml b/CHANGELOG.next.toml
@@ -478,3 +478,9 @@ x-amzn-errortype: com.example.service#InvalidRequestException
 references = ["smithy-rs#1982"]
 meta = { "breaking" = true, "tada" = false, "bug" = false, "target" = "server" }
 author = "david-perez"
+
+[[smithy-rs]]
+message = "Make generated enum `values()` functions callable in const contexts."
+references = []
+meta = { "breaking" = false, "tada" = false, "bug" = false, "target" = "client" }
+author = "lsr0"
diff --git a/...c/main/kotlin/software/amazon/smithy/rust/codegen/core/smithy/generators/EnumGenerator.kt b/...c/main/kotlin/software/amazon/smithy/rust/codegen/core/smithy/generators/EnumGenerator.kt
@@ -141,7 +141,7 @@ open class EnumGenerator(
             }
 
             docs("Returns all the `&str` representations of the enum members.")
-            rustBlock("pub fn $Values() -> &'static [&'static str]") {
+            rustBlock("pub const fn $Values() -> &'static [&'static str]") {
                 withBlock("&[", "]") {
                     val memberList = sortedMembers.joinToString(", ") { it.value.dq() }
                     rust(memberList)
@@ -198,7 +198,7 @@ open class EnumGenerator(
             }
 
             rust("/// Returns all the `&str` values of the enum members.")
-            rustBlock("pub fn $Values() -> &'static [&'static str]") {
+            rustBlock("pub const fn $Values() -> &'static [&'static str]") {
                 withBlock("&[", "]") {
                     val memberList = sortedMembers.joinToString(", ") { it.value.doubleQuote() }
                     write(memberList)