Refactor the ObjectLock class

I think there are something should be refactored.
(1) Use ReentrantLock and ReentrantReadWriteLock to replace the old wait() and notify();
(2) Simplify the usage of ObjectLock’s apis, for example, confirm ();
(3) Refactor some variables’ names to make them more intuitive.

What have been done in the objectlock_patch1.txt.
(1) Rename some variables, for example, _writeLock, _readLock to _writeTx, _readTxes
(2) Use AtomicInteger to instead of LOCK and _idcount
(3) Use an Enum instead of ACTION_READ, ACTION_WRITE, ACTION_CREATE, ACTION_UPDATE
(4) Use HashSet to instead LinkedTx

What should be done in the next patch?
(1) Use ReetrantLock and Condition to instead of "synchronized, wait, notifyAll". Escpicially, the Condition will make the code more understandable.
(2) Extract the duplicated code in acquireLoadLock, acquireUpdateLock, update methods into a new method.

Rename some variables, for example, _writeLock, _readLock to _writeTx, _readTxes

Can you please avoid using abbreviations for variable/property names ? writeTransactions and readTransactions are so much easier to read and do NOT require any guessing from the human side. Apart from that, nice work.

Thank you for your sugguestion, it is very useful.
I just changed as what you said. see attachment objectlock_patch2.txt

Final patch with some minor changes and one bugfix. At upgrade _readTransactions had been set to null which caused lots of NPE at cpactf.

What has been done in patch-C3085-20110526.txt?
(1) Extract the duplicated code in acquireLoadLock, acquireUpdateLock, update methods into a new method.

Committed patch patch-C3085-20110526.txt with some minor code formatting.

What has been done in patch-C3085-20110531.txt?
(1) Use ReetrantLock and Condition to instead of "synchronized, wait, notifyAll". The Condition will make the code more understandable.
(2) Simplify the meaning of the property _delete. It means that if the object is deleted, than the _deleted is true forever.
(3) Delete the property _waitCount, _waitCount can be replaced by Condition now.

Some cleanup for ObjectLock.
(1) Modify the visibility of some methods
(2) Delete _waitCount field
(3) Modify the try-catch in delete()
(4) MOdify some comments

Once all subtask has been resolved we can also close this one.