diff options
author | Joshua Haberman <jhaberman@gmail.com> | 2017-01-24 10:38:50 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-01-24 10:38:50 -0800 |
commit | 73a48a46f83402669e654e8c0d0c297e2c5390a0 (patch) | |
tree | 09e1f9d42c000d54c31653dc9d8b9e570c41a597 /upb/msg.c | |
parent | 72ae34c28fb93ca1e45920d9e8714c74c7292dee (diff) | |
parent | 3b7dc27fb5497903145a52413116fad25fbaf138 (diff) |
Merge pull request #78 from haberman/axerefcount
Fixed amalgamated build and added test.
Diffstat (limited to 'upb/msg.c')
-rw-r--r-- | upb/msg.c | 23 |
1 files changed, 10 insertions, 13 deletions
@@ -26,7 +26,7 @@ void *upb_map_pack(const upb_map *map, void *p, size_t *ofs, size_t size); #define CHARPTR_AT(msg, ofs) ((char*)msg + ofs) #define ENCODE_MAX_NESTING 64 -#define CHECK_RETURN(x) if (!(x)) { return false; } +#define CHECK_TRUE(x) if (!(x)) { return false; } /** upb_msgval ****************************************************************/ @@ -492,7 +492,7 @@ struct upb_visitor { upb_sink *sink; }; -static upb_selector_t getsel(const upb_fielddef *f, upb_handlertype_t type) { +static upb_selector_t getsel2(const upb_fielddef *f, upb_handlertype_t type) { upb_selector_t ret; bool ok = upb_handlers_getselector(f, type, &ret); UPB_ASSERT(ok); @@ -572,34 +572,31 @@ static bool upb_visitor_visitmsg2(const upb_msg *msg, } else if (upb_fielddef_isstring(f)) { /* TODO putstr(); */ } else { - upb_selector_t sel = getsel(f, upb_handlers_getprimitivehandlertype(f)); + upb_selector_t sel = getsel2(f, upb_handlers_getprimitivehandlertype(f)); UPB_ASSERT(upb_fielddef_isprimitive(f)); switch (upb_fielddef_type(f)) { case UPB_TYPE_FLOAT: - CHECK_RETURN(upb_sink_putfloat(sink, sel, upb_msgval_getfloat(val))); + CHECK_TRUE(upb_sink_putfloat(sink, sel, upb_msgval_getfloat(val))); break; case UPB_TYPE_DOUBLE: - CHECK_RETURN( - upb_sink_putdouble(sink, sel, upb_msgval_getdouble(val))); + CHECK_TRUE(upb_sink_putdouble(sink, sel, upb_msgval_getdouble(val))); break; case UPB_TYPE_BOOL: - CHECK_RETURN(upb_sink_putbool(sink, sel, upb_msgval_getbool(val))); + CHECK_TRUE(upb_sink_putbool(sink, sel, upb_msgval_getbool(val))); break; case UPB_TYPE_ENUM: case UPB_TYPE_INT32: - CHECK_RETURN(upb_sink_putint32(sink, sel, upb_msgval_getint32(val))); + CHECK_TRUE(upb_sink_putint32(sink, sel, upb_msgval_getint32(val))); break; case UPB_TYPE_UINT32: - CHECK_RETURN( - upb_sink_putuint32(sink, sel, upb_msgval_getuint32(val))); + CHECK_TRUE(upb_sink_putuint32(sink, sel, upb_msgval_getuint32(val))); break; case UPB_TYPE_INT64: - CHECK_RETURN(upb_sink_putint64(sink, sel, upb_msgval_getint64(val))); + CHECK_TRUE(upb_sink_putint64(sink, sel, upb_msgval_getint64(val))); break; case UPB_TYPE_UINT64: - CHECK_RETURN( - upb_sink_putuint64(sink, sel, upb_msgval_getuint64(val))); + CHECK_TRUE(upb_sink_putuint64(sink, sel, upb_msgval_getuint64(val))); break; case UPB_TYPE_STRING: case UPB_TYPE_BYTES: |