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

Ensure decompressed metadata is null-terminated #36

Merged
merged 1 commit into from May 2, 2016
Merged

Ensure decompressed metadata is null-terminated #36

merged 1 commit into from May 2, 2016

Conversation

neilmayhew
Copy link
Contributor

@neilmayhew neilmayhew commented May 2, 2016

Fixes #35, Refresh hangs


This change is Reviewable

@neilmayhew
Copy link
Contributor Author

I realized that g_strndup was a cleaner and more efficient way to do it.

@ximion ximion merged commit 9b9f479 into ximion:master May 2, 2016
@ximion
Copy link
Owner

ximion commented May 3, 2016

I made some more general improvements on that code now, it should be much easier to read and also be a bit more friendly to memory now (the previous code was in some occasions duplicating the data twice, which didn't make any sense).

@neilmayhew neilmayhew mentioned this pull request May 10, 2016
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

2 participants