Mailing List Archive: remove use of indirect syntax with "import"

remove use of indirect syntax with "import"

Nov 7, 2023, 6:04 AM

Post #1 of 5 (128 views)

I've just had a PR raised against IO::Compress (https://github.com/pmqs/IO-Compress/pull/51). It to remove the indirect syntax used in the module. So "import Test::NoWarnings" becomes "Test::NoWarnings->import".

Is this a follow up to the changes made a while back to eliminate the use of "new Some::Module"?

cheers
Paul

Re: remove use of indirect syntax with "import" [ In reply to ]

perl5-porters at perl

Nov 7, 2023, 8:33 AM

Post #2 of 5 (128 views)

Permalink

Den 07.11.2023 15:04, skrev Paul Marquess:
>
> I’ve just had a PR raised against IO::Compress
> (https://github.com/pmqs/IO-Compress/pull/51). It to remove the
> indirect syntax used in the module. So “import Test::NoWarnings”
> becomes “Test::NoWarnings->import”.
>
> Is this a follow up to the changes made a while back to eliminate the
> use of “new Some::Module”?
>
I believe so. Elvin has been submitting a lot of PRs to core, and they
are also merged in, removing the usage of indirect syntax in core:

https://github.com/search?q=repo%3APerl%2Fperl5+indirect&type=commits

--

Nicolas Mendoza

Re: remove use of indirect syntax with "import" [ In reply to ]

rwp.primary at gmail

Nov 7, 2023, 7:54 PM

Post #3 of 5 (125 views)

Permalink

????

Elvin reacted via Gmail
<https://www.google.com/gmail/about/?utm_source=gmail-in-product&utm_medium=et&utm_campaign=emojireactionemail#app>

On Tue, Nov 7, 2023, 3:04 PM Paul Marquess <paul.marquess@outlook.com>
wrote:

> I’ve just had a PR raised against IO::Compress (
> https://github.com/pmqs/IO-Compress/pull/51). It to remove the indirect
> syntax used in the module. So “import Test::NoWarnings” becomes
> “Test::NoWarnings->import”.
>
>
>
> Is this a follow up to the changes made a while back to eliminate the use
> of “new Some::Module”?
>
>
>
>
>
> cheers
>
> Paul
>

Re: remove use of indirect syntax with "import" [ In reply to ]

rwp.primary at gmail

Nov 7, 2023, 7:54 PM

Post #4 of 5 (125 views)

Permalink

????

Elvin reacted via Gmail
<https://www.google.com/gmail/about/?utm_source=gmail-in-product&utm_medium=et&utm_campaign=emojireactionemail#app>

On Tue, Nov 7, 2023, 5:33 PM Nicolas Mendoza <mendoza@pvv.ntnu.no> wrote:

> Den 07.11.2023 15:04, skrev Paul Marquess:
>
> I’ve just had a PR raised against IO::Compress (
> https://github.com/pmqs/IO-Compress/pull/51). It to remove the indirect
> syntax used in the module. So “import Test::NoWarnings” becomes
> “Test::NoWarnings->import”.
>
>
>
> Is this a follow up to the changes made a while back to eliminate the use
> of “new Some::Module”?
>
>
>
> I believe so. Elvin has been submitting a lot of PRs to core, and they are
> also merged in, removing the usage of indirect syntax in core:
>
> https://github.com/search?q=repo%3APerl%2Fperl5+indirect&type=commits
>
> --
>
> Nicolas Mendoza
>

Re: remove use of indirect syntax with "import" [ In reply to ]

rwp.primary at gmail

Nov 7, 2023, 8:00 PM

Post #5 of 5 (125 views)

Permalink

Hello, and thanks for noticing that.

It's an independent effort to reduce indirect syntax in Core IO-Compress
being a Core distribution.

This PR followed Getopt-Long and Encode which were merged.

On Tue, Nov 7, 2023, 3:04 PM Paul Marquess <paul.marquess@outlook.com>
wrote:

> I’ve just had a PR raised against IO::Compress (
> https://github.com/pmqs/IO-Compress/pull/51). It to remove the indirect
> syntax used in the module. So “import Test::NoWarnings” becomes
> “Test::NoWarnings->import”.
>
>
>
> Is this a follow up to the changes made a while back to eliminate the use
> of “new Some::Module”?
>
>
>
>
>
> cheers
>
> Paul
>