Page MenuHomePhabricator

Ensure All Builtin Functions Throw an Error when Null Input, unless Null input is allowed
Closed, ResolvedPublic

Event Timeline

I'd like to push back on this one. Using a strongly typed functional language as a base, null should not be an allowed input to any function, since it does not match a ZType. I'd suggest that we catch null as a bad argument and throw a Z5 rather than gracefully fail within the builtin itself.

DVrandecic renamed this task from Ensure All Builtin Functions Can Handle Null Input to Ensure All Builtin Functions Throw an Error when Null Input, unless Null input is allowed.Sep 29 2021, 4:09 PM
DVrandecic triaged this task as Medium priority.Oct 20 2021, 4:38 PM

I can't find any evidence that functions can still receive null inputs; our argument validation should now catch any such cases. We can close this.