On 17/06/16 14:57, Pavel Rappo wrote:
Roger,
On 17 Jun 2016, at 14:14, Roger Riggs <roger.ri...@oracle.com> wrote:
Subclassing is the normal (one) way to override/augment behavior of an existing
class.
And I was looking for a minimal change since there have been many issues in
this area.
Also, I avoided the extra object and performance overhead of delegation by
extending FIS.
Looks good.
Though the code reuse in this case would've been the most of my concerns (after
the bug fix of course). I wonder how much is an extra cost per method dispatch?
As for the RFR, could you please add `@Override` annotation on top of your skip?
Thanks,
-Pavel
Looks good to me too.
best regards,
-- daniel