spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

In der Ref ist doch alles drin. Man sieht alle geänderten Files und geänderten Lines im SVN-Log.
Das händisch rauszusuchen würde länger dauern als den Code zu schreiben...

Du hast aktuell 4 Storry zum checken und bist in den letzten 35 ref 13 mal mit von der Party und der Ref.-Text bezieht sich nicht speziel auf eine geänderte Datei.
Also mir würde es Helfen, meinet wegen nur dir Ref nummer dann such ich es mir raus, ich fände es schöner die Files zu wissen weil ich mir die in nicht unbedingt im SVN-Log anschau (testen ect. geht halt besser im VC).

Es ist wirklich nicht so schwer, siehe angehängte Bilder.
Statt einer einzelnen Revision kann man auch mehrere markieren.

Danke für die mühe, ich weiß wie es geht.

Ich muss übrigens mal ein Lob los lassen oder ein paar mehr

Ich bin aktuell sehr zufrieden mit dem Fortschritt und auch mit der Toleranz, die die Entwickler manchmal für meine Wünsche aufbringen müssen

Es gibt hier und da mal einen Schnitzer, den die Team-Mitglieder gegenseitig glatt bügeln (was übrigens auch bei erfahrenen Berufs-Entwicklern noch vorkommt), aber es gibt auch ein paar sehr schöne Lichtblicke und Ideen bei den Umsetzungen. Das gilt beides übrigens für jeden.

Klingt doch super.

Zwei Kleinigkeiten, die mir in HUDElement.hpp aufgefallen sind:

• Virtueller Destruktor fehlt
  
• Inkonsistentes Interface: position im ctor als const ref, aber beim setter per value
  

Sind solche Sachen Bestandteil des Reviews oder legt ihr da nicht so viel Wert drauf? - Ersteres könnte schon zu Problemen führen, zweiteres ist einfach unschön. Rein für den Lerneffekt sollte das imo schon auch korrigiert werden.

Doch, das sowas ist eigentlich Teil des Reviews, aber ich war nach viel Hin und Her an diesem Ding scheinbar etwas schlampig beim Review

Sorry, Jungs, dass es mit den Abnahmen im Moment gerade schleift, bei mir ist die private Zeit ziemlich knapp im Moment. Bei den letzten überfliegenden Blicken auf die Commits, habe ich aber den ein oder anderen kleinen Bug gesehen und würde mir daher für die Reviews die nötige Zeit nehmen. Da sind scheinbar ein paar Dinge unentdeckt geblieben, die wir uns aber nicht als technische Schulden aufhalsen sollten, nur damit etwas abgenommen ist. Ich bitte daher um Verständnis

Jo, ist und bleibt ein Hobbyprojekt!

Also lass dir Zeit