Skip to content

urldata: simplify bytecounters - #3627

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/simplify-bytecounters
Closed

bagder wants to merge 1 commit into
masterfrom
bagder/simplify-bytecounters

Conversation

@bagder

@bagder bagder commented Feb 28, 2019

Copy link
Copy Markdown
Member
  • no need to have them protocol specific

  • no need to set pointers to them with the Curl_setup_transfer() call

  • make Curl_setup_transfer() operate on a transfer pointer, not
    connection

  • switch some counters from long to the more proper curl_off_t type

@bagder
bagder force-pushed the bagder/simplify-bytecounters branch from c50c636 to 7e8557b Compare February 28, 2019 12:45
@bagder
bagder force-pushed the bagder/simplify-bytecounters branch from 7e8557b to 3dc673b Compare February 28, 2019 12:54
- no need to have them protocol specific

- no need to set pointers to them with the Curl_setup_transfer() call

- make Curl_setup_transfer() operate on a transfer pointer, not
  connection

- switch some counters from long to the more proper curl_off_t type

Closes #3627
@bagder
bagder force-pushed the bagder/simplify-bytecounters branch from e658841 to a5ae6bf Compare February 28, 2019 19:48
@bagder bagder closed this in 65eb65f Mar 1, 2019
@bagder
bagder deleted the bagder/simplify-bytecounters branch March 1, 2019 16:31
@lock lock Bot locked as resolved and limited conversation to collaborators Jun 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant