DecoderBuffer: export the struct

Eliminates the functions _new() and _free().

src/decoder/DecoderBuffer.cxx

@@ -21,38 +21,9 @@
 #include "DecoderBuffer.hxx"
 #include "DecoderAPI.hxx"
 #include "util/ConstBuffer.hxx"
-#include "util/DynamicFifoBuffer.hxx"
 
 #include <assert.h>
 
-struct DecoderBuffer {
-	Decoder *const decoder;
-	InputStream &is;
-
-	DynamicFifoBuffer<uint8_t> buffer;
-
-	DecoderBuffer(Decoder *_decoder, InputStream &_is,
-		      size_t _size)
-		:decoder(_decoder), is(_is), buffer(_size) {}
-};
-
-DecoderBuffer *
-decoder_buffer_new(Decoder *decoder, InputStream &is,
-		   size_t size)
-{
-	assert(size > 0);
-
-	return new DecoderBuffer(decoder, is, size);
-}
-
-void
-decoder_buffer_free(DecoderBuffer *buffer)
-{
-	assert(buffer != nullptr);
-
-	delete buffer;
-}
-
 const InputStream &
 decoder_buffer_get_stream(const DecoderBuffer *buffer)
 {

src/decoder/DecoderBuffer.hxx

@@ -21,38 +21,37 @@
 #define MPD_DECODER_BUFFER_HXX
 
 #include "Compiler.h"
+#include "util/DynamicFifoBuffer.hxx"
 
 #include <stddef.h>
 
+struct Decoder;
+class InputStream;
+template<typename T> struct ConstBuffer;
+
 /**
  * This objects handles buffered reads in decoder plugins easily.  You
  * create a buffer object, and use its high-level methods to fill and
  * read it.  It will automatically handle shifting the buffer.
  */
-struct DecoderBuffer;
+struct DecoderBuffer {
+	Decoder *const decoder;
+	InputStream &is;
 
-struct Decoder;
-class InputStream;
-
-template<typename T> struct ConstBuffer;
+	DynamicFifoBuffer<uint8_t> buffer;
 
 	/**
 	 * Creates a new buffer.
 	 *
- * @param decoder the decoder object, used for decoder_read(), may be nullptr
- * @param is the input stream object where we should read from
- * @param size the maximum size of the buffer
- * @return the new decoder_buffer object
+	 * @param _decoder the decoder object, used for decoder_read(),
+	 * may be nullptr
+	 * @param _is the input stream object where we should read from
+	 * @param _size the maximum size of the buffer
 	 */
-DecoderBuffer *
-decoder_buffer_new(Decoder *decoder, InputStream &is,
-		   size_t size);
-
-/**
- * Frees resources used by the decoder_buffer object.
- */
-void
-decoder_buffer_free(DecoderBuffer *buffer);
+	DecoderBuffer(Decoder *_decoder, InputStream &_is,
+		      size_t _size)
+		:decoder(_decoder), is(_is), buffer(_size) {}
+};
 
 gcc_pure
 const InputStream &

src/decoder/plugins/FaadDecoderPlugin.cxx

@@ -308,7 +308,7 @@ static std::pair<bool, SignedSongTime>
 faad_get_file_time(InputStream &is)
 {
 	DecoderBuffer *buffer =
-		decoder_buffer_new(nullptr, is,
+		new DecoderBuffer(nullptr, is,
 				  FAAD_MIN_STREAMSIZE * MAX_CHANNELS);
 	auto duration = faad_song_duration(buffer, is);
 	bool recognized = !duration.IsNegative();
@@ -326,7 +326,7 @@ faad_get_file_time(InputStream &is)
 		NeAACDecClose(decoder);
 	}
 
-	decoder_buffer_free(buffer);
+	delete buffer;
 
 	return std::make_pair(recognized, duration);
 }
@@ -415,7 +415,7 @@ static void
 faad_stream_decode(Decoder &mpd_decoder, InputStream &is)
 {
 	DecoderBuffer *buffer =
-		decoder_buffer_new(&mpd_decoder, is,
+		new DecoderBuffer(&mpd_decoder, is,
 				  FAAD_MIN_STREAMSIZE * MAX_CHANNELS);
 
 	/* create the libfaad decoder */
@@ -427,7 +427,7 @@ faad_stream_decode(Decoder &mpd_decoder, InputStream &is)
 	/* cleanup */
 
 	NeAACDecClose(decoder);
-	decoder_buffer_free(buffer);
+	delete buffer;
 }
 
 static bool