Documentfoto's blijven allemaal op volle decode in het geheugen #2727

New issue

Open

opened 2026-11-02 22:20:41 +00:00 by brenno · 0 comments

brenno commented

2026-11-02 22:20:41 +00:00

Owner

Wat er mis is

Een rapport tekent het hele document als één kolom. Elke afbeelding decodeert tot de bomgrens van 4096 px, niet tot de kolom. Tien foto’s van 4000×3000 zijn ongeveer 460 MiB. Twintig foto’s op de grens zijn ongeveer 1,3 GiB. Die bitmaps blijven leven zolang het document open is. Het commentaar bij de provider zegt dat twintig foto’s het geheugen niet opeten. Dat klopt per bestand (64 MiB), niet voor de som.

Bewijs

lib/widgets/reader/parts/document_markdown_image.dart: cappedFileImage zonder kolomrand, met het commentaar over twintig foto’s.
lib/widgets/reader/document_markdown_view.dart bouwt elk blok in een kolom, dus de providers blijven in de boom.
Dia’s gebruiken decodeEdgeForBox. Dit pad niet. docs/PERFORMANCE_GUIDE.md noemt 4096² × 4 als ongeveer 64 MiB per bestand, niet als plafond voor alles wat tegelijk gedecodeerd in beeld staat.

Voorstel

Dezelfde tredes als de dia’s (1024 / 2048 / 4096) op de kolombreedte. 4096 blijft de bomgrens. Het commentaar corrigeren zodat het de som niet wegcijfert.

Geen nieuwe afhankelijkheid. Het bestandsformaat verandert niet.

## Wat er mis is Een rapport tekent het hele document als één kolom. Elke afbeelding decodeert tot de bomgrens van 4096 px, niet tot de kolom. Tien foto’s van 4000×3000 zijn ongeveer 460 MiB. Twintig foto’s op de grens zijn ongeveer 1,3 GiB. Die bitmaps blijven leven zolang het document open is. Het commentaar bij de provider zegt dat twintig foto’s het geheugen niet opeten. Dat klopt per bestand (64 MiB), niet voor de som. ## Bewijs - `lib/widgets/reader/parts/document_markdown_image.dart`: `cappedFileImage` zonder kolomrand, met het commentaar over twintig foto’s. - `lib/widgets/reader/document_markdown_view.dart` bouwt elk blok in een kolom, dus de providers blijven in de boom. - Dia’s gebruiken `decodeEdgeForBox`. Dit pad niet. `docs/PERFORMANCE_GUIDE.md` noemt 4096² × 4 als ongeveer 64 MiB per bestand, niet als plafond voor alles wat tegelijk gedecodeerd in beeld staat. ## Voorstel Dezelfde tredes als de dia’s (1024 / 2048 / 4096) op de kolombreedte. 4096 blijft de bomgrens. Het commentaar corrigeren zodat het de som niet wegcijfert. Geen nieuwe afhankelijkheid. Het bestandsformaat verandert niet.

brenno added the

bug

label

2026-11-02 22:20:41 +00:00

brenno added the

in-progress

label

2026-11-03 01:03:57 +00:00