Metadata: bool begin()

Author: pschatzmann

src/AudioTools/CoreAudio/AudioMetaData/AbstractMetaData.h

@@ -38,7 +38,7 @@ class AbstractMetaData {
         // defines the callback which provides the metadata information
         virtual void setCallback(void (*fn)(MetaDataType info, const char* str, int len)) = 0 ;
         // starts the processing
-        virtual void begin() = 0;
+        virtual bool begin() = 0;
         // ends the processing
         virtual void end() = 0;
         // provide audio data which contains the metadata to be extracted

src/AudioTools/CoreAudio/AudioMetaData/MetaDataICY.h

@@ -54,9 +54,10 @@ class MetaDataICY : public AbstractMetaData {
   }
 
   /// Resets all counters and restarts the prcessing
-  virtual void begin() override {
+  virtual bool begin() override {
     clear();
     LOGI("mp3_blocksize: %d", mp3_blocksize);
+    return true;
   }
 
   /// Resets all counters and restarts the prcessing

src/AudioTools/CoreAudio/AudioMetaData/MetaDataID3.h

@@ -119,11 +119,12 @@ class MetaDataID3V1  : public MetaDataID3Base {
     MetaDataID3V1() = default;
 
     /// (re)starts the processing
-    void begin() {
+    bool begin() {
         end();
         status = TagNotFound;
         use_bytes_of_next_write = 0;
         memset(tag_str, 0, 5);
+        return true;
     }
 
     /// Ends the processing and releases the memory
@@ -340,13 +341,14 @@ class MetaDataID3V2 : public MetaDataID3Base  {
     MetaDataID3V2() = default;
 
     /// (re)starts the processing
-    void begin() {
+    bool begin() {
         status = TagNotFound;
         use_bytes_of_next_write = 0;
         actual_tag = nullptr;
         tag_active = false;
         tag_processed = false;
         result.resize(result_size);
+        return true;
     }
 
     /// Ends the processing and releases the memory
@@ -582,10 +584,11 @@ class MetaDataID3 : public AbstractMetaData {
         this->filter = sel;
     }
 
-    void begin() {
+    bool begin() {
         TRACEI();
-        id3v1.begin();
-        id3v2.begin();
+        if (!id3v1.begin()) return false;
+        if (!id3v2.begin()) return false;
+        return true;
     }
 
     void end() {

src/AudioTools/CoreAudio/AudioPlayer.h

@@ -164,9 +164,19 @@ class AudioPlayer : public AudioInfoSupport, public VolumeSupport {
 
     // start dependent objects
     out_decoding.begin();
-    p_source->begin();
-    meta_out.begin();
-    volume_out.begin();
+
+    if (!p_source->begin()){
+      LOGE("Could not start audio source");
+      return false;
+    }
+    if (!meta_out.begin()){
+      LOGE("Could not start metadata output");
+      return false;
+    }
+    if (!volume_out.begin()){
+      LOGE("Could not start volume control");
+      return false;
+    }
 
     if (index >= 0) {
       setStream(p_source->selectStream(index));