diff --git a/include/battle_controllers.h b/include/battle_controllers.h
index 0716ae2e54..3eddc95578 100644
--- a/include/battle_controllers.h
+++ b/include/battle_controllers.h
@@ -82,19 +82,19 @@ enum {
    (gBattleControllerExecFlags & (1u << battler))
 
 #define MARK_BATTLE_CONTROLLER_MESSAGE_OUTBOUND_OVER_LINK(battler) \
-   gBattleControllerExecFlags |= (1u << battler) << (32 - MAX_BATTLERS_COUNT)
+   gBattleControllerExecFlags |= ((1u << battler) << (32 - MAX_BATTLERS_COUNT))
 
 #define MARK_BATTLE_CONTROLLER_MESSAGE_SYNCHRONIZED_OVER_LINK(battler) \
    gBattleControllerExecFlags &= ~((1 << 28) << (battler))
 
 #define MARK_BATTLE_CONTROLLER_ACTIVE_FOR_PLAYER(battler, playerId) \
-   gBattleControllerExecFlags |= (1u << battler) << ((playerId) << 2)
+   gBattleControllerExecFlags |= ((1u << battler) << ((playerId) << 2))
 
 #define MARK_BATTLE_CONTROLLER_IDLE_FOR_PLAYER(battler, playerId) \
-   gBattleControllerExecFlags &= ~(1u << battler) << ((playerId) * 4)
+   gBattleControllerExecFlags &= ~((1u << battler) << ((playerId) * 4))
 
 #define IS_BATTLE_CONTROLLER_ACTIVE_FOR_PLAYER(battler, playerId) \
-   (gBattleControllerExecFlags & (1u << battler) << ((playerId) * 4))
+   (gBattleControllerExecFlags & ((1u << battler) << ((playerId) * 4)))
 
 // This actually checks if a specific controller is active on any player or if
 // *any* controller is pending sync over link communications, but the macro name