jni memory leak fix

app/src/main/cpp/CoverArtRepository.h

@@ -80,13 +80,6 @@ namespace repository {
 
         std::stringstream buff(data);
         std::vector<char> vals(data.begin(), data.end());
-        //vals.reserve(data.size());
-        //vals.assign(data.begin(), data.end());
-        /**
-        vals.assign(std::istream_iterator<unsigned char>(buff),
-                std::istream_iterator<unsigned char>());
-         */
-        //std::copy(data.begin(), data.end(), &vals);
 
         return vals;
     }

app/src/main/cpp/Demo.cpp

@@ -79,6 +79,12 @@ jobject songToObj(JNIEnv *env, const model::Song& song)
     env->CallVoidMethod(songObj, songYear, year);
     env->CallVoidMethod(songObj, songCoverArtId, coverArtId);
 
+    env->DeleteLocalRef(title);
+    env->DeleteLocalRef(album);
+    env->DeleteLocalRef(albumArtist);
+    env->DeleteLocalRef(artist);
+    env->DeleteLocalRef(genre);
+
     return songObj;
 }
 
@@ -290,8 +296,12 @@ Java_com_example_mear_repositories_TrackRepository_retrieveSongs(
     auto allSongs = retrieveSongs(tk, uri);
     jobjectArray songs = env->NewObjectArray(allSongs.size(), songClass, nullptr);
     for (auto i = 0; i != allSongs.size(); ++i) {
+        try {
             auto song = songToObj(env, allSongs[i]);
             env->SetObjectArrayElement(songs, i, song);
+        } catch (std::exception& ex) {
+            auto msg = ex.what();
+        }
     }
 
     return songs;

app/src/main/cpp/SongRepository.cpp

@@ -15,7 +15,7 @@ namespace repository {
         if (fullUri.at(fullUri.size()-1) != '/') {
             fullUri.append("/");
         }
-        //fullUri.append("api/v1/song");
+
         fullUri.append(songRecordEndpoint());
         std::vector<model::Song> songs;
 
@@ -120,11 +120,4 @@ namespace repository {
         ((std::string*)e)->append((char*)ptr, size * nmemb);
         return size * nmemb;
     }
-
-
-    /**
-    constexpr auto SongRepository::songRecordEndpoint() noexcept {
-        return "api/v1/song";
-    }
-     */
 }