develooper Front page | perl.perl5.changes | Postings from June 2020

[Perl/perl5] 82b153: Suppress 2 runtime warnings

From:
James E Keenan
Date:
June 30, 2020 23:10
Subject:
[Perl/perl5] 82b153: Suppress 2 runtime warnings
Message ID:
Perl/perl5/push/refs/heads/io-core-p7-20200630/07fc80-e85b50@github.com
  Branch: refs/heads/io-core-p7-20200630
  Home:   https://github.com/Perl/perl5
  Commit: 82b1530c16dca02fcc23f8868bdfe4aa0edc97d5
      https://github.com/Perl/perl5/commit/82b1530c16dca02fcc23f8868bdfe4aa0edc97d5
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/argv.t

  Log Message:
  -----------
  Suppress 2 runtime warnings


  Commit: 621d004711893f946bb2b6ee08da04154d76b0bf
      https://github.com/Perl/perl5/commit/621d004711893f946bb2b6ee08da04154d76b0bf
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/argv.t

  Log Message:
  -----------
  Add descriptions to most tests lacking them


  Commit: 89d55f93168d144d6aff8b178ca7bfc9bf1525e5
      https://github.com/Perl/perl5/commit/89d55f93168d144d6aff8b178ca7bfc9bf1525e5
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/layers.t

  Log Message:
  -----------
  t/io/layers.t: Handle uninitialized warnings

Running with warnings enabled reveals 3 instances of uninitialized value
warnings.  The tied object is not defined even though the code appears
to assign a value to it.  Add tests that demonstrate this, then suppress
the uninitialized value warnings.


  Commit: 7994ae8a108ec92d8a273a8d88008cce00a21a3b
      https://github.com/Perl/perl5/commit/7994ae8a108ec92d8a273a8d88008cce00a21a3b
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/pipe.t

  Log Message:
  -----------
  t/io/pipe.t:  Suppress "Can't exec ..." warning

File did not previously run with warnings.  Once they're default, this
became apparent and unavoidable.


  Commit: d4176a1260c8a751c93db8b6b3c68f2fce12655e
      https://github.com/Perl/perl5/commit/d4176a1260c8a751c93db8b6b3c68f2fce12655e
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/print.t

  Log Message:
  -----------
  Suppress 3 warnings

File did not previously run with warnings.  Adapt.


  Commit: 2e58dfb56dcf409db09babb0e9aead60a0866de2
      https://github.com/Perl/perl5/commit/2e58dfb56dcf409db09babb0e9aead60a0866de2
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/read.t

  Log Message:
  -----------
  Suppress one 'used only once' warning


  Commit: bcaa690fa238ff5accdaf80c5d3b2951b571f613
      https://github.com/Perl/perl5/commit/bcaa690fa238ff5accdaf80c5d3b2951b571f613
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/read.t

  Log Message:
  -----------
  t/io/read.t:  Add descriptions and test for return values


  Commit: 93ceebd7bdf148ac11ac2ad059b69fdd1b5dd6fe
      https://github.com/Perl/perl5/commit/93ceebd7bdf148ac11ac2ad059b69fdd1b5dd6fe
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/say.t

  Log Message:
  -----------
  t/io/say.t: Suppress one "used only once" warning

Now that strict and warnings are on by default, use "no strict 'refs'"
only where it's absolutely needed for compilation.


  Commit: 9f15e1b7c138ef1e0a65c20ef611eff4a4c03bf0
      https://github.com/Perl/perl5/commit/9f15e1b7c138ef1e0a65c20ef611eff4a4c03bf0
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/tell.t

  Log Message:
  -----------
  t/io/tell.t:  Suppress warnings

Use "no strict 'refs'" only where it's absolutely needed for
compilation.


  Commit: e85b50a7f95e21d69b4e016470be685b941ad6f3
      https://github.com/Perl/perl5/commit/e85b50a7f95e21d69b4e016470be685b941ad6f3
  Author: James E Keenan <jkeenan@cpan.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M t/io/utf8.t

  Log Message:
  -----------
  t/io/utf8.t:  Suppress or capture warnings

Also, use "no strict" only where absolutely necessary.

Some blocks containing loops of tests had to be disaggregated to get a
clear focus on what was throwing warnings and what was not.


Compare: https://github.com/Perl/perl5/compare/07fc808dec36...e85b50a7f95e



nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About