From be27cf764964e2538d8d7ed7700817a05a416f45 Mon Sep 17 00:00:00 2001 From: Murai Takashi Date: Fri, 21 Dec 2018 06:44:12 +0900 Subject: [PATCH] accelerants/common: Fix PVS 1116, 1117 Fix there are two 'if' statements with identical conditional expressions. Change-Id: I3396d4e34b70dd293957a892cd603033d6664699 Reviewed-on: https://review.haiku-os.org/789 Reviewed-by: Adrien Destugues --- .../accelerants/common/validate_display_mode.cpp | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/add-ons/accelerants/common/validate_display_mode.cpp b/src/add-ons/accelerants/common/validate_display_mode.cpp index df3a2cb1f3..64558cef16 100644 --- a/src/add-ons/accelerants/common/validate_display_mode.cpp +++ b/src/add-ons/accelerants/common/validate_display_mode.cpp @@ -146,14 +146,14 @@ is_display_mode_within_bounds(display_mode& mode, const display_mode& low, return false; // Check vertical timing - if (mode.timing.h_display < low.timing.h_display - || mode.timing.h_display > high.timing.h_display - || mode.timing.h_sync_start < low.timing.h_sync_start - || mode.timing.h_sync_start > high.timing.h_sync_start - || mode.timing.h_sync_end < low.timing.h_sync_end - || mode.timing.h_sync_end > high.timing.h_sync_end - || mode.timing.h_total < low.timing.h_total - || mode.timing.h_total > high.timing.h_total) + if (mode.timing.v_display < low.timing.v_display + || mode.timing.v_display > high.timing.v_display + || mode.timing.v_sync_start < low.timing.v_sync_start + || mode.timing.v_sync_start > high.timing.v_sync_start + || mode.timing.v_sync_end < low.timing.v_sync_end + || mode.timing.v_sync_end > high.timing.v_sync_end + || mode.timing.v_total < low.timing.v_total + || mode.timing.v_total > high.timing.v_total) return false; // Check pixel clock