Error handling

[lu-zero]

API surface :

• Errors as enums
• Errors as strings

Open questions

• Capture panics in unwind mode?

[dwbuiten]

rav1e_get_last_error(ctx)?

Or some sort of struct/string.

[lu-zero]

Yes, something along those lines.

The functions should all do something like this :

Return value	Status
0	Success
> 0	Recoverable
< 0	Fatal

And store the actual error in the Context.

So ideally if you want to have more information it should be provided by functions like:

const char *s = rav1e_get_last_error(ctx)

[dwbuiten]

That sounds good to me, though it would be nice to have something more akin to how we do it at FFMS2: https://github.com/FFMS/ffms2/blob/master/include/ffms.h#L89-L94

Maybe not exactly that struct, but some way to see an enum-like error is much easier to handle (e.g. EAGAIN) without strncmp. Opinion?