Hello Priya,

The specific changes looks ok to me.

As a separate effort, I recommend searching for other accesses in the Doclet
to javax.lang.model through calls in Utils, these need to be investigated.

Note: there maybe legitimate cases where raw members and types are needed,
but for the most part the Doclet should use VMT for all these purposes. I
removed
some of these while doing the VMT work (Index generators), but there maybe
more,
in any case this effort needs to be done cautiously with a before/after
comparison.

Kumar


On Mon, May 28, 2018 at 4:05 AM, Priya Lakshmi Muthuswamy <
[email protected]> wrote:

> Hi,
>
> Kindly review the fix for https://bugs.openjdk.java.net/browse/JDK-8202627
> webrev : http://cr.openjdk.java.net/~pmuthuswamy/8202627/webrev.00/
>
> Thanks,
> Priya
>

Reply via email to