rhuij...@apache.org wrote on Mon, Apr 13, 2015 at 14:28:39 -0000:
> -      *has_props = (noderev->prop_rep->expanded_size > 4);
> +      *has_props = (noderev->prop_rep->expanded_size > 4
> +                    || (noderev->prop_rep->expanded_size == 0
> +                        && noderev->prop_rep->size > 4));

Having to remember, on every use of EXPANDED_SIZE, to check if it's zero
is madness.  Is there a good reason not to make the deserializer
function handle this?  If needed, by inventing a getter or a new struct
member that doesn't have this quirk?

Reply via email to