Hi, Alan

  Thanks. Pushed before saw your email, could not credit you on reviewers.


Thanks

Yumin

On 9/4/20 2:10 AM, Alan Bateman wrote:
On 04/09/2020 05:37, Yumin Qi wrote:
HI, Sundar

  Thanks for review.

On 9/3/20 6:34 PM, sundararajan.athijegannat...@oracle.com wrote:
Looks good to me.

Few minor comment:

* traceFileStream (and even the preexisting mainArgument) is accessed only 
inside GenerateJLIClassesPlugin. Could be private?

I will fix them before push. (Certainly will build first to verify that).
I went through the refactoring of the jlink plugin and I think it looks good 
too.

-Alan.

Reply via email to