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

regexp.h apidoc was misspelled #22408

Merged
merged 1 commit into from
Jul 17, 2024
Merged

Conversation

khwilliamson
Copy link
Contributor

No description provided.

@khwilliamson khwilliamson requested a review from demerphq July 15, 2024 23:13
@khwilliamson
Copy link
Contributor Author

The more I look at this, I suspect that the misspelling was intentional, and that this is not intended to be documented at this time. @demerphq please clarify

Copy link
Collaborator

@demerphq demerphq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, I don't think this was deliberate.

@jkeenan
Copy link
Contributor

jkeenan commented Jul 17, 2024

The more I look at this, I suspect that the misspelling was intentional, and that this is not intended to be documented at this time. @demerphq please clarify

Here's the original commit:

ommit 1f6e74eb3536a13f38c1016477cd5b62dfe4e3da
Author:     Karl Williamson <[email protected]>
AuthorDate: Sat Oct 17 07:26:56 2020 -0600
Commit:     Karl Williamson <[email protected]>
CommitDate: Wed Dec 30 07:08:52 2020 -0700

    Document regexp, regmatch_info

diff --git a/regexp.h b/regexp.h
index cfb8d443ce..43a5168f74 100644
--- a/regexp.h
+++ b/regexp.h
@@ -97,6 +97,7 @@ struct reg_code_blocks {
 
 
 /*
+= for apidoc AyT||regexp
...

@khwilliamson
Copy link
Contributor Author

Well, it looks like it was me who screwed up back then

@khwilliamson khwilliamson merged commit a075295 into Perl:blead Jul 17, 2024
33 checks passed
@khwilliamson khwilliamson deleted the regexp_api branch July 17, 2024 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants