af0815 hat geschrieben:GITHUB: Daten sind in WP_XYZ_Version eingepflegt.
Damit wir uns richtig verstehen: Meine Änderungen haben auf wip-samples aufgebaut, wo alle Versionen vereinigt sind. Daher sollten sie eigentlich nach wip-samples.
af0815 hat geschrieben:GITHUB: Daten sind in WP_XYZ_Version eingepflegt.
Mitja, in "TImageInfo.Readcomments", im else-Zweig von "if pos('ASCII', p^.Row) = 1", [...]
af0815 hat geschrieben:a) Richtiger Lazarus Fork, dann aber mit anderen Namen Bsp. -> LaxEXIF
af0815 hat geschrieben:b) Wenn nicht, wer kontrolliert gegen Delphi und vor allen gegen welche Versionen ?
af0815 hat geschrieben:c) Zu welcher Lazarus Version soll das das kompatibel bleiben ?
wp_xyz hat geschrieben:Folgende Wünsche von mir:
Wenn der Code einmal halbwegs stabil ist, müsste sich einer die Mühe machen, das Durcheinander von Methoden, die von der Logik her private oder protected sein sollten, im public-Abschnitt aufzuräumen und richtige Properties anstatt der eingefügten öffentlichen Objekt-Felder einzuführen (ich habe in der heute verschickten Version mit Properties Height und Width etc angefangen - man hat dadurch einfach viel mehr Möglichkeiten). Außerdem würde ich mir einen weitgehend einheitlichen Codierstil wünschen, Einrückungen kreuz und quer und unsystematische Groß/Kleinschreibung machen einem die Arbeit in dem nicht ganz einfachen Code unnötig schwer.
wp_xyz hat geschrieben:Nein, einen GitHub-Account habe ich nicht. Mal sehen, muss mich erst einarbeiten... Aber ohne, wird es problematisch, ich habe eben gesehen, dass von meinen geschickten Dateien so gut wie nichts im Master angekommen ist...
af0815 hat geschrieben:Test mit Lazarus 1.4.4 r55637 FPC 2.6.4 i386-win32-win32/win64 gehen nicht. Das lässt sich schon mal nicht kompilieren. TFPReaderJpeg.ImageSize gabs damals noch nicht.