encoder/vorbis: reset the Ogg stream after flush

Without the ogg_stream_reset() call, the "e_o_s" flag never gets reset, and libogg writes EOS packets over and over.

encoder/vorbis: reset the Ogg stream after flush
b6303313 · Max Kellermann · 6dcec366 · b6303313 · b6303313
Commit b6303313 authored Mar 16, 2011 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

NEWS NEWS +2 -0

vorbis_encoder.c src/encoder/vorbis_encoder.c +2 -0

No files found.
--- a/NEWS
+++ b/NEWS
@@ -4,6 +4,8 @@ ver 0.16.2 (2011/??/??)
 * decoder:
  - tremor: fix configure test
  - gme: detect end of song
+* encoder:
+  - vorbis: reset the Ogg stream after flush
 * output:
  - httpd: fix uninitialized variable
  - httpd: include sys/socket.h

--- a/src/encoder/vorbis_encoder.c
+++ b/src/encoder/vorbis_encoder.c
@@ -276,6 +276,8 @@ vorbis_encoder_flush(struct encoder *_encoder, G_GNUC_UNUSED GError **error)
 	vorbis_analysis_init(&encoder->vd, &encoder->vi);
 	vorbis_block_init(&encoder->vd, &encoder->vb);

+	ogg_stream_reset(&encoder->os);
+
 	encoder->flush = true;
 	return true;
 }