Feature suggestion to PointUnreachableException

[positron96]

Hey there!
I'd like to make a feature suggestion: in 0.9.1 class PointUnreachableException has only 1 constructor without parameters. However it would be nice to have a constructor with string argument, like most other Exceptions do. This is probably a bug since the code of this class says:

Code: Select all

public class PointUnreachableException extends Exception {
   // Inherits everything from Exception
}


However, constructors are not inherited in java this way, they should be explicitly defined and use super(...) calls.

[positron96]

I can even suggest an improved version. Don't know how to attach files here, so:

Code: Select all

package lejos.robotics.kinematics;

public class PointUnreachableException extends Exception {
   // Inherits everything from Exception

   public PointUnreachableException() {
      super();
   }
   
   public PointUnreachableException(String message) {
      super(message);
   }
   
   public PointUnreachableException(String message, Throwable cause) {
      super(message, cause);
   }
   
   public PointUnreachableException(Throwable cause) {
      super(cause);
   }
}


[skoehler]

Probably, somebody though constructors were inherited as well, which they aren't. However, it's fixed:
http://lejos.svn.sourceforge.net/viewvc ... threv=6732

[positron96]

This is great! Thanks for the reply!