[Checkins] [zopefoundation/ZODB] 532fd7: Modernize and cleanup locking

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/zopefoundation/ZODB
  Commit: 532fd78a10e2fe0c468e2b4241fe9b6c8377cbd7
      https://github.com/zopefoundation/ZODB/commit/532fd78a10e2fe0c468e2b4241fe9b6c8377cbd7
  Author: Jim Fulton <jim at zope.com>
  Date:   2016-06-18 (Sat, 18 Jun 2016)

  Changed paths:
    M src/ZODB/ActivityMonitor.py
    M src/ZODB/BaseStorage.py
    M src/ZODB/DB.py
    M src/ZODB/DemoStorage.py
    M src/ZODB/FileStorage/FileStorage.py
    M src/ZODB/FileStorage/fspack.py
    M src/ZODB/FileStorage/interfaces.py
    M src/ZODB/MappingStorage.py
    M src/ZODB/blob.py
    M src/ZODB/scripts/zodbload.py
    M src/ZODB/tests/MVCCMappingStorage.py
    M src/ZODB/tests/testDemoStorage.py
    M src/ZODB/tests/test_storage.py
    M src/ZODB/utils.py
    M src/ZODB/utils.txt

  Log Message:
  -----------
  Modernize and cleanup locking

- Move from try/finally to with

- Start phasing out the _lock_acquire/_lock_release shortcuts

- Replace simpler @locked decorators with with statements.

  I've come to prefer the with style tp the decorator style.
  I think it's clearer.

  (The decorators preceeded the Python with statement.)

  I left the decorators in cases where they were used with preconditions.

This is mostly pretty mechanical, although it got a little delicate in
places and ... tests


  Commit: 0ed7579d8cde8bc6f9a6774d7d3b4ff450a04688
      https://github.com/zopefoundation/ZODB/commit/0ed7579d8cde8bc6f9a6774d7d3b4ff450a04688
  Author: Jim Fulton <jim at zope.com>
  Date:   2016-06-19 (Sun, 19 Jun 2016)

  Changed paths:
    M src/ZODB/MappingStorage.py
    M src/ZODB/utils.py

  Log Message:
  -----------
  de-obfuscated some delicate locking code


  Commit: 82daec9f92cb023dfcf3d034d4afcb23f94036c5
      https://github.com/zopefoundation/ZODB/commit/82daec9f92cb023dfcf3d034d4afcb23f94036c5
  Author: Jim Fulton <jim at jimfulton.info>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M src/ZODB/ActivityMonitor.py
    M src/ZODB/BaseStorage.py
    M src/ZODB/DB.py
    M src/ZODB/DemoStorage.py
    M src/ZODB/FileStorage/FileStorage.py
    M src/ZODB/FileStorage/fspack.py
    M src/ZODB/FileStorage/interfaces.py
    M src/ZODB/MappingStorage.py
    M src/ZODB/blob.py
    M src/ZODB/scripts/zodbload.py
    M src/ZODB/tests/MVCCMappingStorage.py
    M src/ZODB/tests/testDemoStorage.py
    M src/ZODB/tests/test_storage.py
    M src/ZODB/utils.py
    M src/ZODB/utils.txt

  Log Message:
  -----------
  Merge pull request #72 from zopefoundation/clean-up-locking

Modernize and cleanup locking


Compare: https://github.com/zopefoundation/ZODB/compare/f9f3ddaebac1...82daec9f92cb