Branch: refs/heads/davem/xscomments
  Home:   https://github.com/Perl/perl5
  Commit: 7bf8a2d3f140fae623a98a7f9e9ec877503b8361
      
https://github.com/Perl/perl5/commit/7bf8a2d3f140fae623a98a7f9e9ec877503b8361
  Author: David Mitchell <[email protected]>
  Date:   2024-08-02 (Fri, 02 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: 8c96e6b0c00b6da70de914d7d949e21532b53e06
      
https://github.com/Perl/perl5/commit/8c96e6b0c00b6da70de914d7d949e21532b53e06
  Author: David Mitchell <[email protected]>
  Date:   2024-08-02 (Fri, 02 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: 612db79ea80f833a9e69b165be0cba53f0092686
      
https://github.com/Perl/perl5/commit/612db79ea80f833a9e69b165be0cba53f0092686
  Author: David Mitchell <[email protected]>
  Date:   2024-08-02 (Fri, 02 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: 5ff8d8ccd6667275ba344d1a11b3ff635525e267
      
https://github.com/Perl/perl5/commit/5ff8d8ccd6667275ba344d1a11b3ff635525e267
  Author: David Mitchell <[email protected]>
  Date:   2024-08-02 (Fri, 02 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: 526eea3e0a15d98911b20ca835cb60819ed12193
      
https://github.com/Perl/perl5/commit/526eea3e0a15d98911b20ca835cb60819ed12193
  Author: David Mitchell <[email protected]>
  Date:   2024-08-02 (Fri, 02 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: 82436f91ec2ddb591be434fe8e32774321b9f6a9
      
https://github.com/Perl/perl5/commit/82436f91ec2ddb591be434fe8e32774321b9f6a9
  Author: David Mitchell <[email protected]>
  Date:   2024-08-02 (Fri, 02 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/ddfb8338d7d1...82436f91ec2d

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

Reply via email to