From dcf1b50dfe8c66f7a5579856d86e328e02ff986d Mon Sep 17 00:00:00 2001 From: Uwe Hermann Date: Mon, 1 Jul 2019 00:27:17 +0200 Subject: [PATCH] modbus: Make the 'framegap' option an integer. --- decoders/modbus/pd.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/decoders/modbus/pd.py b/decoders/modbus/pd.py index d5cb811..cc42721 100644 --- a/decoders/modbus/pd.py +++ b/decoders/modbus/pd.py @@ -848,7 +848,7 @@ class Decoder(srd.Decoder): 'default': rxtx_channels[0], 'values': rxtx_channels}, {'id': 'cschannel', 'desc': 'Client -> server channel', 'default': rxtx_channels[1], 'values': rxtx_channels}, - {'id': 'framegap', 'desc': 'Inter-frame bit gap', 'default': '28'}, + {'id': 'framegap', 'desc': 'Inter-frame bit gap', 'default': 28}, ) def __init__(self): @@ -912,7 +912,7 @@ class Decoder(srd.Decoder): # somewhere between seems fine. # A character is 11 bits long, so (3.5 + 1.5)/2 * 11 ~= 28 # TODO: Display error for too short or too long. - if (ss - ADU.last_read) <= self.bitlength * int(self.options['framegap']): + if (ss - ADU.last_read) <= self.bitlength * self.options['framegap']: ADU.add_data(ss, es, data) else: # It's been too long since the last part of the ADU! -- 2.30.2