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

Constify some missed static local variables (in src/perl, src/test) #1338

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

ac000
Copy link
Member

@ac000 ac000 commented Jun 21, 2024

These somehow got missed in my previous set of patches to constify the likes of

static nxt_str_t  foo_str = nxt_string("foobar");

NOTE: There is a also a patch do similar in the Python language module which is waiting for #1336 to be merged.

@ac000 ac000 marked this pull request as ready for review June 24, 2024 02:39
@ac000 ac000 requested a review from hongzhidao June 24, 2024 02:39
ac000 added 2 commits June 24, 2024 22:30
These somehow got missed in my previous constification patches...

Signed-off-by: Andrew Clayton <[email protected]>
These somehow got missed in my previous constification patches...

Signed-off-by: Andrew Clayton <[email protected]>
@ac000
Copy link
Member Author

ac000 commented Jun 24, 2024

Rebased with master

@ac000 ac000 merged commit ab1b3f9 into nginx:master Jun 24, 2024
21 of 23 checks passed
@ac000 ac000 deleted the constify branch June 24, 2024 21:33
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.

2 participants