Improve performance of functions counting the number of plays

See #22
<#22 (comment)>

It looks worse and I think I've done this the first time, then went
to creating aspects, dervied Eq for them, and compared those directly
for it to look nicer. But seems like allocating a new aspect every time
is very time-consuming. Comparing Strings directly without re-allocating
anything is the way to go apparently.

For normal single usage it doesn't matter. But I discovered this while
working on plots and there I have to call such a function for every data
point (bc I don't want to lower resolution it's equivalent to every time
the aspect exists in the dataset heh). There it makes the code 6x faster

src/display.rs

@@ -377,9 +377,7 @@ fn gather_artist(entries: &Vec<SongEntry>, art: &Artist) -> ArtistPlays {
     let mut artist_asp = ArtistPlays(art.clone(), 0);
 
     for entry in entries {
-        let artist = Artist::new(entry.artist.clone());
-
-        if artist == *art {
+        if entry.artist.eq(&art.name) {
             artist_asp.1 += 1;
         }
     }
@@ -392,9 +390,7 @@ fn gather_album(entries: &Vec<SongEntry>, alb: &Album) -> AlbumPlays {
     let mut album_asp = AlbumPlays(alb.clone(), 0);
 
     for entry in entries {
-        let album = Album::new(entry.album.clone(), entry.artist.clone());
-
-        if album == *alb {
+        if entry.artist.eq(&alb.artist.name) && entry.album.eq(&alb.name) {
             album_asp.1 += 1;
         }
     }
@@ -407,13 +403,10 @@ fn gather_song(entries: &Vec<SongEntry>, son: &Song) -> SongPlays {
     let mut song_asp = SongPlays(son.clone(), 0);
 
     for entry in entries {
-        let song = Song::new(
-            entry.track.clone(),
-            entry.album.clone(),
-            entry.artist.clone(),
-        );
-
-        if song == *son {
+        if entry.artist.eq(&son.album.artist.name)
+            && entry.album.eq(&son.album.name)
+            && entry.track.eq(&son.name)
+        {
             song_asp.1 += 1;
         }
     }

src/display/date.rs

@@ -97,9 +97,7 @@ fn gather_artist_date(
     let mut artist_asp = ArtistPlays(art.clone(), 0);
 
     for entry in entries {
-        let artist = Artist::new(entry.artist.clone());
-
-        if entry.timestamp.ge(start) && entry.timestamp.le(end) && artist == *art {
+        if entry.timestamp.ge(start) && entry.timestamp.le(end) && entry.artist.eq(&art.name) {
             artist_asp.1 += 1;
         }
     }
@@ -119,9 +117,11 @@ fn gather_album_date(
     let mut album_asp = AlbumPlays(alb.clone(), 0);
 
     for entry in entries {
-        let album = Album::new(entry.album.clone(), entry.artist.clone());
-
-        if entry.timestamp.ge(start) && entry.timestamp.le(end) && album == *alb {
+        if entry.timestamp.ge(start)
+            && entry.timestamp.le(end)
+            && entry.artist.eq(&alb.artist.name)
+            && entry.album.eq(&alb.name)
+        {
             album_asp.1 += 1;
         }
     }
@@ -141,13 +141,12 @@ fn gather_song_date(
     let mut song_asp = SongPlays(son.clone(), 0);
 
     for entry in entries {
-        let song = Song::new(
-            entry.track.clone(),
-            entry.album.clone(),
-            entry.artist.clone(),
-        );
-
-        if entry.timestamp.ge(start) && entry.timestamp.le(end) && song == *son {
+        if entry.timestamp.ge(start)
+            && entry.timestamp.le(end)
+            && entry.artist.eq(&son.album.artist.name)
+            && entry.album.eq(&son.album.name)
+            && entry.track.eq(&son.name)
+        {
             song_asp.1 += 1;
         }
     }