Skip to content

lib/unicore/mktables: remove unused variable #23541

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 5, 2025

Conversation

jkeenan
Copy link
Contributor

@jkeenan jkeenan commented Aug 5, 2025

Variable '$full_filename' is declared and assigned to, but not used anywhere else in the program. The string '$full_filename' is found nowhere else in the core distribution. Hence, the variable is superfluous.

After making that change, had to run porting programs:

$ ./perl -Ilib regen/regcharclass.pl
Changed: regcharclass.h
$ ./perl -Ilib regen/mk_invlists.pl
Changed: regexp_constants.h charclass_invlists.inc
lib/unicore/uni_keywords.pl uni_keywords.h


  • This set of changes does not require a perldelta entry.

Variable '$full_filename' is declared and assigned to, but not used
anywhere else in the program.  The string '$full_filename' is found
nowhere else in the core distribution.  Hence, the variable is
superfluous.

After making that change, had to run porting programs:

  $ ./perl -Ilib regen/regcharclass.pl
  Changed: regcharclass.h
  $ ./perl -Ilib regen/mk_invlists.pl
  Changed: regexp_constants.h charclass_invlists.inc
           lib/unicore/uni_keywords.pl uni_keywords.h
@jkeenan jkeenan requested a review from khwilliamson August 5, 2025 01:37
@jkeenan jkeenan added the Infrastructure Things needed to maintain Perl development label Aug 5, 2025
@jkeenan jkeenan merged commit 9df4510 into Perl:blead Aug 5, 2025
34 checks passed
@jkeenan jkeenan deleted the unused-subroutine-20250804 branch August 5, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Things needed to maintain Perl development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants