frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sync: compress two if statements 

No functional changes, just merges a > and == condition into a >= condition.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Keith Packard <keithp@keithp.com>
This commit is contained in:
Peter Hutterer 2013-10-15 10:11:20 +10:00
parent 7cf1b595c8
commit 5c7cfd4c69
1 changed files with 19 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
Xext/sync.c
View File

@ -1031,42 +1031,29 @@ SyncComputeBracketValues(SyncCounter * pCounter)
} }
else if (pTrigger->test_type == XSyncNegativeTransition && else if (pTrigger->test_type == XSyncNegativeTransition &&
ct != XSyncCounterNeverIncreases) { ct != XSyncCounterNeverIncreases) {
if (XSyncValueGreaterThan(pCounter->value, pTrigger->test_value) && if (XSyncValueGreaterOrEqual(pCounter->value, pTrigger->test_value) &&
XSyncValueGreaterThan(pTrigger->test_value, psci->bracket_less)) XSyncValueGreaterThan(pTrigger->test_value, psci->bracket_less)) {
{ /*
psci->bracket_less = pTrigger->test_value; * If the value is exactly equal to our threshold, we want one
pnewltval = &psci->bracket_less; * more event in the negative direction to ensure we pick up
} * when the value is less than this threshold.
else if (XSyncValueEqual(pCounter->value, pTrigger->test_value) && */
XSyncValueGreaterThan(pTrigger->test_value, psci->bracket_less = pTrigger->test_value;
psci->bracket_less)) { pnewltval = &psci->bracket_less;
/*
* The value is exactly equal to our threshold. We want one
* more event in the negative direction to ensure we pick up
* when the value is less than this threshold.
*/
psci->bracket_less = pTrigger->test_value;
pnewltval = &psci->bracket_less;
} }
} }
else if (pTrigger->test_type == XSyncPositiveTransition && else if (pTrigger->test_type == XSyncPositiveTransition &&
ct != XSyncCounterNeverDecreases) { ct != XSyncCounterNeverDecreases) {
if (XSyncValueLessThan(pCounter->value, pTrigger->test_value) && if (XSyncValueLessOrEqual(pCounter->value, pTrigger->test_value) &&
XSyncValueLessThan(pTrigger->test_value, psci->bracket_greater)) XSyncValueLessThan(pTrigger->test_value, psci->bracket_greater)) {
{ /*
psci->bracket_greater = pTrigger->test_value; * If the value is exactly equal to our threshold, we
pnewgtval = &psci->bracket_greater; * want one more event in the positive direction to
} * ensure we pick up when the value *exceeds* this
else if (XSyncValueEqual(pCounter->value, pTrigger->test_value) && * threshold.
XSyncValueLessThan(pTrigger->test_value, */
psci->bracket_greater)) { psci->bracket_greater = pTrigger->test_value;
/* pnewgtval = &psci->bracket_greater;
* The value is exactly equal to our threshold. We want one
* more event in the positive direction to ensure we pick up
* when the value *exceeds* this threshold.
*/
psci->bracket_greater = pTrigger->test_value;
pnewgtval = &psci->bracket_greater;
} }
} }
} /* end for each trigger */ } /* end for each trigger */