Changed parse API to know about msgdefs.

This should make it both easier to use and easier to
optimize, in exchange for a small amount of generality.
In practice, any remotely normal case is still very
natural.

1 files changed, 4 insertions, 7 deletions

diff --git a/src/upb.h b/src/upb.h
index e8ec001..cc09ab1 100644
--- a/src/upb.h
+++ b/src/upb.h
@@ -140,13 +140,10 @@ union upb_value_ptr {
   void *_void;
 };
 
-// Unfortunately there is no way to define this so that it can be used as a
-// generic expression, a la:
-//   foo(UPB_VALUE_ADDROF(bar));
-// ...you have to use it as the initializer of a upb_value_ptr:
-//   union upb_value_ptr p = UPB_VALUE_ADDROF(bar);
-//   foo(p);
-#define UPB_VALUE_ADDROF(val) {(void*)&val._double}
+INLINE union upb_value_ptr upb_value_addrof(union upb_value *val) {
+  union upb_value_ptr ptr = {&val->_double};
+  return ptr;
+}
 
 /**
  * Converts upb_value_ptr -> upb_value by reading from the pointer.  We need to