Branch: refs/heads/davem/xscomments
  Home:   https://github.com/Perl/perl5
  Commit: c752269e22e4a9b897c9c64f83cf6e28f26e3e37
      
https://github.com/Perl/perl5/commit/c752269e22e4a9b897c9c64f83cf6e28f26e3e37
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-07-29 (Mon, 29 Jul 2024)

  Changed paths:
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/Constants.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/CountLines.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/Eval.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/Utilities.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/Typemaps.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/Typemaps/Cmd.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/Typemaps/InputMap.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/Typemaps/OutputMap.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/Typemaps/Type.pm
    M dist/ExtUtils-ParseXS/lib/perlxs.pod

  Log Message:
  -----------
  bump ExtUtils::ParseXS versions to 3.53


  Commit: ec1b011f41a093a579b266b2b65d810b7e59d336
      
https://github.com/Perl/perl5/commit/ec1b011f41a093a579b266b2b65d810b7e59d336
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-07-29 (Mon, 29 Jul 2024)

  Changed paths:
    M MANIFEST

  Log Message:
  -----------
  MANIFEST: point out XS-Typemap isn't installed

It's easy to confuse these two:

    XS::Typemap
    ExtUtils::Typemaps

Update the description of the XS::Typemap files in MANIFEST to aid
in deconfusion.


  Commit: 24382a39ec2f9d9028f50ff4780e1726eab0d94c
      
https://github.com/Perl/perl5/commit/24382a39ec2f9d9028f50ff4780e1726eab0d94c
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-07-29 (Mon, 29 Jul 2024)

  Changed paths:
    M lib/ExtUtils/typemap

  Log Message:
  -----------
  lib/ExtUtils/typemap: add some comments

Add a few lines to the top of this file explaining what its purpose is.


  Commit: fee5e5b933aa4792bf576f1657f015df25bf4cc7
      
https://github.com/Perl/perl5/commit/fee5e5b933aa4792bf576f1657f015df25bf4cc7
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-07-29 (Mon, 29 Jul 2024)

  Changed paths:
    M MANIFEST
    M Porting/exec-bit.txt
    A dist/ExtUtils-ParseXS/author/mksnapshot.pl

  Log Message:
  -----------
  ExtUtils-ParseXS: add author/mksnapshot.pl utility

Crude tool to collect before and after snapshots of all .c files created
by xsubpp, in order to see what (if anything) has changed in
functionality while working on ParseXS files.


  Commit: 339e005a3a31e47da916b678dbca0a2949ae9ed2
      
https://github.com/Perl/perl5/commit/339e005a3a31e47da916b678dbca0a2949ae9ed2
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-07-29 (Mon, 29 Jul 2024)

  Changed paths:
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/Utilities.pm

  Log Message:
  -----------
  ExtUtils::ParseXS: update pod for error handling

Consolidate the POD for the various warning- and error-raising methods
into a single block of text, to give a better overview of which method
should be used when (e.g. warning vs deferred errors vs die immediately)

The diff looks confusing, but its basically deleting the individual
chunk of pod directly above each method, then adding a complete new
block of pod. This contains completely new text, rather than copying
and pasting.


  Commit: 7244f6488fd013b68fa490990ed15515c8a64289
      
https://github.com/Perl/perl5/commit/7244f6488fd013b68fa490990ed15515c8a64289
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-07-30 (Tue, 30 Jul 2024)

  Changed paths:
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/Constants.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS/Utilities.pm
    M dist/ExtUtils-ParseXS/lib/ExtUtils/Typemaps.pm

  Log Message:
  -----------
  Extutils::ParseXS *.pm: add many code comments

To aid the understanding of this module and its sub-modules:

 - add lots of code comments
 - add blank lines
 - reformat and/or line-wrap a few long code lines

There should be no functional changes.

In particular, the line count of ParseXS.pm is increased by about 60%
with this commit.

I've tried to consistently use the word 'emit' rather than 'print' or
'output' in comments about the code that gets generated and ends up in
the .c file. At the moment most of this code code is indeed just
immediately printed to STDOUT, but in the longer term I would like to
separate out code generation and output stages.


Compare: https://github.com/Perl/perl5/compare/c752269e22e4%5E...7244f6488fd0

To unsubscribe from these emails, change your notification settings at 
https://github.com/Perl/perl5/settings/notifications

Reply via email to