When there's no need to know whether the user has set the value of a configurable option (for logging for instance) the return of xf86ReturnOptValBool() can be assigned directly.
Signed-off-by: Egbert Eich <[email protected]> --- src/vesa.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/vesa.c b/src/vesa.c index 931341a..dbe4ba6 100644 --- a/src/vesa.c +++ b/src/vesa.c @@ -838,8 +838,8 @@ VESAPreInit(ScrnInfoPtr pScrn, int flags) xf86ProcessOptions(pScrn->scrnIndex, pScrn->options, pVesa->Options); /* Use shadow by default */ - if (xf86ReturnOptValBool(pVesa->Options, OPTION_SHADOW_FB, TRUE)) - pVesa->shadowFB = TRUE; + pVesa->shadowFB = xf86ReturnOptValBool(pVesa->Options, OPTION_SHADOW_FB, + TRUE); /* * Use default refresh by default. Too many VBE 3.0 @@ -848,10 +848,9 @@ VESAPreInit(ScrnInfoPtr pScrn, int flags) pVesa->defaultRefresh = xf86ReturnOptValBool(pVesa->Options, OPTION_DFLT_REFRESH, TRUE); - pVesa->ModeSetClearScreen = FALSE; - if (xf86ReturnOptValBool(pVesa->Options, OPTION_MODESET_CLEAR_SCREEN, - FALSE)) - pVesa->ModeSetClearScreen = TRUE; + pVesa->ModeSetClearScreen = + xf86ReturnOptValBool(pVesa->Options, + OPTION_MODESET_CLEAR_SCREEN, FALSE); if (!pVesa->defaultRefresh && !pVesa->strict_validation) VBESetModeParameters(pScrn, pVesa->pVbe); -- 1.8.1.4 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
