[XE] Bug in TDirectory.Copy, gibts schon einen QC eintrag?

[Faxe]

Aber die Hilfe redet von Pfaden:

Use Copy to copy a directory and its contents from a given path to another path.

SourceDirName
The path of the source directory that will be copied.

DestDirName
The destination path to which the directory will be copied.

[Sir Rufo]

Zitat von Faxe:

Aber die Hilfe redet von Pfaden:

Use Copy to copy a directory and its contents from a given path to another path.

SourceDirName
The path of the source directory that will be copied.

DestDirName
The destination path to which the directory will be copied.

Ja, auch bei Emba gibt es Menschen, die den Unterschied zwischen einem Verzeichnis und einem Pfad nicht kennen.
Noch schlimmer ist es allerdings, dass es auch Entwickler bei Emba gibt, die den nicht kennen und Verzeichnis schreiben und Pfad meinen, bzw. Pfad schreiben und Verzeichnis meinen.

[Uwe Raabe]

Also bei XE2-XE7 wird unterschieden, ob das letzte Zeichen ein Path-Separator ist oder nicht.

[Sir Rufo]

Zitat von Uwe Raabe:

Also bei XE2-XE7 wird unterschieden, ob das letzte Zeichen ein Path-Separator ist oder nicht.

Ähm, du meinst sicher TPath.DirectorySeparatorChar und nicht TPath.PathSeparator

[Uwe Raabe]

Zitat von Sir Rufo:

Zitat von Uwe Raabe:

Also bei XE2-XE7 wird unterschieden, ob das letzte Zeichen ein Path-Separator ist oder nicht.

Ähm, du meinst sicher TPath.DirectorySeparatorChar und nicht TPath.PathSeparator

Nein, ich habe ja auch gar nicht TPath.PathSeparator erwähnt.

Die Abfrage heißt

        if not TPath.IsPathSeparator(Ch) then

was zwar in der Nomenklatur obigen Schluss zulassen würde, aber ganz anders implementiert ist.

[Sir Rufo]

Zitat von Uwe Raabe:

Zitat von Sir Rufo:

Zitat von Uwe Raabe:

Also bei XE2-XE7 wird unterschieden, ob das letzte Zeichen ein Path-Separator ist oder nicht.

Ähm, du meinst sicher TPath.DirectorySeparatorChar und nicht TPath.PathSeparator

Nein, ich habe ja auch gar nicht TPath.PathSeparator erwähnt.

Die Abfrage heißt

        if not TPath.IsPathSeparator(Ch) then

was zwar in der Nomenklatur obigen Schluss zulassen würde, aber ganz anders implementiert ist.

Oh, da stimmt die Namensgebung auch nicht so richtig überein ...

[Uwe Raabe]

Zitat von Sir Rufo:

Oh, da stimmt die Namensgebung auch nicht so richtig überein ...

Nicht nur das - die Implementation ist auch buggy: Endet der Pfad mit einem Doppelpunkt, wird fälschlicherweise nämlich kein Backslash eingefügt. Aber wer kopiert auch schon ganze Laufwerke, nicht wahr?

[Memnarch]

Naja, alles was nicht mit einem Backslash endet scheint eher ein Dateiname zu sein(Soweit sehen es zumindest z.B. die Winapi funktionen).
Und auch ExtractRelativePath würde das letzte Element als Dateiname nehmen, wenn "\" fehlt.

Und zumindest hätte ich bei der Exception-Politik erwartet, dass es dann mit einer Exception in die binsen geht. Zumindest den Resultwert von TFile.Copy hätte man verwerten können. Es kann schließlich auch nicht sein, dass eine funktion ohne Probleme durchrennt und dann nicht funktioniert hat.