Skip to content

lib: convert Curl_get_line to use dynbuf - #12846

Closed
bagder wants to merge 4 commits into
masterfrom
bagder/read-line-dynbuf
Closed

bagder wants to merge 4 commits into
masterfrom
bagder/read-line-dynbuf

Conversation

@bagder

@bagder bagder commented Feb 1, 2024

Copy link
Copy Markdown
Member

Create the line in a dynbuf. Aborts the reading of the file on errors. Avoids having to always allocate maximum amount from the start. Avoids direct malloc.

@github-actions github-actions Bot added the tests label Feb 1, 2024
Create the line in a dynbuf. Aborts the reading of the file on
errors. Avoids having to always allocate maximum amount from the
start. Avoids direct malloc.
@bagder
bagder force-pushed the bagder/read-line-dynbuf branch from e35d3f0 to 6d8677c Compare February 6, 2024 09:19
@bagder
bagder marked this pull request as ready for review February 6, 2024 12:11
@bagder bagder closed this in 142ac25 Feb 7, 2024
@bagder
bagder deleted the bagder/read-line-dynbuf branch February 7, 2024 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant