Aktuell gibt es viele Warnungen im RCP Code.
Hier wird versucht, so viele wie möglich davon zu entfernen.
Viele der Warnungen sind in diesem Stil:
Dafür kann es anscheinend mehrere Ursachen geben:
Verschiedene Versionen im Build-Path -> Alle Versionen in MANIFEST.MF Dateien wurden entfernt
"Falsche" Reihenfolge der Abhängigkeiten. Im RCP wird z. B. Inject oft markiert, weil javax.inject im aero.minova.rcp.dataservice Projekt importiert wurde. Das dataservice Projekt wiederum wird in vielen Unterprojekten verwendet, und stand in den MANIFEST.MF Dateien oft über dem javax.inject Import -> Die Warnung im Screenshot lässt sich oft schon dadurch entfernen, den entsprechenden Import in der zugehörigen MANIFEST.MF Datei ganz nach oben zu schieben
Um die Reihenfolge besser steuern zu können wurden soweit möglich "Require-Bundle" statt "Import-Package" genutzt
Der Großteil der verbleibenden Warnungen kommt vom Logger:
Hierfür konnte ich noch keine gute Lösung finden (außer die access rules anzupassen, was Anpassungen am .classpath bewirkt. Diese Dateien sollten wir eigentlich nicht einchecken). Ich habe Lars Vogel eine Mail geschrieben, wie mit dieser Warnung korrekt umgegangen werden soll.
Update: Laut Lars wurde das Logger-Bundle nie freigegeben. Stattdessen soll org.eclipse.core.runtime.ILog genutzt werden.
ILog logger = Platform.getLog(this.getClass());
Außerdem wurden viele allgemeine Warnungen und Code Smells entfernt.
Ziel: Code ohne Warnungen
Aktuell gibt es viele Warnungen im RCP Code. Hier wird versucht, so viele wie möglich davon zu entfernen.
Viele der Warnungen sind in diesem Stil:
Dafür kann es anscheinend mehrere Ursachen geben:
Injectoft markiert, weiljavax.injectimaero.minova.rcp.dataserviceProjekt importiert wurde. DasdataserviceProjekt wiederum wird in vielen Unterprojekten verwendet, und stand in denMANIFEST.MFDateien oft über demjavax.injectImport -> Die Warnung im Screenshot lässt sich oft schon dadurch entfernen, den entsprechenden Import in der zugehörigenMANIFEST.MFDatei ganz nach oben zu schiebenDer Großteil der verbleibenden Warnungen kommt vom
Logger:Hierfür konnte ich noch keine gute Lösung finden (außer die
access rulesanzupassen, was Anpassungen am.classpathbewirkt. Diese Dateien sollten wir eigentlich nicht einchecken). Ich habe Lars Vogel eine Mail geschrieben, wie mit dieser Warnung korrekt umgegangen werden soll.Update: Laut Lars wurde das Logger-Bundle nie freigegeben. Stattdessen soll
org.eclipse.core.runtime.ILoggenutzt werden.Außerdem wurden viele allgemeine Warnungen und Code Smells entfernt.