AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren
Zurück Delphi-PRAXiS Programmierung allgemein Algorithmen, Datenstrukturen und Klassendesign [Firemonkey, Android] TImageList Weak Bitmap im Cache
Thema durchsuchen
Ansicht
Themen-Optionen

[Firemonkey, Android] TImageList Weak Bitmap im Cache

Ein Thema von Rollo62 · begonnen am 1. Mai 2017 · letzter Beitrag vom 1. Mai 2017
Antwort Antwort
Rollo62

Registriert seit: 15. Mär 2007
3.914 Beiträge
 
Delphi 12 Athens
 
#1

[Firemonkey, Android] TImageList Weak Bitmap im Cache

  Alt 1. Mai 2017, 08:52
Hallo zusammen,

wie funktioniert der Weak-Mechanismus intern ?
Ich habe mir das nicht genau angesehen, ich vermute aber das hierbei einfach keine Referenzen gezählt werden sollten.
Leider habe ich Probleme mit 10.2 und ImageList und ListBox Glyphs.

Im Code ImageList steht Folgendes:
Delphi-Quellcode:
  TCache = class
  private type
    TBitmapCacheItem = record
      Size: TSize;
      Index: Integer;
      [Weak] Bitmap: TBitmap; // Das Bitmap sollte WEAK gespeichert sein
    end;



procedure TCache.Add(const Size: TSize; const Index: Integer; const Bitmap: TBitmap);
var
  NewItem: TBitmapCacheItem;
begin
  if (FCacheSize > 0) and (Bitmap <> nil) then
  begin
    if FList <> nil then
      while FCacheSize <= FList.Count do
      begin
        FList[0].Bitmap.DisposeOf;
        FList.Delete(0);
      end
    else
      FList := TList<TBitmapCacheItem>.Create;
    NewItem.Size := Size;
    NewItem.Index := Index;
    NewItem.Bitmap := Bitmap;
    {$IFDEF AUTOREFCOUNT}
    NewItem.Bitmap.__ObjAddRef; // Ist das korrekt ? Das wird zumindest bei Android aufgerufen.
    {$ENDIF}
    FList.Add(NewItem);
  end;
end;
Ist das so korrekt, oder sollte das RefCount nicht eigentlich ignoriert werden, oder zumindest mit einen weiteren Define WEAKREFCOUNT oder so abgesichert sein ?

Rollo

Geändert von Rollo62 ( 1. Mai 2017 um 10:08 Uhr)
  Mit Zitat antworten Zitat
Benutzerbild von himitsu
himitsu

Registriert seit: 11. Okt 2003
Ort: Elbflorenz
43.164 Beiträge
 
Delphi 12 Athens
 
#2

AW: [Firemonkey, Android] TImageList Weak Bitmap im Cache

  Alt 1. Mai 2017, 14:42
Delphi-Quellcode:
{$IFDEF AUTOREFCOUNT}
NewItem.Bitmap.__ObjAddRef;
{$ENDIF}
Damit erzeugst du eine virtuelle Variable und umgehst das Weak doch wieder, da RefCount nie 0 wird.

[Weak] in iOS/Android (sonst gibt es ARC nirgendwo) bedeutet nur, dass diese Variable erstmal nicht gezählt wird
und wenn die verlinkte Instanz freigegeben wird (RefCount=0), dann wird diese Variable auf nil gesetzt. (dafür gibt es eine globale Liste aller Weak-Variablen)

Also, das Bitmap ist irgendwo noch mindestens in einer anderen Variable und wenn diese anderen Variablen alle geleert sind, dann wird auch deine Weak-Variable auf nil gesetzt.

PS: mit NewItem.Bitmap.FreeNotification(Self); würde auch noch ein TCache.Notification aufgerufen und du könntest auch das ganze Cache-Item gleich mit entsorgen.
Garbage Collector ... Delphianer erzeugen keinen Müll, also brauchen sie auch keinen Müllsucher.
my Delphi wish list : BugReports/FeatureRequests

Geändert von himitsu ( 1. Mai 2017 um 14:48 Uhr)
  Mit Zitat antworten Zitat
Rollo62

Registriert seit: 15. Mär 2007
3.914 Beiträge
 
Delphi 12 Athens
 
#3

AW: [Firemonkey, Android] TImageList Weak Bitmap im Cache

  Alt 1. Mai 2017, 18:43
Hallo Himitzu,

Ok, dann sehe ich das Richtig das dieser Code ein Bug in dem Android TImageList Cache ist.
Das könnte die "Geisterbilder" erklärem die ich seit Rx10.2 in den Apps sehe wenn es auf ListBox-Glyphs per ImageList verlinkt ist, aber auch TGlyph macht Probleme.
Wie gesagt, nur Android, iOS ist noch Alles OK.

Ich versuche mal rauszufinden was da in Rx10.2 geändert worden ist, ich hoffe es hält sich in Grenzen.

Rollo
  Mit Zitat antworten Zitat
Antwort Antwort


Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 10:16 Uhr.
Powered by vBulletin® Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024 by Thomas Breitkreuz