← Back to branch summary

~vanvugt/compiz/blacklist

« back to all changes in this revision

Viewing changes to src/event.cpp

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.

Show diffs side-by-side

added added

removed removed

Lines of Context:
src/event.cpp
972
972
                w = screen->findWindow (event->xclient.window);
973
973
                if (w)
974
974
                {
975
 
                    unsigned int i;
976
 
 
977
 
                    for (i = 0; i < SCREEN_EDGE_NUM; i++)
 
975
                    for (unsigned int i = 0; i < SCREEN_EDGE_NUM; i++)
978
976
                    {
979
977
                        if (event->xclient.window == screenEdge[i].id)
980
978
                        {
1016
1014
                w = screen->findWindow (event->xclient.window);
1017
1015
                if (w)
1018
1016
                {
1019
 
                    unsigned int i;
1020
 
 
1021
 
                    for (i = 0; i < SCREEN_EDGE_NUM; i++)
 
1017
                    for (unsigned int i = 0; i < SCREEN_EDGE_NUM; i++)
1022
1018
                    {
1023
1019
                        if (xdndWindow == screenEdge[i].id)
1024
1020
                        {
1696
1692
            if (w)
1697
1693
            {
1698
1694
                unsigned long wState, state;
1699
 
                int           i;
1700
1695
 
1701
1696
                wState = w->state ();
1702
1697
 
1703
 
                for (i = 1; i < 3; i++)
 
1698
                for (int i = 1; i < 3; i++)
1704
1699
                {
1705
1700
                    state = cps::windowStateMask (event->xclient.data.l[i]);
1706
1701
                    if (state & ~CompWindowStateHiddenMask)