Skip to content

Optimized some strlen() calls when possible #42158

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 17, 2021
Merged

Optimized some strlen() calls when possible #42158

merged 1 commit into from
Jul 17, 2021

Conversation

javiereguiluz
Copy link
Member

Q A
Branch? 4.4
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Done with PhpStorm refactoring tools and only when it's safe to do.

@javiereguiluz javiereguiluz added this to the 4.4 milestone Jul 16, 2021
@javiereguiluz javiereguiluz requested a review from chalasr as a code owner July 16, 2021 13:33
@Tobion
Copy link
Contributor

Tobion commented Jul 17, 2021

Thank you @javiereguiluz.

@Tobion Tobion merged commit 482d2ce into symfony:4.4 Jul 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stof stof stof approved these changes

@derrabus derrabus derrabus approved these changes

@chalasr chalasr Awaiting requested review from chalasr chalasr is a code owner

+1 more reviewer

@Tobion Tobion Tobion approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
5 participants
@javiereguiluz @Tobion @stof @derrabus @carsonbot