Skip to content

Remove useless call to setContentLength() #6715

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
Nov 6, 2019
Merged

Remove useless call to setContentLength() #6715

merged 3 commits into from
Nov 6, 2019

Conversation

devyte
Copy link
Collaborator

@devyte devyte commented Nov 5, 2019

Fixes #2132

@devyte devyte self-assigned this Nov 5, 2019
@devyte devyte added this to the 2.6.0 milestone Nov 5, 2019
@devyte devyte merged commit ffa9e3c into master Nov 6, 2019
@devyte devyte deleted the devyte-issue2132 branch November 6, 2019 00:50
devyte added a commit that referenced this pull request Nov 6, 2019
Apparently I messed up testing #6715 and something broke. This rolls back that PR.
That means #2132 remains closed with no action.
devyte added a commit that referenced this pull request Nov 6, 2019
Apparently I messed up testing #6715 and something broke. This rolls back that PR.
That means #2132 remains closed with no action.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shouldn't _contentLength / setContentLength be factored out?
1 participant