{"action":"created","issue":{"url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/1984","labels_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/1984/labels{/name}","comments_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/1984/comments","events_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/1984/events","html_url":"https://github.com/wiredtiger/wiredtiger/pull/1984","id":78342253,"number":1984,"title":"Remove the WT_ATOMIC_CAS_VAL family of macros","user":{"login":"michaelcahill","id":1264383,"avatar_url":"https://avatars.githubusercontent.com/u/1264383?v=3","gravatar_id":"","url":"https://api.github.com/users/michaelcahill","html_url":"https://github.com/michaelcahill","followers_url":"https://api.github.com/users/michaelcahill/followers","following_url":"https://api.github.com/users/michaelcahill/following{/other_user}","gists_url":"https://api.github.com/users/michaelcahill/gists{/gist_id}","starred_url":"https://api.github.com/users/michaelcahill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/michaelcahill/subscriptions","organizations_url":"https://api.github.com/users/michaelcahill/orgs","repos_url":"https://api.github.com/users/michaelcahill/repos","events_url":"https://api.github.com/users/michaelcahill/events{/privacy}","received_events_url":"https://api.github.com/users/michaelcahill/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-20T02:47:52Z","updated_at":"2015-05-21T22:26:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/wiredtiger/wiredtiger/pulls/1984","html_url":"https://github.com/wiredtiger/wiredtiger/pull/1984","diff_url":"https://github.com/wiredtiger/wiredtiger/pull/1984.diff","patch_url":"https://github.com/wiredtiger/wiredtiger/pull/1984.patch"},"body":"Use the boolean versions everywhere.  Having both makes my head hurt, and (unless I'm missing something) doesn't add any more functionality.\r\n\r\nrefs WT-1938"},"comment":{"url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/comments/104439771","html_url":"https://github.com/wiredtiger/wiredtiger/pull/1984#issuecomment-104439771","issue_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/1984","id":104439771,"user":{"login":"agorrod","id":1930121,"avatar_url":"https://avatars.githubusercontent.com/u/1930121?v=3","gravatar_id":"","url":"https://api.github.com/users/agorrod","html_url":"https://github.com/agorrod","followers_url":"https://api.github.com/users/agorrod/followers","following_url":"https://api.github.com/users/agorrod/following{/other_user}","gists_url":"https://api.github.com/users/agorrod/gists{/gist_id}","starred_url":"https://api.github.com/users/agorrod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agorrod/subscriptions","organizations_url":"https://api.github.com/users/agorrod/orgs","repos_url":"https://api.github.com/users/agorrod/repos","events_url":"https://api.github.com/users/agorrod/events{/privacy}","received_events_url":"https://api.github.com/users/agorrod/received_events","type":"User","site_admin":false},"created_at":"2015-05-21T22:26:56Z","updated_at":"2015-05-21T22:26:56Z","body":"> Is there any reason not to use the boolean form of __sync_val_compare_and_swap?\r\n\r\nYup, clang has a compiler optimizer bug related to the bool version:\r\nhttps://github.com/wiredtiger/wiredtiger/blob/develop/src/include/gcc.h#L91"}}