Changes

Use newer minimal java source and target version
To be able to compile the project with java versions greater than 1.8 a
minimal java version of at least 1.8 is needed. Change the minimal java
source and target version and use a minimal java compiler version 1.8 to
deal with that.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Change junit output format to xml
As preparatory patch to use Jenkins junit bindings the output format of
junit needs to be changed to XML.
Change the output format to XML.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Remove unused imports
Some imports are not used any more. Remove them.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Remove unused variables
The connection, domain and network pointer in the error class are 
leftover from previous implementation. These pointers are not only 
unused, they furthermore inhibit proper serialization. Drop these
pointers.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Declare inner class static
For a better encapsulation and stronger access rules define inner 
classes as static.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Specify string encoding
String manipulation with unknown encoding may result in undefined 
behaviour. Set the string encoding to UTF-8 and add error handling.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Mark methods final
A couple of properties are not changed after initialization. Mark them
final.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Change field name according to Java naming convention
The Java naming convention claims: Constants should be written in
uppercase characters separated by underscores. Constant names may also
contain digits if appropriate, but not as the first character.
Convert field names to uppercase according to the naming convention.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Add default handling to switch and revisit error handling
The Java naming convention claims: Every switch statement should include
a default case.
Revisit all switch statements and add a default case if needed. While at
it check the error handling.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Revisit C-string conversion
The former C-string conversion returned a null pointer without need. 
Avoid the null pointer and return a feasible string instead.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Add missing documentation
Revisit JavaDoc warnings and add necessary documentation.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Whitespace cleanup
Checkstyle grouse  about whitespace damage. Revisit the code and do a
cleanup. No functional change.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Javadoc: Escape tag in documentation
Description of XML tags in Javadoc need to be escaped. Add missing 
escape sequence.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Mark methods and parameter final
To inhibit unnoticed modifications these methods and parameter can be 
marked immutable with the modifiere final. Add modifier final, where
appropriate.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Trim linelength
Trim overlengthy source code lines for better readability. While at it
convert some inline comments to JavaDoc.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Add missing braces
If statements require braces according to the Java styleguide. Add the
missing braces.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Use Java style brackets
Instead of using C style brackets convert to Java style.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Remove redundant modifier
Interfaces are abstract by definition so the abstract modifier on the 
interface is redundant. Classes inside of interfaces are public and
static by definition, so the public and static modifiers on the inner
classes are redundant. Fields in interfaces and annotations are
automatically public, static and final, so these modifiers are redundant
as well.
Remove redundant modifier and use a common modifier order.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Reformat constructor
Reformat a constructor according to the Java styleguide.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Rename according to Java style
The Java style guide define some naming conventions. Rename methods, 
parameter, fields etc. where appropriate.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Avoid magic constant values
Add named constants for better readability of the code.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Rework enum switch case inquiry
Enum entries may contain a defined value. Rework the lengthy enum switch 
case inquiry by using the defined values.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Add bitflag helper
Instead of using the (1 << x) notation causing magic constant warnings 
introduce a bitflag helper function.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Remove useless elvis operator
There is no need for an elvis operator returning true and false for 
boolean values. Remove the useless elvis operator.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>

Remove useless parentheses
Useless parentheses bedevil the source. Remove them.
Signed-off-by: Benedikt Spranger <b.spranger@linutronix.de> Reviewed-by:
Bastian Germann <bage@linutronix.de>