From: Christopher Dykes <cdykes@fb.com>
Date: Wed, 4 May 2016 17:26:43 +0000 (-0700)
Subject: Don't attempt to implicitly convert an atomic<T> to bool
X-Git-Tag: 2016.07.26~285
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=36f174f920a35c7de58906b216008f8b56bd7231;p=folly.git

Don't attempt to implicitly convert an atomic<T> to bool

Summary: MSVC doesn't like it, so the other uses were changed to .load() explicitly a while back, but this use has been added since then.

Reviewed By: yfeldblum

Differential Revision: D3255261

fb-gh-sync-id: 11d49902eb3ef7113a7c57cc760353554a487496
fbshipit-source-id: 11d49902eb3ef7113a7c57cc760353554a487496
---

diff --git a/folly/AtomicIntrusiveLinkedList.h b/folly/AtomicIntrusiveLinkedList.h
index a78b685f..e7455422 100644
--- a/folly/AtomicIntrusiveLinkedList.h
+++ b/folly/AtomicIntrusiveLinkedList.h
@@ -68,7 +68,7 @@ class AtomicIntrusiveLinkedList {
   }
 
   bool empty() const {
-    return head_ == nullptr;
+    return head_.load() == nullptr;
   }
 
   /**