>>>>> "Phil" == Phil Muldoon <pmuldoon@redhat.com> writes: Phil> I'm not sure what to do in the 'to_string' returns 'None' and the Phil> missing 'children' method above. I think it is fine as-is. People just shouldn't do this, just like they shouldn't write printers that fail in other weird ways. This patch is ok. Thanks. Tom