commit 41335a03becd3df1371de035dcbc787c4ce49ae3
author Eric Salo <salo@google.com> Mon Oct 10 13:53:47 2022 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Oct 10 13:54:21 2022 -0700
tree 8020671aff0070ca4eb76292b8e8a1852f026eeb
parent 1a3803ce8b16fa5cd8f0fff8c10169fbfa70062c

normalize upb_Message_New()

We had _upb_Message_New(), which created a message from a mini table and was
being used outside of upb even though it is an internal-only function.

We also had upb_Message_New(), which created a message from a message def.

Now there is a single public function, upb_Message_New(), which creates a
message from a mini table and covers all use cases. (The internal version has the same definition and is used for inlining.)

PiperOrigin-RevId: 480169804

python/map.c

diff --git a/python/map.c b/python/map.c
index dfadf41..bdfb34d 100644
--- a/python/map.c
+++ b/python/map.c
@@ -204,7 +204,9 @@
     map = PyUpb_MapContainer_EnsureReified(_self);
     upb_Arena* arena = PyUpb_Arena_Get(self->arena);
     if (upb_FieldDef_IsSubMessage(val_f)) {
-      u_val.msg_val = upb_Message_New(upb_FieldDef_MessageSubDef(val_f), arena);
+      const upb_Message* m = upb_FieldDef_MessageSubDef(val_f);
+      const upb_MiniTable* layout = upb_MessageDef_MiniTable(m);
+      u_val.msg_val = upb_Message_New(layout, arena);
     } else {
       memset(&u_val, 0, sizeof(u_val));
     }