← Back to branch summary

~compiz-team/compiz/compiz.fix_1088399

~compiz-team/compiz/compiz.fix_1088399

« back to all changes in this revision

Viewing changes to kde/window-decorator-kde4/window.cpp

Committer: Tarmac
Author(s): MC Return
Date: 2012-12-05 10:52:35 UTC
mfrom: (3473.3.96 compiz.merge-performance-improvements-prevent-unnecessary-operations-if-we-return)
Revision ID: tarmac-20121205105235-2i3ttv54ln1fjeqt

Minor Performance Optimizations:

* Return ASAP./Prevent executing any unnecessary operations if we return.
* Used De Morgan's laws to merge and simplify if statements.

Other Changes:

* C++ Style: Declared iterator variables inside the for loops they are used in.
* No logic changes have been made.

Approved by Daniel van Vugt.

files modified:
compizconfig/integration/gnome/src/ccs_gnome_integration.c

compizconfig/libcompizconfig/src/compiz.cpp

kde/window-decorator-kde4/window.cpp

plugins/animation/src/grid.cpp

plugins/composite/src/screen.cpp

plugins/composite/src/window.cpp

plugins/cube/src/cube.cpp

plugins/cubeaddon/src/cubeaddon.cpp

plugins/decor/src/decor.cpp

plugins/expo/src/expo.cpp

plugins/firepaint/src/firepaint.cpp

plugins/group/src/selection.cpp

plugins/imgpng/src/imgpng.cpp

plugins/imgsvg/src/imgsvg.cpp

plugins/move/src/move.cpp

plugins/obs/src/obs.cpp

plugins/opengl/src/matrix.cpp

plugins/opengl/src/vector.cpp

plugins/regex/src/regex.cpp

plugins/scale/src/scale.cpp

plugins/scaleaddon/src/scaleaddon.cpp

plugins/shift/src/shift.cpp

plugins/showmouse/src/showmouse.cpp

plugins/switcher/src/switcher.cpp

plugins/trailfocus/src/trailfocus.cpp

plugins/wall/src/wall.cpp

plugins/wobbly/src/wobbly.cpp

plugins/workspacenames/src/workspacenames.cpp

src/action.cpp

src/event.cpp

src/modifierhandler.cpp

src/option.cpp

src/screen.cpp

src/session.cpp

src/window.cpp

Show diffs side-by-side

added added

removed removed

kde/window-decorator-kde4/window.cpp

1570

1570

1571

1571

if (status)

1572

1572

{

1573

int i;

1574

1575

for (i = 0; i < n; i++)

1573

for (int i = 0; i < n; i++)

1576

1574

{

1577

1575

if (p[i] == Atoms::wmTakeFocus)

1578

1576

mSupportTakeFocus = true;

1627

1625

{

1628

1626

NETRootInfo *rootInfo = Decorator::rootInfo ();

1629

1627

QString name;

1630

int i;

1631

1628

int winDesktop = desktop ();

1632

1629

QAction *action;

1633

1630

const int BASE = 10;

1642

1639

action->setChecked (winDesktop == NET::OnAllDesktops);

1643

1640

mDesktopMenu->addSeparator ();

1644

1641

1645

for (i = 1; i <= numberOfDesktops; i++)

1642

for (int i = 1; i <= numberOfDesktops; i++)

1646

1643

{

1647

1644

QString basic_name ("%1 %2");

1648

1645

if (i < BASE)

Older »