This looks good to me. Reviewed-By: Derek Foreman <[email protected]>
On 19/03/15 03:58 AM, Jonas Ådahl wrote: > Without this patch, the scanner would generate invalid C which wouldn't > compile anyway, so lets be nice and fail earlier and point out where the > error is. > > Signed-off-by: Jonas Ådahl <[email protected]> > --- > src/scanner.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/scanner.c b/src/scanner.c > index 1f1e59a..64431a1 100644 > --- a/src/scanner.c > +++ b/src/scanner.c > @@ -551,6 +551,10 @@ end_element(void *data, const XML_Char *name) > strcmp(name, "event") == 0) { > ctx->message = NULL; > } else if (strcmp(name, "enum") == 0) { > + if (wl_list_empty(&ctx->enumeration->entry_list)) { > + fail(&ctx->loc, "enumeration %s was empty", > + ctx->enumeration->name); > + } > ctx->enumeration = NULL; > } > } > _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
