Explicit void cast for unused return values

This commit is contained in:
Pavel Kirienko
2014-04-19 17:45:03 +04:00
parent b2c021397d
commit f31d46ea6c
12 changed files with 26 additions and 26 deletions
+1 -1
View File
@@ -56,7 +56,7 @@ void bitarrayCopy(const unsigned char* src_org, int src_offset, int src_len, uns
if (byte_len)
{
std::memcpy(dst, src, byte_len);
(void)std::memcpy(dst, src, byte_len);
src += byte_len;
dst += byte_len;
}
+2 -2
View File
@@ -94,9 +94,9 @@ std::string BitStream::toString() const
}
out += ' ';
}
if (out.length())
if (out.length() > 0)
{
out.erase(out.length() - 1, 1);
(void)out.erase(out.length() - 1, 1);
}
return out;
}
+1 -1
View File
@@ -67,7 +67,7 @@ uint16_t IEEE754Converter::nativeNonIeeeToHalf(float value)
return hbits | 0x7C00U;
}
int exp;
std::frexp(value, &exp);
(void)std::frexp(value, &exp);
if (exp > 16)
{
return hbits | 0x7C00U;
@@ -231,7 +231,7 @@ DataTypeSignature GlobalDataTypeRegistry::computeAggregateSignature(DataTypeKind
void GlobalDataTypeRegistry::getDataTypeIDMask(DataTypeKind kind, DataTypeIDMask& mask) const
{
mask.reset();
(void)mask.reset();
const List* list = selectList(kind);
if (!list)
{
@@ -131,7 +131,7 @@ int NetworkCompatibilityChecker::checkOneNode(NodeID nid)
int NetworkCompatibilityChecker::checkNodes()
{
nid_mask_checked_.reset();
(void)nid_mask_checked_.reset();
while (true)
{
const NodeID nid = findNextUncheckedNode();
+5 -5
View File
@@ -41,7 +41,7 @@ int Frame::setPayload(const uint8_t* data, unsigned len)
return maxlen;
}
len = std::min(unsigned(maxlen), len);
std::copy(data, data + len, payload_);
(void)std::copy(data, data + len, payload_);
payload_len_ = len;
return len;
}
@@ -85,7 +85,7 @@ bool Frame::parse(const CanFrame& can_frame)
{
dst_node_id_ = NodeID::Broadcast;
payload_len_ = can_frame.dlc;
std::copy(can_frame.data, can_frame.data + can_frame.dlc, payload_);
(void)std::copy(can_frame.data, can_frame.data + can_frame.dlc, payload_);
break;
}
case TransferTypeServiceResponse:
@@ -102,7 +102,7 @@ bool Frame::parse(const CanFrame& can_frame)
}
dst_node_id_ = can_frame.data[0] & 0x7F;
payload_len_ = can_frame.dlc - 1;
std::copy(can_frame.data + 1, can_frame.data + can_frame.dlc, payload_);
(void)std::copy(can_frame.data + 1, can_frame.data + can_frame.dlc, payload_);
break;
}
default:
@@ -142,7 +142,7 @@ bool Frame::compile(CanFrame& out_can_frame) const
case TransferTypeMessageBroadcast:
{
out_can_frame.dlc = payload_len_;
std::copy(payload_, payload_ + payload_len_, out_can_frame.data);
(void)std::copy(payload_, payload_ + payload_len_, out_can_frame.data);
break;
}
case TransferTypeServiceResponse:
@@ -152,7 +152,7 @@ bool Frame::compile(CanFrame& out_can_frame) const
assert((payload_len_ + 1) <= sizeof(out_can_frame.data));
out_can_frame.data[0] = dst_node_id_.get();
out_can_frame.dlc = payload_len_ + 1;
std::copy(payload_, payload_ + payload_len_, out_can_frame.data + 1);
(void)std::copy(payload_, payload_ + payload_len_, out_can_frame.data + 1);
break;
}
default:
@@ -16,7 +16,7 @@ std::string TransferBufferManagerKey::toString() const
{
using namespace std; // For snprintf()
char buf[24];
snprintf(buf, sizeof(buf), "nid=%i tt=%i", int(node_id_.get()), int(transfer_type_));
(void)snprintf(buf, sizeof(buf), "nid=%i tt=%i", int(node_id_.get()), int(transfer_type_));
return std::string(buf);
}
#endif
@@ -239,7 +239,7 @@ int StaticTransferBufferImpl::read(unsigned offset, uint8_t* data, unsigned len)
len = max_write_pos_ - offset;
}
assert((offset + len) <= max_write_pos_);
std::copy(data_ + offset, data_ + offset + len, data);
(void)std::copy(data_ + offset, data_ + offset + len, data);
return len;
}
@@ -259,7 +259,7 @@ int StaticTransferBufferImpl::write(unsigned offset, const uint8_t* data, unsign
len = size_ - offset;
}
assert((offset + len) <= size_);
std::copy(data, data + len, data_ + offset);
(void)std::copy(data, data + len, data_ + offset);
max_write_pos_ = std::max(offset + len, unsigned(max_write_pos_));
return len;
}
@@ -46,7 +46,7 @@ int SingleFrameIncomingTransfer::read(unsigned offset, uint8_t* data, unsigned l
len = payload_len_ - offset;
}
assert((offset + len) <= payload_len_);
std::copy(payload_ + offset, payload_ + offset + len, data);
(void)std::copy(payload_ + offset, payload_ + offset + len, data);
return len;
}
@@ -49,7 +49,7 @@ int TransferSender::send(const uint8_t* payload, int payload_len, MonotonicTime
buf[0] = crc.get() & 0xFF; // Transfer CRC, little endian
buf[1] = (crc.get() >> 8) & 0xFF;
std::copy(payload, payload + BUFLEN - 2, buf + 2);
(void)std::copy(payload, payload + BUFLEN - 2, buf + 2);
const int write_res = frame.setPayload(buf, BUFLEN);
if (write_res < 2)