fix FB warning: Write to static field from instance method

Write to static field aQute.jpm.service.ServiceMain.last
from instance method aQute.jpm.service.ServiceMain.setMessage(String)

This instance method writes to a static field. This is tricky
to get correct if multiple instances are being manipulated, and
generally bad practice.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
bnd: make out and err final

They were never assigned (except in the unused method setOut,
which is removed here)

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
jpm: Main: make out and err final

They were never assigned elsewhere

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix FB warning: Format string should use %n

Format string should use %n rather than \n
in aQute.libg.asn1.BER.dump(PrintStream, int, long, String)

This format string include a newline character (\n).
In format strings, it is generally preferable better to use %n,
which will produce the platform-specific line separator.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix FB warning: inefficient use of keySet iterator

aQute.lib.getopt.CommandLine.getOptions(Class, List) makes
inefficient use of keySet iterator instead of entrySet iterator

This method accesses the value of a Map entry, using a key that
was retrieved from a keySet iterator. It is more efficient to use
an iterator on the entrySet of the map, to avoid the Map.get(key)
lookup.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Added some rationale why this class is useful since
Ferry wanted to know why I was not just using sorted
maps.
Issue #175, complaint that the range macro did not 
properly work since it was expanded too early, before
the implicit @ version was set. The range macro now
fails if there is no implicit version nor an explicit
this will make the macro return itself.
fix FB warning: Unread field

This field is never read.  Consider removing it from the class.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix an eclipse warning on fall-through

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: Unnecessary semicolon

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: Unnecessary cast

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: Unnecessary @SuppressWarnings("unchecked")

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: parameter should not be assigned

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: Unnecessary semicolon

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
suppress some eclipse cast warnings

these cast are really needed

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix an eclipse warning on fall-through

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: Unnecessary cast

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
fix eclipse warnings: Statement unnecessarily nested within else clause

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Changed version command to show the Analyzer.getBndVersion method for testing
Removed test because it accessed a remote server