Details
-
Task
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
Description
NULL_IF_NULL functions that can return errors eg. divide currently look at the validity bits in each function (to avoid returning spurious errors).
divide(TYPE in1, boolean is_valid1, TYPE in2, boolean is_valid2, ..) { if (!is_valid1 || !is_valid2) { return 0; } if (in2 == 0) { /* set error */ } }
This validity check is duplicated for multiple functions and should be moved to the common layer (for all NULL_IF_NULL functions that can return error).
Attachments
Issue Links
- links to