From 4a113a01d093085e50deeffd449982f567df0d57 Mon Sep 17 00:00:00 2001 From: Adrien Destugues Date: Wed, 17 Jul 2019 20:32:37 +0200 Subject: [PATCH] PVS V792: use short circuit operators Avoids calling all the functions, we can stop once one of them is true. Change-Id: I92437a60d3d52efd5a11547a11b88e3bf232e66b Reviewed-on: https://review.haiku-os.org/c/1624 Reviewed-by: waddlesplash --- src/preferences/mail/AutoConfigView.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/preferences/mail/AutoConfigView.cpp b/src/preferences/mail/AutoConfigView.cpp index c9ec5454ae..1fb74208e2 100644 --- a/src/preferences/mail/AutoConfigView.cpp +++ b/src/preferences/mail/AutoConfigView.cpp @@ -469,9 +469,9 @@ void ServerSettingsView::_DetectMenuChanges() { bool changed = _HasMarkedChanged(fInboundAuthMenu, fInboundAuthItemStart) - | _HasMarkedChanged(fInboundEncryptionMenu, fInboundEncrItemStart) - | _HasMarkedChanged(fOutboundAuthMenu, fOutboundAuthItemStart) - | _HasMarkedChanged(fOutboundEncryptionMenu, fOutboundEncrItemStart); + || _HasMarkedChanged(fInboundEncryptionMenu, fInboundEncrItemStart) + || _HasMarkedChanged(fOutboundAuthMenu, fOutboundAuthItemStart) + || _HasMarkedChanged(fOutboundEncryptionMenu, fOutboundEncrItemStart); if (changed) { BMessage msg(kServerChangedMsg);