Branch: refs/heads/davem/xscomments
  Home:   https://github.com/Perl/perl5
  Commit: 18a602866084154d4f29f84d70a468c7b7e8fb7f
      
https://github.com/Perl/perl5/commit/18a602866084154d4f29f84d70a468c7b7e8fb7f
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-08-01 (Thu, 01 Aug 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: bd9fc9f5f769b710a53e30d089c39871abeb1e5b
      
https://github.com/Perl/perl5/commit/bd9fc9f5f769b710a53e30d089c39871abeb1e5b
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-08-01 (Thu, 01 Aug 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: 1f63227b105f69097978fecd9dd8eb1bbfe34bde
      
https://github.com/Perl/perl5/commit/1f63227b105f69097978fecd9dd8eb1bbfe34bde
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-08-01 (Thu, 01 Aug 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: 42944d3208f1fd7d11c54cd8d8e6518d54256d8b
      
https://github.com/Perl/perl5/commit/42944d3208f1fd7d11c54cd8d8e6518d54256d8b
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-08-01 (Thu, 01 Aug 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: af7f95477403a3800e85cfecf0c968aa35fd5bd6
      
https://github.com/Perl/perl5/commit/af7f95477403a3800e85cfecf0c968aa35fd5bd6
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-08-01 (Thu, 01 Aug 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: ddfb8338d7d16ce48721f0d9cef7fe20645c27c8
      
https://github.com/Perl/perl5/commit/ddfb8338d7d16ce48721f0d9cef7fe20645c27c8
  Author: David Mitchell <da...@iabyn.com>
  Date:   2024-08-01 (Thu, 01 Aug 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/7244f6488fd0...ddfb8338d7d1

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

Reply via email to