> >> +int cdat_table_parse_sslbis(struct cdat_header *table,
> >> + cdat_tbl_entry_handler handler, void *arg)
> >
> > Feels like these ones should take a typed arg. Sure you'll loose
> > that again to use the generic handling code, but at this level we can
> > do it I think.
>
> while DSMAS and DSLBIS takes a list_head, SSLBIS takes an xarray. I can
> create a union.
I don't understand why, If you drop the macro usage introduced in
a later patch you can just have each one take the right thing.
That macro isn't a huge saving anyway.
Jonathan
>
> >
> >> +{
> >> + struct cdat_subtable_proc proc = {
> >> + .handler = handler,
> >> + .arg = arg,
> >> + };
> >> +
> >> + return cdat_table_parse_entries(CDAT_TYPE_SSLBIS, table, &proc);
> >> +}
> >
>
