diff --git a/upb/descriptor/reader.c b/upb/descriptor/reader.c index 2b9e0e9167..e219f18653 100644 --- a/upb/descriptor/reader.c +++ b/upb/descriptor/reader.c @@ -651,7 +651,6 @@ static void *msg_startext(void *closure, const void *hd) { return r; } -#include static void *msg_startfield(void *closure, const void *hd) { upb_descreader *r = closure; r->f = upb_fielddef_new(&r->f); @@ -667,6 +666,8 @@ static bool msg_endfield(void *closure, const void *hd) { bool ok; UPB_UNUSED(hd); + /* Oneof fields are added to the msgdef through their oneof, so don't need to + * be added here. */ if (upb_fielddef_containingoneof(r->f) == NULL) { ok = upb_msgdef_addfield(m, r->f, &r->f, NULL); UPB_ASSERT(ok); diff --git a/upb/refcounted.h b/upb/refcounted.h index 1537414480..6698d38df7 100644 --- a/upb/refcounted.h +++ b/upb/refcounted.h @@ -28,6 +28,8 @@ * For this reason we don't enable it by default, even in debug builds. */ +/* #define UPB_DEBUG_REFS */ + #ifdef __cplusplus namespace upb { class RefCounted;