Hi Joel,
Revised ccc request finalized for current version of the spec; should be
approved in a day or two.
Once that is approved, I think the current version can be pushed,
enabling the refactorings you've alluded to to occur later.
Thanks,
-Joe
On 12/13/2015 12:26 PM, Joel Borggrén-Franck wrote:
Hi Joe,
Thanks for the comments,
On Thu, 10 Dec 2015 at 22:18 joe darcy <joe.da...@oracle.com
<mailto:joe.da...@oracle.com>> wrote:
Hi Joel,
On 12/10/2015 12:27 PM, Joel Borggrén-Franck wrote:
Question, is it better to remove the throws clauses for the cases
that return null?
I think so; they aren't applicable in those case and it is fine to
remove exceptions in subtypes of course.
Done.
Please also add @Override annotations to the methods in the
subtypes as a check that a new method is not accidentally being
declared.
Doh! Fixed.
New webrev:
http://cr.openjdk.java.net/~jfranck/8057804/webrev.02/
<http://cr.openjdk.java.net/%7Ejfranck/8057804/webrev.02/>
Diff of patch 01 and patch 02 (a diff-diff):
http://cr.openjdk.java.net/~jfranck/8057804/diff_v1-v2.patch
<http://cr.openjdk.java.net/%7Ejfranck/8057804/diff_v1-v2.patch>
Please add an @implSpec note in AnnotatedType saying that "this
implementation returns null", or words to that effect.
Done.
Shouldn't some of the implementation overrides in
AnnotatedTypeFactory.java which throw null now be removed? Ah, I
see the BaseImpl type is in the way. Is there an easy way to
refactor that?
I think it deserves a separate commit. I have another bugfix lined up,
then I plan to refactor refactor the code a bit and clean up and
improve testing.
(I'll take care of the ccc changes once the new spec is finalized.)
Thanks!
New webrev: http://cr.openjdk.java.net/~jfranck/8057804/webrev.03/
<http://cr.openjdk.java.net/%7Ejfranck/8057804/webrev.03/>
Delta vs 01:
http://cr.openjdk.java.net/~jfranck/8057804/diff_v1-v3.patch
<http://cr.openjdk.java.net/%7Ejfranck/8057804/diff_v1-v3.patch>
cheers
/Joel
