Zitat von
radekj:
Das OpenSource Projekt hat an der DeCAL nichts gemacht.
Das ist ja das Problem. 6 Jahre in dieser Branche sind eine halbe Ewigkeit. Und wenn ich so über den Code blättere fallen mir da schon ein paar Stellen auf, die man lieber anders lösen sollte (abgesehen vom in Pascal/Delphi unüblichen lowerCamelCase). Schon alleine die vielen globalen Funktionen (getString(), getCurrency(), ...) würden mich abschrecken. Die würden besser in einen Namespace gehören und nicht wie wild im
Unit Namespace herumliegen. Bei getString und Konsorten wäre das einbauen in DInterator sogar besser. Das UniqueString() im getString() bremst auch nur aus. Wenn der Aufrufer den String in einen PChar umwandelt möchte, sollte er sich selbst darum kümmern, dass der String unique ist was er sonst ja auch muss. Das einfach mal prinzipiell durchzuführen ist nichts anderes als eine unnötige Geschwindigkeitsbremse.
Und das nur beim schnellen drüberschauen.