>>>>> "Phil" == Phil Muldoon <pmuldoon@redhat.com> writes: Phil> I've replaced the BPPY_VALID_P macro with a static function in this Phil> patch. I also updated the s/ask/task typo. What do you think? Looks ok to me. Thanks for doing this. Tom