Easy3DGL - Eine Bibliothek für OpenGL

[Edlmann]

Zitat von Satty67:

Zitat von Edlmann:

g) Den Fehler finde ich einfach nicht.

Kann sein dass ich mich falsch erinnere...

Aber schau Dir mal die Schleife an... meine da war Schleifenvariable x und Zugriff über y (oder umgekehrt)

Ne, Variable ist beidesmal x...

[Satty67]

Sehe ich anders...

Delphi-Quellcode:

// FreeGL
  for y := 0 to TransparentList.Count - 1 do
  begin     
    if Assigned(SolidList[x]) then
    begin
    CItem := TransparentList[x];
    CItem.Free;
    end;
  end;

Das auf SolidList geprüft wird statt TransparentList[], war zudem wohl der C&P Fehler

[Edlmann]

Mmh...bei mir steht da:

Delphi-Quellcode:

  try
  for x := SolidList.Count - 1 downto 0 do
  begin
    if Assigned(SolidList[x]) then
    begin
    CItem := SolidList[x];
    SolidList.Remove(CItem);
    CItem.Destroy;
    end;
  end;
  for x := TransparentList.Count - 1 downto 0 do
  begin
    if Assigned(TransparentList[x]) then
    begin
    CItem := TransparentList[x];
    TransparentList.Remove(CItem);
    CItem.Destroy;
    end;
  end;
  except

  end;

kann sein dass ich das schon gefixxt habe, und nur noch nicht hochgeladen, schau ich gleich mal

[Satty67]

Also ist bei mir natürlich noch die ungefixte Version, aber dachte darauf bezog sich die Liste von rollstuhlfahrer. Wenn es gefixt ist, ist ja gut.

[Edlmann]

Sagen wir die CopyPaste Fehler sind gefixxt... die AV kommt noch immer... ^^

[mleyen]

Hab mir die neue Version noch nicht angeschaut, aber statt .Destroy sollte man lieber FreeAndNil() verwenden.
Und ein private-Sichtbarkeitsabschnitt wäre für Sachen wie FreeGL von Vorteil.

[rollstuhlfahrer]

Ich hab die AV gefunden: In der TransparentList ist an Position 1 (bei mir) ein schon aus dem Speicher genommenes Objekt, welches fälschlicherweise von Assigned als existent erachtet wird. Jetzt nur noch schauen, wo das Objekt gelöscht wird.

Bernhard

ADD: AV entgültig gefunden: Bei mir wurde ein GLLicht erstellt. Dieses wurde zu der genannten Liste hinzugefügt. Leider wurde dieses Licht vorher bei den Lichtern aus dem Speicher genommen und ist somit nicht mehr da. Dann soll es allerdings wieder aus dem Speicher genommen werden und dann knallt der nächste Zugriff (das war bei mir die Abfrage nach dem Klassenname).

ADD2: Was mir noch auffällt: Meine Objekte sind alle DOPPELT in der Liste, sie werden also auch 2x gezeichnet. Was machen die da doppelt drin?
ADD3: Warum die alle doppelt drin sind? - Das liegt an den Konstuktoren: In GLQuader (Beispiel) wird das Material gesetzt. Dann wird über inherited der Konstruktor von GLObject aufgerufen und dort gleich nochmal das Material gesetzt. -> macht 2 Listeneinträge, von denen einer zu viel ist.

ADD4: Ich habe mal eine geänderte Version hochgeladen (für Delphi ohne Umlaute). Alles, was ich geändert habe, habe ich mit "Änderung" versehen. Teilweise stehen auch die Gründe direkt dabei.