The fix looks good to me.

Thanks,
Alexandr.

On 10/05/16 20:42, Semyon Sadetsky wrote:

On 4/28/2016 7:46 PM, Sergey Bylokhov wrote:
On 28.04.16 19:30, Semyon Sadetsky wrote:
ok, then just make the new field private and do not use the interned
string as a key.
Why do you think it should be private? This flag may be used as a sign
of the upcoming  global UI update.

Because right now it is not used outside of the class, and was a private before the fix.

I made the flag key private. Please review http://cr.openjdk.java.net/~ssadetsky/8046031/webrev.01/

--Semyon


Reply via email to