]> sigrok.org Git - libsigrokdecode.git/commit
sle44xx: rephrase ss/es passing for annotation emission
authorGerhard Sittig <redacted>
Sat, 29 Aug 2020 10:31:31 +0000 (12:31 +0200)
committerGerhard Sittig <redacted>
Sun, 30 Aug 2020 05:23:58 +0000 (07:23 +0200)
commit1dfaf1e8a7df8e4cc251aeea4afb35cfa3fa219c
treec1dac13f50d5ee8b00ff2a69b4dace63f826e5f6
parente4f70391abbb153364a534d45a59026961ce1b1f
sle44xx: rephrase ss/es passing for annotation emission

Coupling the logic which interprets input signals at different levels
and the helpers which emit annotations by means of "global" variables
(public members of the decoder object) is unfortunate. It complicates
the logic, adds unnecessary dependencies, and makes maintenance rather
tedious and error prone.

Pass ss/es times to put() routines the same way as annotation classes
and annotation texts are passed. This simplifies the logic where bits
and bytes levels and additional rows are handled. The data values and
their spans all become local information that gets determined in the
same context. Which dramatically simplifies review.
decoders/sle44xx/pd.py