public inbox for cygwin-apps-cvs@sourceware.org help / color / mirror / Atom feed
From: corinna@sourceware.org To: cygwin-apps-cvs@sourceware.org Subject: [setup] branch master, updated. release_2.873-4-g8c34cc9 Date: Thu, 07 Jan 2016 21:28:00 -0000 [thread overview] Message-ID: <20160107212808.89233.qmail@sourceware.org> (raw) https://sourceware.org/git/gitweb.cgi?p=cygwin-setup.git;h=8c34cc982b182a7e1d808668dfb572d7e3332170 commit 8c34cc982b182a7e1d808668dfb572d7e3332170 Author: Corinna Vinschen <corinna@vinschen.de> Date: Thu Jan 7 22:28:03 2016 +0100 Fix logical expressions checking if buttons are checked * AntiVirus.cc (AntiVirusPage::OnActivate): Fix logical expressions checking if a button is checked. * net.cc (NetPage::OnInit): Ditto. * source.cc (SourcePage::OnActivate): Ditto. Signed-off-by: Corinna Vinschen <corinna@vinschen.de> Diff: --- AntiVirus.cc | 14 +++++--------- net.cc | 11 ++++------- source.cc | 15 ++++++--------- 3 files changed, 15 insertions(+), 25 deletions(-) diff --git a/AntiVirus.cc b/AntiVirus.cc index 7dd01c4..876eb02 100644 --- a/AntiVirus.cc +++ b/AntiVirus.cc @@ -85,15 +85,11 @@ AntiVirusPage::OnActivate () { load_dialog (GetHWND ()); // Check to see if any radio buttons are selected. If not, select a default. - if ((!SendMessage - (GetDlgItem (IDC_DISABLE_AV), BM_GETCHECK, 0, - 0) == BST_CHECKED) - && (!SendMessage (GetDlgItem (IDC_LEAVE_AV), BM_GETCHECK, 0, 0) - == BST_CHECKED)) - { - SendMessage (GetDlgItem (IDC_LEAVE_AV), BM_SETCHECK, - BST_CHECKED, 0); - } + if (SendMessage (GetDlgItem (IDC_DISABLE_AV), BM_GETCHECK, 0, 0) + != BST_CHECKED + && SendMessage (GetDlgItem (IDC_LEAVE_AV), BM_GETCHECK, 0, 0) + != BST_CHECKED) + SendMessage (GetDlgItem (IDC_LEAVE_AV), BM_SETCHECK, BST_CHECKED, 0); } bool diff --git a/net.cc b/net.cc index 1402c67..aa734da 100644 --- a/net.cc +++ b/net.cc @@ -135,13 +135,10 @@ NetPage::OnInit () CheckIfEnableNext(); // Check to see if any radio buttons are selected. If not, select a default. - if ((!SendMessage (GetDlgItem (IDC_NET_IE5), BM_GETCHECK, 0, 0) == - BST_CHECKED) - && (!SendMessage (GetDlgItem (IDC_NET_PROXY), BM_GETCHECK, 0, 0) - == BST_CHECKED)) - { - SendMessage (GetDlgItem (IDC_NET_DIRECT), BM_CLICK, 0, 0); - } + if (SendMessage (GetDlgItem (IDC_NET_IE5), BM_GETCHECK, 0, 0) != BST_CHECKED + && SendMessage (GetDlgItem (IDC_NET_PROXY), BM_GETCHECK, 0, 0) + != BST_CHECKED) + SendMessage (GetDlgItem (IDC_NET_DIRECT), BM_CLICK, 0, 0); } long diff --git a/source.cc b/source.cc index 70751d2..90c98b5 100644 --- a/source.cc +++ b/source.cc @@ -103,15 +103,12 @@ SourcePage::OnActivate () load_dialog (GetHWND ()); // Check to see if any radio buttons are selected. If not, select a default. - if ((!SendMessage - (GetDlgItem (IDC_SOURCE_DOWNLOAD), BM_GETCHECK, 0, - 0) == BST_CHECKED) - && (!SendMessage (GetDlgItem (IDC_SOURCE_LOCALDIR), BM_GETCHECK, 0, 0) - == BST_CHECKED)) - { - SendMessage (GetDlgItem (IDC_SOURCE_NETINST), BM_SETCHECK, - BST_CHECKED, 0); - } + if (SendMessage (GetDlgItem (IDC_SOURCE_DOWNLOAD), BM_GETCHECK, 0, 0) + != BST_CHECKED + && SendMessage (GetDlgItem (IDC_SOURCE_LOCALDIR), BM_GETCHECK, 0, 0) + != BST_CHECKED) + SendMessage (GetDlgItem (IDC_SOURCE_NETINST), BM_SETCHECK, + BST_CHECKED, 0); } long
reply other threads:[~2016-01-07 21:28 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160107212808.89233.qmail@sourceware.org \ --to=corinna@sourceware.org \ --cc=cygwin-apps-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).