sendI2cCommand writes the wrong character if lenght =n*i2cTransactionSize + 1

[Valrakk]

The line

SparkFun_u-blox_GNSS_Arduino_Library/src/SparkFun_u-blox_GNSS_Arduino_Library.cpp

Line 2952 in 9f301c8

_i2cPort->write(outgoingUBX->payload, 1);

Seems to be a intended to work with *outgoingUBX->payload += len; which moves the payload pointer, but is now commented. I think it should be _i2cPort->write(outgoingUBX->payload[startSpot], 1);

[PaulZC]

Hi @Valrakk ,

Wow. Great find! That bug has been in there for years! (Back even before I started working on the library!)

I'm curious... How did you find it? Was there a message being Not-ACKnowledged?

Thanks again - I will fix this shortly,
Paul

[PaulZC]

Corrected in v2.0.14 - thanks again!

[Valrakk]

I was looking at the code to make a small driver to test a MAX-M8C module in a STM32 mcu, using i2c, and that line just caught my attention.

No problem!