make drop_item less confusing

src/ReceiveSubcommands.cc

@@ -960,10 +960,14 @@ static bool drop_item(
     float z,
     uint16_t request_id) {
 
-  PlayerInventoryItem item;
+  // If the game is not BB, forward the request to the leader instead of
+  // generating the item drop command
+  if (l->version != GameVersion::BB) {
+    return false;
+  }
 
   // If the game is BB, run the rare + common drop logic
-  if (l->version == GameVersion::BB) {
+  PlayerInventoryItem item;
   if (!l->item_creator.get()) {
     throw runtime_error("received box drop subcommand without item creator present");
   }
@@ -975,12 +979,6 @@ static bool drop_item(
     item.data = l->item_creator->on_box_item_drop(area);
   }
 
-    // If the game is not BB, forward the request to the leader instead of
-    // generating the item drop command
-  } else {
-    return false;
-  }
-
   item.data.id = l->generate_item_id(0xFF);
 
   if (l->flags & Lobby::Flag::ITEM_TRACKING_ENABLED) {