Bug #121013 for Scalar-List-Utils: can we add documentation and tests for interaction of the various functions with undef as argument?

Fri Apr 07 16:31:46 2017 MITHALDU [...] cpan.org - Ticket created

Subject:

can we add documentation and tests for interaction of the various functions with undef as argument?

Due to LU being XS it is a bit of a black box, and "check the code and reason it out" doesn't work so well for figuring out what it does with undefined arguments. The tests themselves don't ensure that a certain behavior happens either. As such, can we define what the functions *should* do, document that, and add tests? I'm happy to write the tests if needed.

Fri Apr 07 16:56:33 2017 MITHALDU [...] cpan.org - Correspondence added

Related: Handling of strings is also involved, so a general bit about non-numerical inputs is warranted. Brainstorming is happening in IRC, will add results tomorrow.

Fri Apr 07 16:57:46 2017 leonerd-cpan [...] leonerd.org.uk - Correspondence added

21:55 <Grinnz> "Non-numerical values, such as undef and strings, are compared as with core numeric operators; namely by treating them as zero and raising a warning in the numeric category" -- Paul Evans

Fri Apr 07 16:57:47 2017 The RT System itself - Status changed from 'new' to 'open'

Sun Apr 09 11:34:51 2017 MITHALDU [...] cpan.org - Correspondence added

I wrote a number of tests for functions where it seemed like having undef arguments would be usefully different from other types of arguments. https://github.com/Dual-Life/Scalar-List-Utils/pull/54 If i missed out on any that could deserve some tests, i'm happy to add them. I also marked a number of them with XXX where i thought they might deserve double-checking of whether the behavior is as expected. And lastly, it includes a bit of documentation about the behavior checked by these tests. Feedback on that much appreciated.