Show whole topic
Jan 29, 2014 5:13 pm
Jan 29, 2014 5:13 pm
Subject: Re: ResInstaller compilieren
Erstmal Danke für den Patch!
Habe die Änderungen mal durchgesehen und es sieht gut aus und kompiliert soweit! Testen konnte ich es zwar noch nicht, aber das sollte schon stimmen.
Nur zwei kleine Anmerkungen:
- Im Code bitte auch die Kommentare auf Englisch. Wir haben mittlerweile Entwickler, die nicht deutschsprachig sind.
- Achte darauf, die gleiche Formatierung wir im Rest des Codes zu verwenden. z.B hast du zum Einrücken zwei Leerzeichen verwendet, sonst wurden im Code aber überall Tabs verwendet oder sind im Rest des Codes z.B. auch die geschweiften Klammern in einer eigenen Zeile, du hattest das anders. Hier kann man sich natürlich ewig drüber streiten was besser und was schlechter ist, darum geht es aber egtl. gar nicht. Nur eines ist wichtig: einheitlich muss es sein! Irgendwo im Forum gibt es glaube ich einen Thread wo ein paar Regeln definiert wurden. Ob die noch eingehalten werden, wage ich aber zu bezweifeln. Am besten einfach schauen, wie der Code so formatiert ist und einfach genau so machen
Auf jeden Fall habe ich die paar Anmerkungen behoben und das Ganze eingecheckt!
Über weitere Patches würden wir uns natürlich freuen
Dann noch zum kompilieren von maxr:
Das sollte definitiv auch (noch) mit VS2010 und VS2012 gehen. Wenn Fehler kommen, poste sie bitte hier. Ich kann das leider nur nicht mehr testen, weil ich nur noch VS2013 installiert habe. Trotzdem kann ich dir nur empfehlen auf die neuen Versionen zu upgraden, wenn du die Möglichkeit hast (z.B. bei der Express Edition). Bedenke, dass du dann aber ggf. etwaige Bibliotheken, die du selbst gebaut hast, neu bauen musst.
Erstmal Danke für den Patch!
Habe die Änderungen mal durchgesehen und es sieht gut aus und kompiliert soweit! Testen konnte ich es zwar noch nicht, aber das sollte schon stimmen.
Nur zwei kleine Anmerkungen:
- Im Code bitte auch die Kommentare auf Englisch. Wir haben mittlerweile Entwickler, die nicht deutschsprachig sind.
- Achte darauf, die gleiche Formatierung wir im Rest des Codes zu verwenden. z.B hast du zum Einrücken zwei Leerzeichen verwendet, sonst wurden im Code aber überall Tabs verwendet oder sind im Rest des Codes z.B. auch die geschweiften Klammern in einer eigenen Zeile, du hattest das anders. Hier kann man sich natürlich ewig drüber streiten was besser und was schlechter ist, darum geht es aber egtl. gar nicht. Nur eines ist wichtig: einheitlich muss es sein! Irgendwo im Forum gibt es glaube ich einen Thread wo ein paar Regeln definiert wurden. Ob die noch eingehalten werden, wage ich aber zu bezweifeln. Am besten einfach schauen, wie der Code so formatiert ist und einfach genau so machen
Auf jeden Fall habe ich die paar Anmerkungen behoben und das Ganze eingecheckt!
Über weitere Patches würden wir uns natürlich freuen
Dann noch zum kompilieren von maxr:
Das sollte definitiv auch (noch) mit VS2010 und VS2012 gehen. Wenn Fehler kommen, poste sie bitte hier. Ich kann das leider nur nicht mehr testen, weil ich nur noch VS2013 installiert habe. Trotzdem kann ich dir nur empfehlen auf die neuen Versionen zu upgraden, wenn du die Möglichkeit hast (z.B. bei der Express Edition). Bedenke, dass du dann aber ggf. etwaige Bibliotheken, die du selbst gebaut hast, neu bauen musst.
Albert Ziegenhagel