tag/Id3Scan: return std::unique_ptr<Tag>

src/decoder/plugins/MadDecoderPlugin.cxx

@@ -351,10 +351,10 @@ MadDecoder::ParseId3(size_t tagsize, Tag **mpd_tag)
 	}
 
 	if (mpd_tag) {
-		Tag *tmp_tag = tag_id3_import(id3_tag);
+		auto tmp_tag = tag_id3_import(id3_tag);
 		if (tmp_tag != nullptr) {
 			delete *mpd_tag;
-			*mpd_tag = tmp_tag;
+			*mpd_tag = tmp_tag.release();
 		}
 	}
 

src/tag/Id3Scan.cxx

@@ -330,14 +330,14 @@ scan_id3_tag(struct id3_tag *tag,
 	tag_id3_import_ufid(tag, handler, handler_ctx);
 }
 
-Tag *
+std::unique_ptr<Tag>
 tag_id3_import(struct id3_tag *tag)
 {
 	TagBuilder tag_builder;
 	scan_id3_tag(tag, add_tag_handler, &tag_builder);
 	return tag_builder.empty()
 		? nullptr
-		: tag_builder.CommitNew().release();
+		: tag_builder.CommitNew();
 }
 
 bool

src/tag/Id3Scan.hxx

@@ -22,6 +22,8 @@
 
 #include "check.h"
 
+#include <memory>
+
 class InputStream;
 struct TagHandler;
 struct Tag;
@@ -31,7 +33,7 @@ bool
 tag_id3_scan(InputStream &is,
 	     const TagHandler &handler, void *handler_ctx);
 
-Tag *
+std::unique_ptr<Tag>
 tag_id3_import(id3_tag *);
 
 /**