diff options
| author | Steve Reinhardt <steve.reinhardt@amd.com> | 2016-02-06 17:21:20 -0800 |
|---|---|---|
| committer | Steve Reinhardt <steve.reinhardt@amd.com> | 2016-02-06 17:21:20 -0800 |
| commit | f6b828d068b046df17b462a4d05af957c038a3a8 (patch) | |
| tree | 1f7be86e41467862c9377e3ca78fa661cc270aed /src/dev/net | |
| parent | 2d91e741e8ffc8ae3d40f1e849db87e69af7bfa9 (diff) | |
| download | gem5-f6b828d068b046df17b462a4d05af957c038a3a8.tar.xz | |
style: eliminate explicit boolean comparisons
Result of running 'hg m5style --skip-all --fix-control -a' to get
rid of '== true' comparisons, plus trivial manual edits to get
rid of '== false'/'== False' comparisons.
Left a couple of explicit comparisons in where they didn't seem
unreasonable:
invalid boolean comparison in src/arch/mips/interrupts.cc:155
>> DPRINTF(Interrupt, "Interrupts OnCpuTimerINterrupt(tc) == true\n");<<
invalid boolean comparison in src/unittest/unittest.hh:110
>> "EXPECT_FALSE(" #expr ")", (expr) == false)<<
Diffstat (limited to 'src/dev/net')
| -rw-r--r-- | src/dev/net/dist_iface.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/dev/net/dist_iface.cc b/src/dev/net/dist_iface.cc index 45ce651a9..1025dffe3 100644 --- a/src/dev/net/dist_iface.cc +++ b/src/dev/net/dist_iface.cc @@ -517,8 +517,8 @@ void DistIface::RecvScheduler::unserialize(CheckpointIn &cp) { assert(descQueue.size() == 0); - assert(recvDone->scheduled() == false); - assert(ckptRestore == false); + assert(!recvDone->scheduled()); + assert(!ckptRestore); UNSERIALIZE_SCALAR(prevRecvTick); // unserialize the receive desc queue |