nuke UNSYNCHRONIZED
authorPhilip Pronin <philipp@fb.com>
Thu, 29 Dec 2016 00:28:30 +0000 (16:28 -0800)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Thu, 29 Dec 2016 00:33:08 +0000 (16:33 -0800)
commit1c3b25b2e85f87c8097abd71301bbd285a16b21f
tree87b3e84a34134fab8758f82d4cd6e0c48d3b1d0f
parentf4dd104c297892b176b85723f0c6b0fd9214f091
nuke UNSYNCHRONIZED

Summary:
API of `UNSYNCHRONIZED` is confusing as if you have two nested
`SYNCHRONIZED` blocks, `UNSYNCHRONIZED` always unlocks the inner-most,
even if you pass in the variable name used in the outer `SYNCHRONIZED`
block.

The macro was marked as "deprecated" in D3526489, remove it here.

Reviewed By: yfeldblum

Differential Revision: D4371297

fbshipit-source-id: 13ddc1ff77cb3d5045844c5ade0e95dbe2bccf6d
folly/Synchronized.h
folly/test/SynchronizedTest.cpp
folly/test/SynchronizedTestLib-inl.h