On 02/26/10 10:30, Markus Armbruster wrote:
This made a lot of sense if each parser threw largely different errors.
Which isn't the case: we have one so far ("failed to parse"), and we
just encountered a use for a second.
There are actually three: parse error, not found (netdev=<does-not-exist>) and busy (netdev=<already-in-use>).
But, yes, given the small number of error cases it makes sense to generate the error one layer up and not in the parsers themself.
cheers, Gerd
