Re: [perl #132732] use if - behaviour does not match documentation

-----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,
Rob 

• [perl #132732] use if - behaviour does not match documentation by James E Keenan via RT
• Re: [perl #132732] use if - behaviour does not match documentation by sisyphus1
• [perl #132732] use if - behaviour does not match documentation by James E Keenan via RT