#437: Polishing some logics and comments

[kazuki43zoo]

• modify to use Cursor.class.isAssignableFrom(Class<?>)
• modfiy to use ExceptionFactory.wrapException(String,Exception)
• modify copyright year

Please review #437 polishing.

[gdarmont]

@kazuki43zoo
Cursor.class.isAssignableFrom(Class<?>) is broken.
If we have a "SubCursor" interface that extends "Cursor", Cursor.class.isAssignableFrom(SubCursor.class) returns true. But the implementation returned will always be a DefaultCursor, which is not a SubCursor. This will trigger ClassCastException.

As a side note, adding yourself to the authors list for only one line changed (MapperMethod) seems rather weird to me. But I leave this considerations to the projects maintainers.

[emacarron]

Hi guys,

@gdarmont being strict it is OK that you add your name to the author list for any trivial change you make given that you are indeed the author of the change. It is up to the contributor to add himself though we do not see often that behaviour.

I will revert the change until the isAssignableFor issue is clarified.

Anyway please, remember this is the right way. Any contribution is welcomed and discussion is also welcomed so thanks both for your work!