message: Ensuring nullptr passing won't cause explosions. Probably unnecessary based on logic in the function but better to simply drop out of the function on nullptr than waste time comparing to nullptr.
diff: @@ -146,9 +146,14 @@ void FactoryListBox::removeItem(Factory* factory)
  }
  +/**
  + * Sets the current selection.
  + *
  + * \param f Pointer to a Factory object. Save to pass \c nullptr.
  + */
  void FactoryListBox::currentSelection(Factory* f)
  {
  - if (mItems.empty()) { return; }
  + if (mItems.empty() || f == nullptr) { return; }
  for (size_t i = 0; i < mItems.size(); ++i)
  {
  FactoryListBoxItem* item = static_cast<FactoryListBoxItem*>(mItems[i]);
  