Skip Menu |

This queue is for tickets about the List-MoreUtils CPAN distribution.

Report information
The Basics
Id: 63822
Status: resolved
Priority: 0/
Queue: List-MoreUtils

People
Owner: Nobody in particular
Requestors: IRONCAMEL [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.29
Fixed in: (no value)



Subject: Documentation for 'all' function is incorrect for empty lists.
Documentation for 'all' function is incorrect with respect to empty lists. It reads: Returns false otherwise, or if LIST is empty. I believe it should read: Returns true if LIST is empty. Returns false otherwise.
Proposed documentation tweak committed as SVN revision 13602. Tom Wyant
On Sun Jan 16 17:16:53 2011, WYANT wrote: Show quoted text
> Proposed documentation tweak committed as SVN revision 13602. > > Tom Wyant
Proposed (and accepted?) tweak: Returns a true value if any item in LIST meets the criterion given through BLOCK. <CODE FRAG> Returns true if LIST is empty. Returns false otherwise. The proposed documentation tweak makes the function sound like it *only* tests the length of the list. (Linguistically, I think it's a dangling "otherwise" clause. Does it relate to the previous sentence or the one above the code fragment?) I'd suggest: Returns false otherwise, or true if LIST is empty. Heck... that's confusing, too. How 'bout a final line like: Returns false otherwise (unless LIST is empty, in which case it returns true). This final sentence is more parallel with other sentence structures. I think it's very important to keep parallel sentence structure. The proposed solution does not maintain this parallel structure. Nor does my first suggestion. The final suggestion is (mostly) parallel, but the difference in structure points out a difference in behavior, not merely a difference in wording. My $0.02 - hope it helps.
Hi, proving all RT's is scheduled after I finished Proc::ProcessTable rewrite. Please understand that I will try to fix as many issues as possible with one real developing action.
Fixed with 0.401