← Back to branch summary

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

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

« back to all changes in this revision

Viewing changes to plugins/unity-mt-grab-handles/src/unity-mt-grab-handles.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).

files modified:
UnityCore/HomeLens.cpp

dash/ResultView.cpp

dash/ResultViewGrid.cpp

launcher/Launcher.cpp

launcher/LauncherController.cpp

launcher/LauncherModel.cpp

panel/PanelIndicatorEntryView.cpp

panel/PanelMenuView.cpp

plugins/unity-mt-grab-handles/src/unity-mt-grab-handles.cpp

plugins/unitydialog/src/unitydialog.cpp

plugins/unityshell/src/nux-layout-accessible.cpp

plugins/unityshell/src/unity-launcher-accessible.cpp

plugins/unityshell/src/unity-switcher-accessible.cpp

plugins/unityshell/src/unityshell.cpp

unity-shared/BGHash.cpp

unity-shared/DashStyle.cpp

unity-shared/IconRenderer.cpp

unity-shared/PluginAdapterCompiz.cpp

Show diffs side-by-side

added added

removed removed

plugins/unity-mt-grab-handles/src/unity-mt-grab-handles.cpp

292

292

CompWindowVector::const_iterator cit = clientListStacking.begin();

293

293

CompWindowVector::const_iterator oit = mLastClientListStacking.begin();

294

294

295

for (; cit != clientListStacking.end(); cit++, oit++)

295

for (; cit != clientListStacking.end(); ++cit, oit++)

296

296

{

297

297

/* All clients from this point onwards in cit are invalidated

298

298

* so splice the list to the end of the new client list

Older »