Merge #740: dev: bugfix completed download stat

439821c dev: bugfix completed download stat (Cameron Garnham)

Pull request description:

  A peer could repetitively  announce that it is completed, making the download stat higher. This changes the behavior to check if the event for the peer is not already completed.

ACKs for top commit:
  josecelano:
    ACK 439821c

Tree-SHA512: 40e194ca41d51d32aedc836125fe395eade96c34fbe80605c8b8a54e7c1f09e7052acb932b76f715422c70b08f5abe9b45ddbffff20d263a17ffe2908ee6e749

Author: josecelano

src/core/torrent/mod.rs

@@ -99,8 +99,8 @@ impl Entry {
             }
             AnnounceEvent::Completed => {
                 let peer_old = self.peers.insert(peer.peer_id, *peer);
-                // Don't count if peer was not previously known
-                if peer_old.is_some() {
+                // Don't count if peer was not previously known and not already completed.
+                if peer_old.is_some_and(|p| p.event != AnnounceEvent::Completed) {
                     self.completed += 1;
                     did_torrent_stats_change = true;
                 }