From kencotterill@netspace.net.au

This relates to if.pm (version​: 0.0606).

The current documentation refers to loading modules
with or without ARGUMENTS (i.e. for import()).

It doesn't mention using an empty import list
which is not possible with the current implementation.

This patch adds a few lines to the POD to
explain this; it also suggests a workaround.

--- if.pm	2017-04-12 08:26:15.000000000 +1000
+++ if-new.pm	2017-04-12 09:31:05.000000000 +1000
@@ -50,6 +50,15 @@
  If you don't use the fat comma (eg you don't have any ARGUMENTS),
  then you'll need to quote the MODULE.
  
+If you wanted ARGUMENTS to be an empty list, i.e. have the effect of:
+
+    use MODULE ();
+
+you can't do this with the C<if> pragma; however, you can achieve
+exactly this effect, at compile time, with:
+
+    BEGIN { require MODULE if CONDITION }
+
  =head2 EXAMPLES
  
  The following line is taken from the testsuite for L<File::Map>:

From @khwilliamson

On Wed, 12 Apr 2017 20​:21​:58 -0700, kencotterill@​netspace.net.au wrote​:

This relates to if.pm (version​: 0.0606).

The current documentation refers to loading modules
with or without ARGUMENTS (i.e. for import()).

It doesn't mention using an empty import list
which is not possible with the current implementation.

This patch adds a few lines to the POD to
explain this; it also suggests a workaround.

Thanks, I'll look into this for 5.27

--- if.pm 2017-04-12 08​:26​:15.000000000 +1000
+++ if-new.pm 2017-04-12 09​:31​:05.000000000 +1000
@​@​ -50,6 +50,15 @​@​
If you don't use the fat comma (eg you don't have any ARGUMENTS),
then you'll need to quote the MODULE.

+If you wanted ARGUMENTS to be an empty list, i.e. have the effect of​:
+
+ use MODULE ();
+
+you can't do this with the C<if> pragma; however, you can achieve
+exactly this effect, at compile time, with​:
+
+ BEGIN { require MODULE if CONDITION }
+
=head2 EXAMPLES

The following line is taken from the testsuite for L<File​::Map>​:

--
Karl Williamson

The RT System itself - Status changed from 'new' to 'open'

From @jkeenan

On Thu, 11 May 2017 15​:25​:50 GMT, khw wrote​:

On Wed, 12 Apr 2017 20​:21​:58 -0700, kencotterill@​netspace.net.au wrote​:

This relates to if.pm (version​: 0.0606).

The current documentation refers to loading modules
with or without ARGUMENTS (i.e. for import()).

It doesn't mention using an empty import list
which is not possible with the current implementation.

This patch adds a few lines to the POD to
explain this; it also suggests a workaround.

Thanks, I'll look into this for 5.27

Karl, would you like to move forward with the discussion of this request?

Thank you very much.

--
James E Keenan (jkeenan@​cpan.org)

From @khwilliamson

Thanks, applied as
1627961

It makes it easier on us if you attach the patch to the ticket, instead of inlining it.
--
Karl Williamson

@khwilliamson - Status changed from 'open' to 'pending release'

From @khwilliamson

Thank you for filing this report. You have helped make Perl better.

With the release yesterday of Perl 5.28.0, this and 185 other issues have been
resolved.

Perl 5.28.0 may be downloaded via​:
https://metacpan.org/release/XSAWYERX/perl-5.28.0

If you find that the problem persists, feel free to reopen this ticket.

@khwilliamson - Status changed from 'pending release' to 'resolved'