[Perl/perl5] 49733c: Rename some internal functions in op.c for clarity

[email protected] (Paul Evans via perl5-changes) Mon, 27 Jul 2026 03:38:20 -0700
Newsgroups perl.perl5.changes
Message-ID <Perl/perl5/push/refs/heads/blead/[email protected]>
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 49733c3683672e8f1b009562c442ce21820c8245
      https://github.com/Perl/perl5/commit/49733c3683672e8f1b009562c442ce21820c8245
  Author: Paul "LeoNerd" Evans <[email protected]>
  Date:   2026-07-27 (Mon, 27 Jul 2026)

  Changed paths:
    M embed.fnc
    M embed.h
    M op.c
    M proto.h

  Log Message:
  -----------
  Rename some internal functions in op.c for clarity

Of the previous names, `my_kid` was particularly opaque as to its
meaning. This has now been renamed to `declare_var_attributes`, to
suit what the function actually does. I am also trying to reserve the
verb "apply" for specifically the early compiletime effects of an
attribute.

I also renamed the `imopsp` argument to a slightly more meaningful
`import_opsp`, a pointer to a list of import ops.

While I was there I also removed a now-pointless block in the
`import_attributes_module` function that was needed before C99 to apply
the arguments check macro before declaring new variables.


  Commit: bbd6721ffa64903a72d017c5bdf23f00c8addf8e
      https://github.com/Perl/perl5/commit/bbd6721ffa64903a72d017c5bdf23f00c8addf8e
  Author: Paul "LeoNerd" Evans <[email protected]>
  Date:   2026-07-27 (Mon, 27 Jul 2026)

  Changed paths:
    M op.c

  Log Message:
  -----------
  op.c: MGf_LOCAL is always defined so no point #ifdef-wrapping code


Compare: https://github.com/Perl/perl5/compare/d97ad8ae5549...bbd6721ffa64

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