Skip to content

Correct max string length calculation when there are string operators #2292

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 3 commits into from
Jun 1, 2021

Conversation

bbugyi200
Copy link
Contributor

@bbugyi200 bbugyi200 commented May 31, 2021

PR #2286 did not fix the edge-cases (e.g. when the string is just long
enough to cause a line to be 89 characters long). This PR corrects that
mistake.

I also forgot to list the percent symbol as a "string operator".

bbugyi200 added 2 commits May 31, 2021 12:21
PR psf#2286 did not fix the edge-cases (e.g. when the string is just long
enough to cause a line to be 89 characters long). This PR corrects that
mistake.
@bbugyi200 bbugyi200 changed the title Correct max string length calculation after #2286 Correct max string length calculation when there are string operators May 31, 2021
@bbugyi200 bbugyi200 force-pushed the 21.05-fix-max-string-length-calc branch from 964fe52 to d00a729 Compare May 31, 2021 16:56
@JelleZijlstra JelleZijlstra merged commit a4e35b3 into psf:main Jun 1, 2021
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.

None yet

2 participants