-----Original Message----- From: James E Keenan via RT Sent: Tuesday, January 30, 2018 10:10 AM To: OtherRecipients of perl Ticket #132732: Cc: perl5-porters@perl.org Subject: [perl #132732] use if - behaviour does not match documentation ..... >> Please review the new patch attached, 132732- >> 0001-if-module-clarify-documentation-and-test-more-thorou.patch. >> > > I plan to apply the patch on Thu, Feb 01 unless there is objection. Jim, That all looks pretty good to me, and tests fine. The last paragraph in the "SEE ALSO" section of the POD (which pre-dates your rewrite) threw me for a few seconds. My perldoc utility renders it simply as: provide can be used to select one of several possible modules to load, based on what version of Perl is running. Probably not so confusing if you know that there's a module named "provide" - but I didn't know that, and it's not part of core. Maybe something like: =~ s/L<provide>/The L<provide> module from CPAN/ Also, the end of the sentence ("based on what version of Perl is running") makes me grimace a bit, as do the replacements that I come up with - namely, "based on the version of Perl that is running" or just "based on the version of Perl". So I'm not going to make *any* suggestions about that. Thanks for being thorough ... and for doing all the work. Cheers, RobThread Previous | Thread Next