321
96%

Ran 27 Aug 2020 08:05PM UTC

Jobs 6

Files 15

Run time 51s

Badge

Embed ▾

pending completion

Build # 321

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

847 of 884 relevant lines covered (95.81%)

5.75 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	321.1	27 Aug 2020 08:06PM UTC	96.07	Travis Job 321.1
2	321.2	27 Aug 2020 08:06PM UTC	95.82	Travis Job 321.2
3	321.3	27 Aug 2020 08:06PM UTC	95.82	Travis Job 321.3
4	321.4	27 Aug 2020 08:06PM UTC	95.82	Travis Job 321.4
5	321.5	27 Aug 2020 08:06PM UTC	95.82	Travis Job 321.5
6	321.6	27 Aug 2020 08:06PM UTC	96.05	Travis Job 321.6

AdvancedClimateSystems / uModbus / 321
96%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 321

AdvancedClimateSystems / uModbus / 321 96%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 321

AdvancedClimateSystems / uModbus / 321
96%

README BADGES
x