hhuijser/liferay-portal — pull request #2164 by willnewbury (2014-10-10 19:09:06+00:00)
Title: LPS-49294 Correction

I reverted your change to move away from AtomicBooleans - see the email that we got about the PACL tests failing. However, I did fix the formatting that I missed, sorry about that.

I also talked with Brian about what the proper naming convention should be in situations like these, and after some consideration, we decided that it would probably be best if we just removed methods and fields from the list of logically immutable classes.

One question I do have though - why are we only checking for statickable fields when these fields are also potentially immutable.