fix: race condition in use of of atomic variable (#246)

serenity-rs · Mar 7, 2024 · 9def484 · 9def484
1 parent 2cd2ee9
commit 9def484
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/examples/event_handler/main.rs b/examples/event_handler/main.rs
@@ -57,10 +57,12 @@ async fn event_handler(
         }
         serenity::FullEvent::Message { new_message } => {
             if new_message.content.to_lowercase().contains("poise") {
-                let mentions = data.poise_mentions.load(Ordering::SeqCst) + 1;
-                data.poise_mentions.store(mentions, Ordering::SeqCst);
+                let old_mentions = data.poise_mentions.fetch_add(1, Ordering::SeqCst);
                 new_message
-                    .reply(ctx, format!("Poise has been mentioned {} times", mentions))
+                    .reply(
+                        ctx,
+                        format!("Poise has been mentioned {} times", old_mentions + 1),
+                    )
                     .await?;
             }
         }