diff options
| author | Todd Gamblin <tgamblin@llnl.gov> | 2019-12-21 16:23:54 -0800 |
|---|---|---|
| committer | Todd Gamblin <tgamblin@llnl.gov> | 2019-12-23 18:36:56 -0800 |
| commit | eb8fc4f3be7a1b5dd90313f0a8381bdaca346632 (patch) | |
| tree | 9991d1fe04d223c6ebe606419df757bdf093a44a /.readthedocs.yml | |
| parent | 779ac9fe3e1968bbc8b267712c8827aba5dd985e (diff) | |
| download | spack-eb8fc4f3be7a1b5dd90313f0a8381bdaca346632.tar.gz spack-eb8fc4f3be7a1b5dd90313f0a8381bdaca346632.tar.bz2 spack-eb8fc4f3be7a1b5dd90313f0a8381bdaca346632.tar.xz spack-eb8fc4f3be7a1b5dd90313f0a8381bdaca346632.zip | |
lock transactions: fix non-transactional writes
Lock transactions were actually writing *after* the lock was
released. The code was looking at the result of `release_write()` before
writing, then writing based on whether the lock was released. This is
pretty obviously wrong.
- [x] Refactor `Lock` so that a release function can be passed to the
`Lock` and called *only* when a lock is really released.
- [x] Refactor `LockTransaction` classes to use the release function
instead of checking the return value of `release_read()` / `release_write()`
Diffstat (limited to '.readthedocs.yml')
0 files changed, 0 insertions, 0 deletions