Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #9.
ogg_sync_destroy
freesogg->state
, which is then freed again byfree(ogg)
. By allocating this separately and letting libogg free it this is avoided.ogg_stream_init
allocates 3 buffers, which are not properly freed. By separately allocatingogg->stream
and callingogg_stream_destroy
, these are properly freed. Saves ~26kb of lost memory.Finally, libogg recommends
ogg_sync_pageout
be called beforeogg_sync_buffer
to initialise some buffers. Added a call to it insideAVOggInit
to follow this recommendation.