Skip to content

Commit 68409fe

Browse files
authored
fix warning (#161)
* fix warning comparison of int and unsigned int * convert tabs to spaces to remove warnings -Wmisleading-indentation
1 parent 4a4456e commit 68409fe

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

src/Modbus.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,9 +123,9 @@ bool Modbus::removeReg(TAddress address, uint16_t numregs) {
123123
bool Modbus::addReg(TAddress address, uint16_t* value, uint16_t numregs) {
124124
if (0xFFFF - address.address < numregs)
125125
numregs = 0xFFFF - address.address;
126-
for (uint16_t k = 0; k < numregs; k++)
127-
addReg(address + k, value[k]);
128-
return true;
126+
for (uint16_t k = 0; k < numregs; k++)
127+
addReg(address + k, value[k]);
128+
return true;
129129
}
130130

131131
void Modbus::slavePDU(uint8_t* frame) {

src/ModbusRTU.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,8 @@ bool ModbusRTUTemplate::rawSend(uint8_t slaveId, uint8_t* frame, uint8_t len) {
8282
_port->flush();
8383
if (_txPin >= 0)
8484
digitalWrite(_txPin, _direct?LOW:HIGH);
85-
//delay(_t);
86-
return true;
85+
//delay(_t);
86+
return true;
8787
}
8888

8989
uint16_t ModbusRTUTemplate::send(uint8_t slaveId, TAddress startreg, cbTransaction cb, uint8_t unit, uint8_t* data, bool waitResponse) {

src/ModbusTCPTemplate.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ void ModbusTCPTemplate<SERVER, CLIENT>::task() {
232232
for (n = 0; n < MODBUSIP_MAX_CLIENTS; n++) {
233233
if (!tcpclient[n]) continue;
234234
if (!tcpclient[n]->connected()) continue;
235-
while (millis() - taskStart < MODBUSIP_MAX_READMS && tcpclient[n]->available() > sizeof(_MBAP)) {
235+
while (millis() - taskStart < MODBUSIP_MAX_READMS && (size_t)tcpclient[n]->available() > sizeof(_MBAP)) {
236236
tcpclient[n]->readBytes(_MBAP.raw, sizeof(_MBAP.raw)); // Get MBAP
237237

238238
if (__swap_16(_MBAP.protocolId) != 0) { // Check if MODBUSIP packet. __swap is usless there.

0 commit comments

Comments
 (0)