fix 2 compiler warnings #1171

pull Diapolo wants to merge 2 commits into bitcoin:master from Diapolo:fix_compiler_warn changing 2 files +5 −1

Diapolo commented at 11:55 PM on April 30, 2012: none

No description provided.
jgarzik commented at 12:03 AM on May 1, 2012: contributor

Please add the specific warning being fixed to the commit messages. "fix compiler warning" is vague, and will get even more vague over time. Code changes look OK
sipa commented at 2:02 AM on May 1, 2012: member

ACK
jgarzik commented at 2:56 AM on May 1, 2012: contributor

Sorry, to be more specific, please redo the commits to include the warning being fixed inside the git commit itself. Need to get that useful info into the permanent git history.
fix compiler warning "suggest explicit braces to avoid ambiguous "else"
[-Wparentheses]" in net.cpp
f8e4d43be7
fix compiler warning "suggest parentheses around assignment used as truth
value [-Wparentheses]" in util.cpp
1c94f88d5e
Diapolo commented at 9:44 AM on May 1, 2012: none

Renamed the commit-messages to be more detailed.
jgarzik commented at 3:02 PM on May 1, 2012: contributor

ACK
jgarzik referenced this in commit ca97bde8ff on May 1, 2012
jgarzik merged this on May 1, 2012
jgarzik closed this on May 1, 2012
coblee referenced this in commit 225ebff05c on Jul 17, 2012
suprnurd referenced this in commit e3e5b81a8d on Dec 5, 2017
lateminer referenced this in commit 19502c59d0 on Jan 22, 2019
lateminer referenced this in commit cc072687db on May 6, 2020
DrahtBot locked this on Sep 8, 2021

Contributors