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

« back to all changes in this revision

Viewing changes to plugins/unityshell/src/nux-layout-accessible.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:
203
203
 
204
204
  element_list = layout->GetChildren();
205
205
 
206
 
  for (it = element_list.begin(); it != element_list.end(); it++, result++)
 
206
  for (it = element_list.begin(); it != element_list.end(); ++it, result++)
207
207
  {
208
208
    current_area = *it;
209
209
    if (current_area == area)