insane: Drop oe.qa.add_message usage

insane: Drop oe.qa.add_message usage

Drop the oe.qa.add_message() usage in favour of oe.qa.handle_error() which has
code allowing it to be optimised with contains usage.

The patch also drops unused return values which we stopped using a while ago
and drops the now unneeded function parameters, generally leading to cleaner
code.

The code should be functionally equivalent.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>