[Java] Korrekturen im disconnected und Inline-Doku

[remotecontrol]

Hallo Admins,

 

ich habe kleine Korrekturen in der IPConnection und der Inline-Doku des BrickServo gemacht (siehe Anhang). In der IPConnection sind einige Compiler-Warning entfernt, wichtig ist im disconnect aber sowas:

 

anstelle von

try {
    if(socket != null) {
        socket.close();
        socket = null;
    }
} catch(java.io.IOException e) {
    e.printStackTrace();
}

 

nutze ich bei mir

try {
    if(socket != null) {
        socket.close();
    }
} catch(java.io.IOException e) {
    e.printStackTrace();
}
socket = null;

 

Denn der socket.close() kann fehl schlagen, wenn das Netzwerk zusammenbricht. Dann wird der socket nie wieder auf null gesetzt. Analog für die Streams.

 

Ich weiss nicht, wie Ihr mit Vorschlägen für Code-Anpassungen umgeht, darum mal als Attachment.

IPConnection.java

BrickServo.java

[AuronX]

Üblicherweise würde ich einen Pull Request auf Github vorschlagen. Aber wenn Git und Github bisher neu für dich sind wird sich TF sicherlich auch hier über deine Änderungen freuen.

 

LG

Jan

[photron]

Okay, du hast neben den korrigierten null Zuweisungen noch redundante Casts entfernt und die JavaDoc Links verkürzt.

 

Da BrickServo.java generiert wird gehören die JavaDoc Änderungen in generate_java_bindings.py.

 

Wie AuronX richtig sagt ist der einfachste Weg für uns über einen Pull Request auf GitHub. Das setzt allerdings voraus, dass du dich mit git auskennst und vor allem auch weist wie die Generatoren arbeiten usw. Daher ist es auch okay, wenn du Änderungs- und Verbesserungsvorschläge einfach hier im Forum postest.

 

Ich baue deine Änderungen gleich ein.