>typedef enum tcstatus_ TCStatus;
>enum tcstatus_
> TC_ERROR = -1, /* unrecoverable error */
> TC_OK = 0, /* all fine */
> TC_RETRY, /* generic recoverable error */
> /* add more recoverable errors here */
>};
I don't know that we need to make things too complicated, though it
probably couldn't hurt to go over the set of returns we expect to see.
I'm personally still partial to the idea of returning a simple boolean
value to indicate success or failure, and (for functions that need it)
return a detailed status code in a separate pass-by-reference parameter.
[...]
OK, that's fine for me, so let's limit our enum to TC_OK and TC_ERROR.
Last issue: TC_OK should be 0 (zero) or !0 (not zero)?
In that case too I like more the old way (return is error code, so return 0 -> no errors)
but I'm fine in both ways.
Best regards,
Francesco Romani
