From 16a1ba69a682fbe7855682448ac0340574a67035 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kai=20K=C3=B6hne?= Date: Thu, 29 Apr 2021 15:55:11 +0200 Subject: Disable spurious MSVC warning C4273 in Qt Gui, Qt OpenGL For optimized/release builds, cl 19.28 produces some false positive "potential divide by 0" warnings even for code where no valid code path can lead to such a division. Fixes: QTBUG-92940 Pick-to: 6.1 Change-Id: I461f9104d7b9dc2d74839bf30bebb2b9d4cf2db3 Reviewed-by: Friedemann Kleint Reviewed-by: Eirik Aavitsland --- src/gui/image/qimage.cpp | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/gui/image/qimage.cpp') diff --git a/src/gui/image/qimage.cpp b/src/gui/image/qimage.cpp index 8011aa2eb5..95be9772c8 100644 --- a/src/gui/image/qimage.cpp +++ b/src/gui/image/qimage.cpp @@ -77,6 +77,11 @@ QT_BEGIN_NAMESPACE +// MSVC 19.28 does show spurious warning "C4723: potential divide by 0" for code that divides +// by height() in release builds. Anyhow, all the code paths in this file are only executed +// for valid QImage's, where height() cannot be 0. Therefore disable the warning. +QT_WARNING_DISABLE_MSVC(4723) + static inline bool isLocked(QImageData *data) { return data != nullptr && data->is_locked; -- cgit v1.2.3