Dan Smith wrote:
My plan was never to remove the placeholder, but rather to modify the placeholder with logic to look up and return the proper host system. This would give us a control point for switching between host instrumentation, with the option to keep our fake one if nothing else is available. Adding such a function seems like a good idea to me, and as long as it uses the existing internal logic in HostSystem to get the information, it shouldn't need to change later.
Shall I hold off on this set or apply and let you add the generalized function first?
Thanks!
Yes, I'll rework this patch. No need to have the same code in multiple
Kaitlin Rupert wrote: places if we can avoid it. =) -- Kaitlin Rupert IBM Linux Technology Center karupert@us.ibm.com