Skip to content

Conversation

@Rodrigo-Novas
Copy link
Contributor

Related to: #3117

@Rodrigo-Novas
Copy link
Contributor Author

I don't know if this small change warrants a changelog entry.

@pquentin pquentin added the Skip Changelog Pull requests that don't require a changelog entry label Sep 5, 2023
@pquentin
Copy link
Member

pquentin commented Sep 5, 2023

Well we could go one step further and write flush_decoder = amt is None or (amt != 0 and not data). I'll let someone else weigh in as I wrote this code initially and am not sure if any of this makes a big difference.

@Rodrigo-Novas
Copy link
Contributor Author

Well we could go one step further and write flush_decoder = amt is None or (amt != 0 and not data). I'll let someone else weigh in as I wrote this code initially and am not sure if any of this makes a big difference.

Yes good catch!. I think it is not a core change but having the code without elifs makes it more readable and without so much redundancy.

@pquentin
Copy link
Member

Closing given the lack of interest, sorry.

@pquentin pquentin closed this Sep 13, 2023
@pquentin pquentin reopened this Sep 14, 2023
@pquentin pquentin changed the title refactor: change double conditional to one Refactor double conditional into one Sep 15, 2023
@pquentin pquentin merged commit af7c78f into urllib3:main Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog Pull requests that don't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants