diff options
author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2022-10-17 11:48:57 -0700 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2022-10-17 11:48:57 -0700 |
commit | e285658e60d9cf772dda4b2c713ad2a0d4cef569 (patch) | |
tree | 6e528fcfec4d5fc1366a0d54c5e440b0ff35ebf7 | |
parent | 2f2436b4fe70a1f1a47c4ed112e8aefba3c8efe3 (diff) |
VESAValidMode: remove duplicate call to VESAGetRec
Found by cppcheck:
src/vesa.c:356:11: style: Redundant initialization for 'pVesa'. The initialized value is overwritten before it is read. [redundantInitialization]
pVesa = VESAGetRec(pScrn);
^
src/vesa.c:350:19: note: pVesa is initialized
VESAPtr pVesa = VESAGetRec(pScrn);
^
src/vesa.c:356:11: note: pVesa is overwritten
pVesa = VESAGetRec(pScrn);
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
-rw-r--r-- | src/vesa.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -353,8 +353,6 @@ VESAValidMode(SCRN_ARG_TYPE arg, DisplayModePtr p, Bool flag, int pass) DisplayModePtr mode; float v; - pVesa = VESAGetRec(pScrn); - if (pass != MODECHECK_FINAL) { if (!warned) { xf86DrvMsg(pScrn->scrnIndex, X_WARNING, "VESAValidMode called unexpectedly\n"); |