folly/futures/detail/Core.h: avoid FP from clang's -Wconditional-uninitialized
authorJim Meyering <meyering@fb.com>
Wed, 5 Jul 2017 06:56:54 +0000 (23:56 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Wed, 5 Jul 2017 07:04:24 +0000 (00:04 -0700)
Summary:
Avoid this false positive from clang:

  folly/futures/detail/Core.h:369:15: error: variable 'priority' may be uninitialized when used here [-Werror,-Wconditional-uninitialized]

Reviewed By: yfeldblum

Differential Revision: D5369916

fbshipit-source-id: 49649417d1b799cc3945cba456b42ab438ad2ddc

folly/futures/detail/Core.h

index fd73e27656c8b3d0ab84160ecaec4dd722e2a3ea..30d421c05d249035aa7295e9a4d60b0b0ef27624 100644 (file)
@@ -325,7 +325,8 @@ class Core final {
 
   void doCallback() {
     Executor* x = executor_;
-    int8_t priority;
+    // initialize, solely to appease clang's -Wconditional-uninitialized
+    int8_t priority = 0;
     if (x) {
       if (!executorLock_.try_lock()) {
         executorLock_.lock();