Answering to myself here:

On 18/10/2019 13:28, Maurizio Cimadamore wrote:


One high-level gripe which is pointing at a failure of the j.l.model API is the lack of a way to get to the canonical constructor directly; we have this issue both in core reflection and source reflection, and I think we should address that, as both serialization and javadoc has to DYI around that.


We discussed more about this offline today and we concluded that, for the time being, this is a non-issue, both for core reflection and for source reflection. These API are traditionally 'lumpy' meaning that e.g. records and classes have the same representation (e.g. j.l.Class and j.l.model.TypeElement). So, adding an extra method like 'getCanonicalConstructor' feels yet of another of those arguable API moves where we add a method X which only makes sense if the predicate Y yields true. Moving forward we will have better ways to deal with lumpy APIs like these (see pattern matching) - in the meantime, we think it's better to leave this alone and avoid excessive pollution of the reflective APIs. Of course, should this ever become an usability pain point, we can always add helper methods later, as the records feature exits its preview period.

Cheers
Maurizio



Maurizio

On 17/10/2019 20:45, Jonathan Gibbons wrote:
cc: javadoc-dev@openjdk.java.net

--Jon


On 10/17/2019 12:43 PM, Vicente Romero wrote:
Hi,

Please review the javadoc code for JEP 359 (Records), this webrev contains only the javadoc code as we have decided to split the new code in clusters to make the review process easier.

Thanks in advance for the feedback,
Vicente

PS, Jon is the author of this code please keep him in the loop

http://cr.openjdk.java.net/~vromero/records.review/javadoc/webrev.00/

Reply via email to