Added documentation and minor cosmetic changes.

src/items/item_manager.cpp

@@ -322,6 +322,9 @@ void  ItemManager::checkItemHit(AbstractKart* kart)
                 collectedItem(*i, kart);
             else if (NetworkConfig::get()->isServer())
             {
+                // Only the server side detects item being collected
+                // A client does the collection upon receiving the 
+                // event from the server!
                 collectedItem(*i, kart);
                 NetworkWorld::getInstance()->collectedItem(*i, kart);
             }

src/network/network_world.cpp

@@ -66,6 +66,11 @@ bool NetworkWorld::isRaceOver()
 }   // isRaceOver
 
 // ----------------------------------------------------------------------------
+/** Called from the item manager on a server. It triggers a notification to
+ *  all clients in the GameEventsProtocol.
+ *  \param item The item that was collected.
+ *  \param kart The kart that collected the item.
+ */
 void NetworkWorld::collectedItem(Item *item, AbstractKart *kart)
 {
     // this is only called in the server

src/network/protocols/game_events_protocol.cpp

@@ -53,15 +53,15 @@ bool GameEventsProtocol::notifyEvent(Event* event)
             }
             uint32_t item_id = data.gui32();
             uint8_t powerup_type = data.gui8(4);
-            uint8_t kart_race_id = data.gui8(5);
+            uint8_t player_id = data.gui8(5);
             // now set the kart powerup
             AbstractKart* kart = World::getWorld()->getKart(
-                STKHost::get()->getGameSetup()
+                      STKHost::get()->getGameSetup()
-                              ->getProfile(kart_race_id)->getWorldKartID());
+                                    ->getProfile(player_id)->getWorldKartID());
             ItemManager::get()->collectedItem(
-                ItemManager::get()->getItem(item_id),
+                                          ItemManager::get()->getItem(item_id),
-                kart,
+                                          kart,
-                powerup_type);
+                                          powerup_type);
             Log::info("GameEventsProtocol", "Item %d picked by a player.",
                        powerup_type);
         }   break;