← Back to branch summary

~mhr3/unity/places-group-focus-overlay

« back to all changes in this revision

Viewing changes to launcher/LauncherController.cpp

  • Committer: Tarmac
  • Author(s): MC Return
  • Date: 2012-08-03 11:58:29 UTC
  • mfrom: (2528.1.22 unity)
  • Revision ID: tarmac-20120803115829-2isyblrrxygwiarj
Optimized performance and style following suggestions reported by cppcheck:

1. Reduced the scope of various variables.

2. Used prefix ++ operators for non-primitive types, because those can be more efficient than post-increment. Post-increment usually keeps a copy of the previous value, adds extra code and is slower.. Fixes: . Approved by Brandon Schaefer, Marco Trevisan (Treviño).

Show diffs side-by-side

added added

removed removed

Lines of Context:
launcher/LauncherController.cpp
941
941
  LauncherModel::iterator it;
942
942
 
943
943
  // Shortcut to start launcher icons. Only relies on Keycode, ignore modifier
944
 
  for (it = pimpl->model_->begin(); it != pimpl->model_->end(); it++)
 
944
  for (it = pimpl->model_->begin(); it != pimpl->model_->end(); ++it)
945
945
  {
946
946
    if ((XKeysymToKeycode(display, (*it)->GetShortcut()) == key_code) ||
947
947
        ((gchar)((*it)->GetShortcut()) == key_string[0]))