2
96%
master: 96%

Ran 27 Aug 2020 08:06PM UTC

Files 15

Run time 1s

Badge

Embed ▾

Committed 27 Aug 2020 08:05PM UTC coverage: 95.819% (-0.2%) from 96.068%

Job # 321.2

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Fix byte count when for WriteMultipleCoils (#105)

For full bytes, the Byte Count field was calculated too high:
    8 // 8 + 1 = 2
while only one byte is needed to represent 8 coils.

Offset the values length before the truncating division to correctly
round to whole bytes.  This is easier than what the standard
formulates:
    N = Quantity of Outputs / 8, if the
    remainder is different of 0 ==> N = N+1

Note that the actual data bytes were already handled with the correct
length.  So in case of 8 coils, uModbus would send a Byte Count of 2,
but only one data byte before the checksum.

Run Details

848 of 885 relevant lines covered (95.82%)

0.96 hits per line

AdvancedClimateSystems / uModbus / 321 / 2
96%
master: 96%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 321.2

AdvancedClimateSystems / uModbus / 321 / 2 96% master: 96%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 321.2

AdvancedClimateSystems / uModbus / 321 / 2
96%
master: 96%

README BADGES
x