XP Usermanagers 6.1 Open Source

[Delphi-Laie]

Zitat von himitsu:

Mit Debuginfos kompilieren

Delphi 4 und Debuginformationen?

Bei den neueren Delphis (XEx, vielleicht auch schon eher) kann man ja zwischen Debug und Release auswählen.

Bei Delphi 4 macht es aber keinen Unterschied, ob ich in den Projektoptionen unter Debuggen alles ausschalte (ist bei VCL-Formularprogramm voreingestellt aktiviert) oder bei Luckies Usermanager diese Debugoptionen, die als Standard ausgeschaltet sind, einschalte: VCL hat bietet mir in beiden Fällen die Haltepunkte an, Non-VCL hingegen in keinem Fall.

Edit: Stimmt bei VCL nicht, die Haltepunkte verschwinden tatsächlich, aber erst nach dem Speichern der Optionen. Edit 2: Auch nicht in jedem Falle...
Edit 3: Es bringt leider auch nichts, in Luckies DPR-Datei die Compilerschalter {$D+} und/oder {$DEBUGINFO ON} ganz am Anfang einzufügen.

[jaenicke]

Zitat von Delphi-Laie:

Edit: Stimmt bei VCL nicht, die Haltepunkte verschwinden tatsächlich, aber erst nach dem Speichern der Optionen. Edit 2: Auch nicht in jedem Falle...

Diese Optionen standen bei Delphi 7 und früher in der .dof Datei:

Zitat:

DebugInfo=1

Wichtig: Nach der Änderung der Option das Projekt erzeugen, nicht nur normal kompilieren!

[Delphi-Laie]

Vielen Dank, Sebastian!

Ich hatte schon gehofft, mit diesem Deinem Hinweis den XP Usermanager endlich debuggen zu können, aber Delphi 4-7 verweigern sogar nach dem Setzen / Einschalten dieser Option in der dof-Datei hartnäckig das Setzen der Haltepunkte. Wird also immer noch nichts mit dem Debuggen.

Edit: Die von Dir genannte Option steht im Abschnitt "[Linker]". Also schaute ich in der IDE bei den Projektoptionen in die gleichnamige Rubrik, also unter "Linker". Da gibt es sogar eine Option "Mit externen Debug-Symbolen". Die war ausgeschaltet. Aber auch nach ihrem Einschalten gibt es keine Haltepunkte.

[nahpets]

Schau bitte mal alle eingebundenen Units durch, ob da irgendwo ein {$D-} zu finden ist, dann werden dort die Debuginfos wieder ausgeschaltet.

Ist ein leidiges Problem, das ich von Delphi 7 kenne.

[Delphi-Laie]

Zitat von nahpets:

Schau bitte mal alle eingebundenen Units durch, ob da irgendwo ein {$D-} zu finden ist, dann werden dort die Debuginfos wieder ausgeschaltet.

Ist ein leidiges Problem, das ich von Delphi 7 kenne.

Danke, Stephan!

Das war es zwar auch nicht, aber Du brachtest mich auf die richtige Fährte: Es liegt an der eingebundenen bzw. konkret aufgerufenen

{$I CompilerSwitches.inc}

Diese entfernt bzw. auskommentiert, und schon sind die Haltepunkte da!

[Delphi-Laie]

So, einen ersten kleinen Fehler habe ich entdeckt. Klein bezieht sich nicht auf seine Folge - immerhin beendet sich das Programm dann selbst bzw. wird von Windows beendet, allerdings ohne weiteren Schaden - sondern die Wahrscheinlichkeit seines Auftretens.

Wird dem Programm unter Extras -> Benutzer importieren eine ungültige XML-Datei angeboten, endet das Programm mit "Usermanager.exe hat ein Problem festgestellt und muss beendet werden." und danach mit "Exception Exception in Modul Usermanager.exe bei 0001BB90". Das passiert, wenn das Programm separat gestartet wird. Wird es aus der IDE heraus gestartet, kommt hingegen "Runtime Error 217 at 00011F07".

Der Fehler passiert in der Funktion "ImportXMLThread" und erstaunt mich insofern, als daß dort doch mit "try..finally" ihm entgegengewirkt wird.

Lange Rede, diesen try-Block mit einem weiteren try..except umhüllt, also

Delphi-Quellcode:

function ImportXMLThread(p: Pointer): Integer;
var
  ImPort            : TImport;
  Filename          : WideString;
begin

try
  Filename := PThreadParams(p)^.Filename;
  Import := TImport.Create(CurComputer, Filename);
  try
    if not Import.Import then
    begin
      SetCursor(LoadCursor(0, IDC_ARROW));
      case MessageBox(hApp, PChar(Format(rsImportError, [ExtractFilepath(ParamStr(0)) + LOGFILE])), PChar(APPNAME),
        MB_ICONWARNING or MB_YESNO) of
        IDYES: ShellExecute(hApp, 'open', PChar(ExtractFilepath(ParamStr(0)) + LOGFILE), nil, nil, SW_NORMAL)
      end;
    end;
  finally
    Import.Free;
  end;
except
//ggf. eine gewünschte Fehlerausgabe
end;

Result := 0;
end;

, und schon läuft auch das reibungslos.

[Aviator]

Du könntest ein Pull/Merge-Request auf GitHub erstellen damit Sebastian das einpflegen kann.

[jaenicke]

Zitat von Delphi-Laie:

Der Fehler passiert in der Funktion "ImportXMLThread" und erstaunt mich insofern, als daß dort doch mit "try..finally" ihm entgegengewirkt wird.

Oh, böse.
Die Methode Import liefern zwar einen Boolean zurück, schmeißt aber bei Misserfolg eine Exception.
Da das eine Threadmethode ist, landet die dann direkt in den aufrufenden Windows-Routinen.

Bei der Konstellation macht es gar keinen Sinn eine Exception zu werfen. Dazu kommt, dass Exception.Create in Unit Exceptions inherited nicht aufruft und somit die Message der Exception noch dazu leer ist.

try..finally sorgt ja nur dafür, dass Import.Free noch ausgeführt wird, behandelt aber den Fehler nicht.

Und CoDOMDocument40.Create ist auch etwas veraltet, das gibt es auch neueren Systemen vermutlich gar nicht mehr.

Ich schaue mal sobald ich dazu komme. Sollte schnell gehen.

[nahpets]

Zitat von Delphi-Laie:

{$I CompilerSwitches.inc} Diese entfernt bzw. auskommentiert, und schon sind die Haltepunkte da!

Würd' ich nicht so machen, da man dann nie weiß, ob durch andere Schalter nicht irgendwelche Nebeneffekte auftreten. Ändere lieber in der {$I CompilerSwitches.inc} den Debugschalter ab, dann gilt das solange für alle Stellen, an der die {$I CompilerSwitches.inc} eingebunden ist, bis Du die Änderung dort zentral wieder rückgängig machst.

Der Sinn der Includedatei ist es ja, die Änderungen zentral durchführen zu können und nicht nach dem Motto: Entweder alles oder nichts.

Und da die Includedatei in der ersten Zeile ein {.$define DEBUG} enthält, wäre es sinnvoller dort den . zu entfernen, da in der Datei über diesen Compilerschalter weitere Abhängigkeiten berücksichtigt werden.

Interessant wäre es zu prüfen, ob der von Dir entdeckte Fehler nach dem Aktivieren der Include-Datei weiterhin auftritt oder ob es sich hierbei um einen Nebeneffekt handelt, der durch die Änderung der Compilerschalter auftritt.

[Luckie]

Zitat von nahpets:

Und da die Includedatei in der ersten Zeile ein {.$define DEBUG} enthält, wäre es sinnvoller dort den . zu entfernen, da in der Datei über diesen Compilerschalter weitere Abhängigkeiten berücksichtigt werden.

Genau so habe ich es auch immer gemacht. Wenn ich dran gearbeitet habe, habe ich so das Debugging aktiviert und später wieder aktiviert, wenn ich die Version zum Veröffentlichen compiliert habe.