Hi Oleg,

> >> I'd synchronized setting/getting of access field.  And (perhaps) I'd only 
> >> allow
> >> set this field once.
> >
> > Good ideas. I implemented both suggestions. Webrev is updated at the
> > same URL.
> 
> As far as I can see you add synchronization only to setter, but you
> should also add
> synchronization when you read this field.  So you should either make
> access field
> volatile, or add synchronized getter and use it to get the field.

Duh, you are right. I should better think twice before I send new
patch :-). Now it should be correct, what do you think?

Webrev still here: http://kennke.org/~roman/componentaccess/webrev/

/Roman

-- 
http://kennke.org/blog/

Attachment: signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil

Reply via email to