Skip to content
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

Handle partial writes in write_to_stream #401

Merged
merged 1 commit into from
Jan 5, 2018

Conversation

musteresel
Copy link
Contributor

g_output_stream_write may also return without an error if only part of the buffer has been written. This may happen when there's not enough storage in the stream, for example when writing a lot of data (like long paths / URIs). g_output_stream_write_all however only returns either after successfully writing all of the buffer or after an IO error.

g_output_stream_write may also return without an error if only part of
the buffer has been written.  This may happen when there's not enough
storage in the stream, for example when writing a lot of data (like
long paths / URIs).  g_output_stream_write_all however only returns
either after successfully writing all of the buffer or after an IO
error.
Copy link
Member

@keis keis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should fix the immediate problem. If we run into issues with blocking here we should perhaps look into using the _async version later.

@keis keis merged commit bdbb81a into uzbl:master Jan 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants