Skip to content
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

Study changes to t/re/*.t (2 of 3) #244

Closed
jkeenan opened this issue Aug 20, 2020 · 4 comments
Closed

Study changes to t/re/*.t (2 of 3) #244

jkeenan opened this issue Aug 20, 2020 · 4 comments
Assignees
Labels
study Study changes made by developers since 5.32.0

Comments

@jkeenan
Copy link
Collaborator

jkeenan commented Aug 20, 2020

In this ticket we will study our progress in pursuit of Objective 2, "strict-by-default", for the following files in the Perl core distribution:

t/re/pat_psycho.t
t/re/qr-72922.t
t/re/recompile.t
t/re/reg_60508.t
t/re/reg_eval.t
t/re/reg_eval_scope.t
t/re/reg_namedcapture.t
t/re/regex_sets.t
t/re/regex_sets_compat.t
t/re/regexp_noamp.t
t/re/regexp_nonull.t

These files should be evaluated in light of the guidelines presented on this wiki page. Please post the summary of your findings in a single comment in this issue.

First-time contributors: (1) Please post a comment to this ticket indicating you are working on it. That will enable us to assign the ticket to you. (GitHub won't let us do that until it already thinks you're part of the project.) (2) You should subscribe to this repository so that you get email updates.

Thank you for helping to improve the Perl programming language.

@jkeenan jkeenan added the study Study changes made by developers since 5.32.0 label Aug 20, 2020
@brainbuz
Copy link
Collaborator

I'll take it.

@brainbuz
Copy link
Collaborator

file pass strictures not globally specified reviewed diffs  
t/re/pat_psycho.t x --- x strict line 10 use 5.010 line 12
t/re/qr-72922.t x x x  
t/re/recompile.t x --- x strict line 19
t/re/reg_60508.t x x x  
t/re/reg_eval.t x x x  
t/re/reg_eval_scope.t x x x  
t/re/reg_namedcapture.t x x x  
t/re/regex_sets.t x --- x strict line 18
t/re/regex_sets_compat.t x x x  
t/re/regexp_noamp.t x x x  
t/re/regexp_nonull.t x x x  

@atoomic
Copy link
Owner

atoomic commented Sep 25, 2020

thanks a lot @brainbuz for your work and research

jkeenan added a commit that referenced this issue Sep 25, 2020
Issue #244 study for objective strict on default,
@jkeenan
Copy link
Collaborator Author

jkeenan commented Sep 26, 2020

Files have been studied and flaws addressed. Closing this ticket.

Thank you very much.
Jim Keenan

@jkeenan jkeenan closed this as completed Sep 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
study Study changes made by developers since 5.32.0
Projects
None yet
Development

No branches or pull requests

3 participants