I was trying to point out that if a function is expected to return data.. maybe "Variable Data" describes it better, that No Data is a better indication of an error (and a reason to check the ERROR macro) than Error Data, which makes you figure out if what you got was Valid Data or Error Data.

Yes, that is exactly what I said in my earlier post. I know it's bad form to quote yourself but:

Richy says...
When the purpose of a function is to return data then it is reasonable to return either the data on success or a null value on an error - I agree that in this case that returning an error value is unwise.

However, this function never returns any data. It only ever returns an error state.