Bug preloaded filesize overflow

[mmuellr]

Hallo, mir ist schon vor längerer Zeit ein kleiner Bug aufgefallen. Die Darstellung der Preloaded Filesize (unter Einstellungen -> Miscellaneous) läuft bei 2GB über und zeigt dann einen negativen Wert an. Wenn man bei der Berechnung einfach den Cast zu Int weglässt würde die Filesize in Long berechnet werden. Damit könnten dann so ca. eine Größe von 9223372036854775mb (9,2 Exabyte) dargestellt werden. Sollte vorerst reichen.

Subject: [PATCH] Remove the cast to int when calculating file sizes to prevent integer overflow for preloaded files in sum larger than 2GB
---
Index: app/src/main/java/com/pr0gramm/app/ui/SettingsFragment.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/app/src/main/java/com/pr0gramm/app/ui/SettingsFragment.kt b/app/src/main/java/com/pr0gramm/app/ui/SettingsFragment.kt
--- a/app/src/main/java/com/pr0gramm/app/ui/SettingsFragment.kt (revision 6e974f9a1ea188e9fcdd2ebde8faf976311772f4)
+++ b/app/src/main/java/com/pr0gramm/app/ui/SettingsFragment.kt (date 1707683144431)
@@ -98,9 +98,9 @@

                 val totalSize = runInterruptible(Dispatchers.IO) {
                     items.values().sumOf { item ->
-                        item.media.length().toInt() +
-                                item.thumbnail.length().toInt() +
-                                (item.thumbnailFull?.length()?.toInt() ?: 0)
+                        item.media.length() +
+                                item.thumbnail.length() +
+                                (item.thumbnailFull?.length() ?: 0)

                     }
                 }

[mopsalarm]

Ha, blöd von mir. Danke, ja, fix ich morgen. Warum hast du denn keinen PR aufgemacht?

[mmuellr]

War zu blöd. Aber PR ist nun da

[mopsalarm]

Ist gemerged, sorry, dass es so lange gedauert hat.