Merge lp://qastaging/~mc-return/compiz/compiz.merge-minor-speed-improvements into lp://qastaging/compiz/0.9.8
Proposed by
MC Return
| Status: | Merged |
|---|---|
| Approved by: | Sam Spilsbury |
| Approved revision: | 3337 |
| Merged at revision: | 3343 |
| Proposed branch: | lp://qastaging/~mc-return/compiz/compiz.merge-minor-speed-improvements |
| Merge into: | lp://qastaging/compiz/0.9.8 |
| Diff against target: |
43 lines (+4/-4) 2 files modified
plugins/opengl/src/programcache.cpp (+2/-2) plugins/opengl/src/vertexbuffer.cpp (+2/-2) |
| To merge this branch: | bzr merge lp://qastaging/~mc-return/compiz/compiz.merge-minor-speed-improvements |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Sam Spilsbury | Approve | ||
|
Review via email:
|
|||
Commit message
Used 'if (normalData.empty ())' instead of 'if (normalData.size () == 0)', because checking for emptiness can be faster (2x).
Replaced post-increment with pre-increment operators, because those can be more efficient for non-primitive types.
Description of the change
Minor speed improvements.
To post a comment you must log in.
Revision history for this message
| MC Return (mc-return) wrote | # |
Revision history for this message
| Sam Spilsbury (smspillaz) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
As per request I am submitting multiple smaller merge requests for non-related changes.