java.io.File field "path" is not declared final
Alan Bateman
Alan.Bateman at oracle.com
Fri Feb 17 12:41:12 UTC 2012
On 17/02/2012 12:37, David Holmes wrote:
> On 17/02/2012 10:11 PM, Alan Bateman wrote:
>> On 17/02/2012 12:00, Rémi Forax wrote:
>>> :
>>> Better with the attachment inlined :)
>> Thanks Rémi, this looks okay to me although I probably would have used
>> putObjectVolatile for the path field.
>
> You only need one "volatile" store and it should be the last one. The
> approximate affect is:
>
> path = ...
> membar: LoadStore | StoreStore
> prefixlength = ...
> membar: StoreLoad
I understand, I just remarking that I probably would have used a
volatile store for both to make it clearer but that would be less
efficient.
-Alan
More information about the core-libs-dev
mailing list