On 18/02/2016 02:09, Wang Weijun wrote:
IMO a noreg-trivial is enough.
Why ?
Yes -- the fix may be trivial but a testcase helps ensure that no
engineer even reverts this field in the future without seeing this
issue. Code goes through churning and people forget edits made in the
past - esp. the trivial ones.
regards,
Sean.
--Max
在 2016年2月18日，09:28，Xuelei Fan <[email protected]> 写道：
Hi,
A new test case was added. Please review the update:
http://cr.openjdk.java.net/~xuelei/8149417/webrev.01/
Thanks,
Xuelei
On 2/15/2016 9:07 PM, Sean Mullan wrote:
Looks good.
--Sean
On 02/15/2016 01:33 AM, Xuelei Fan wrote:
Hi,
Please review this code cleanup:
http://cr.openjdk.java.net/~xuelei/8149417/webrev.00/
Simple fix, adding final key word to a static flag.
Thanks,
Xuelei
