yaacov / ArduinoModbusSlave

Modbus slave library for Arduino
ISC License
205 stars 75 forks source link

Mega avr support #69

Closed ysmilda closed 3 years ago

ysmilda commented 3 years ago

This fixes issues 68 and 63

yaacov commented 3 years ago

Thanks :green_heart:

Zeron commented 3 years ago

This is broken. You inverted the logic for _serialTransmissionBufferLength and now it does not build for regular AVR processors like the Uno.

ModbusSlave.h:140:41: error: 'SERIAL_BUFFER_SIZE' was not declared in this scope int _serialTransmissionBufferLength = SERIAL_BUFFER_SIZE;

yaacov commented 3 years ago

@Zeron Thanks ! can you make a pull request fixing it ?