commit 2f174a520a6c296e042be00889143d48b9e8dbb6
author Hector Dearman <hjd@google.com> Tue Mar 21 15:21:54 2023 +0000
committer Hector Dearman <hjd@google.com> Tue Mar 21 15:21:54 2023 +0000
tree 45387bab917680ab6a037da40567ab5dc089f92c
parent b6aca6cf116b4f6692c86c0f14c4497c69093ee2

Update FieldMetadata<...> type to take advantage of C++17

Prior to C++17 the type of metadata on protozero generated bindings
had to be a function type since constexpr inline variables were not
supported. Now that the project uses C++17 we can update this type.

Previously for a field 'foo' the variable kFoo was defined as:
`static constexpr FieldMetadata_Foo kFoo() = { return {}; };`
it is now defined as:
`static constexpr FieldMetadata_Foo kFoo;`

This is a breaking change for users who directly access field metadata.

Change-Id: I8b754c63a664cd0552ef05002a10bf6c4e7ee21e

CHANGELOG

diff --git a/CHANGELOG b/CHANGELOG
index 8194fa8..7662692 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -7,7 +7,15 @@
   UI:
     *
   SDK:
-    *
+    * Changed the type of the static constexpr metadata on protozero
+      generated bindings from a function returning the metadata to
+      metadata itself. For a field 'foo' the variable kFoo previously
+      defined as:
+      `static constexpr FieldMetadata_Foo kFoo() = { return {}; };`
+      it is now defined as:
+      `static constexpr FieldMetadata_Foo kFoo;`
+      This is a breaking change for users who directly access field
+      metadata.
 
 v33.0 - 2023-03-02:
   All: