Skip to content

Commit

Permalink
Fix for a bugprone-use-after-move warning
Browse files Browse the repository at this point in the history
Summary:
Fix for a bugprone-use-after-move warning

Saw this on another diff:

```
CLANGTIDY
raised a bugprone-use-after-move warning on
line 163
-
'options' used after it was moved
fbcode/folly/experimental/io/MuxIOThreadPoolExecutor.cpp:129:7: move occurred here
      options_(std::move(options)),
      ^
```

Reviewed By: ot

Differential Revision: D52474932

fbshipit-source-id: 81751cd54f47aa2e6636e903b6efff51d076fc02
  • Loading branch information
Dan Melnic authored and facebook-github-bot committed Jan 2, 2024
1 parent b4ff536 commit 7e99a5c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion folly/experimental/io/MuxIOThreadPoolExecutor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ MuxIOThreadPoolExecutor::MuxIOThreadPoolExecutor(

setNumThreads(numThreads);
registerThreadPoolExecutor(this);
if (options.enableThreadIdCollection) {
if (options_.enableThreadIdCollection) {
threadIdCollector_ = std::make_unique<ThreadIdWorkerProvider>();
}

Expand Down

0 comments on commit 7e99a5c

Please sign in to comment.