On 4/21/23 9:06 AM, Jonathan Cameron wrote:
+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.
Oh I think I understand where you are trying to get at. Ok I'll update.
Jonathan
+{
+ struct cdat_subtable_proc proc = {
+ .handler = handler,
+ .arg = arg,
+ };
+
+ return cdat_table_parse_entries(CDAT_TYPE_SSLBIS, table, &proc);
+}
