> On Dec 14, 2016, at 8:35 PM, Sundararajan Athijegannathan > <sundararajan.athijegannat...@oracle.com> wrote: > > Hi Mandy, > > Thanks for your review. I updated to handle TOP entries in accept callback > uniformly: http://cr.openjdk.java.net/~sundar/8171138/webrev.01/index.html
395 // Copy TOP files of the "java.base" module (only) 396 if ("java.base".equals(file.moduleName())) { It may be better to throw an InternalError for any entry other than “java.base”. Otherwise looks good. You can fix this before you push. No need for a new webrev. Mandy