> >Comments: simple_feature_tweak shouldn't ignore the type argument it got >passed. And requesting macio_unknown in the heathrow IDE functions is just >a hack really, to make macio_find not care about the particular chip >type. The Lombard would need macio_paddington, true heathrow based >machines will need macio_heathrow here. > >macio_find either needs to check for compatible types, or we need to add >another set of paddington specific functions that use macio_paddington >instead of macio_ohare above (though I guess macio_ohare won't work on >heathrow either?). Kind of defeats the whole abstraction purpose. > >Leave it to BenH or Paulus to find a proper fix for this. I could test >this on Lombard only, no idea what macio type the Wallstreet uses.
Actually, your fix is correct and fixes obvious typos from me ;) simple_feature_tweak has to use the passed-in type, and passing macio_unknown is the right fix for heathrow_ide_xxxx as this routine is used for heathrow, gatwick and paddington. Thanks ! Ben.