<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px">Hi compiler-dev,</span><br style="font-family:arial,sans-serif;font-size:12.727272033691406px"><br style="font-family:arial,sans-serif;font-size:12.727272033691406px">
<font face="arial, sans-serif">Please review this fix for JDK-8042441 which fixes two dependency tracking failures in sjavac.</font><br style="font-family:arial,sans-serif;font-size:12.727272033691406px"><br style="font-family:arial,sans-serif;font-size:12.727272033691406px">
<span style="font-family:arial,sans-serif;font-size:12.727272033691406px">Description:</span><div>A fully qualified identifier for example "alfa.A a;" is not tracked,</div><div>but "import alfa.A; ... A a;" is tracked. The fix tracks the fully qualified identifier by instrumenting Attr.java with a reportDependence in the same way that Resolve.java is already instrumented. Thanks Jan Lahoda for the help on where to instrument.</div>
<div><br><font face="arial, sans-serif">Also, the previous change from . to / in the arguments accidentally broke the dependencies inside the javac_state file. A trivial fix is supplied for this, however the tests did not catch this, since the tests did not use packages with more than one level, ie no dots...thus the tests are improved to actually detect this kind of error. </font></div>
<div><br style="font-family:arial,sans-serif"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px">Link to webrev:</span></div><div><font face="arial, sans-serif"><a href="http://cr.openjdk.java.net/~ohrstrom/webrev-8042441/">http://cr.openjdk.java.net/~ohrstrom/webrev-8042441/</a></font><br style="font-family:arial,sans-serif;font-size:12.727272033691406px">
<br style="font-family:arial,sans-serif;font-size:12.727272033691406px"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px">Link to bug report:</span><br style="font-family:arial,sans-serif;font-size:12.727272033691406px">
<a href="https://bugs.openjdk.java.net/browse/JDK-8042441">https://bugs.openjdk.java.net/browse/JDK-8042441</a><br></div></div>