A lot of the helper functions throw checked exceptions (mostly IllegalAccessException, InvocationTargetException and NoSuchMethodException). I'd like to write a patch which converts them all into RuntimeExceptions so the code upstream doesn't have to handle them.
Rationale: When working with properties, you usually expect the code to work. When working with dynamic properties, you don't care about what went wrong but only that something went wrong. In most cases, it's not possible to fix the problem automatically but the code has to pass the exception up.
This all calls for non-checked exceptions.
My patch would be against the SVN version and contain these changes:
- The old methods would still be there with an E at the end (for "throws a checked exception").
- The new code would throw exceptions based on a common exception so you can catch all the exceptions thrown in beanutils in one statement. IAE, ITE, etc. would then be derived from this base class so you can still filter them.
Since this is going to involve some work, I wanted to ask beforehand if you would accept such a patch.