Skip to content

Remove useless call to setContentLength()#6715

Merged
devyte merged 3 commits into
masterfrom
devyte-issue2132
Nov 6, 2019
Merged

Remove useless call to setContentLength()#6715
devyte merged 3 commits into
masterfrom
devyte-issue2132

Conversation

@devyte

@devyte devyte commented Nov 5, 2019

Copy link
Copy Markdown
Collaborator

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