Consistency checks

The global variables of a component based project are declared in as many files as there are components, and nothing in the c language ties those declarations together: a component that reads SharedValue as an unscaled uint16 and a component that writes it as an sint16 in percent with a factor of 0.5 compile perfectly well, link without a word, and produce a value that is wrong by a factor of two - or by 65536, once the sign bit travels. The disagreement is invisible to the compiler because each translation unit only ever sees its own opinion of the variable. DDD sees all of them, because every declaration passes through it before a single line of c is written, and that is what the consistency checks are for: they are the only place in the tool chain where “these two components do not mean the same thing by this name” can still be said cheaply, on a source file, with a location, instead of expensively, at link time or in the calibration lab.

The same checks run on every command that resolves a project - check, generate, list and dump - so the artefacts can never be generated from inputs that were not checked. ddd generate refuses to write anything while an error is outstanding, unless it is told to with --force:

$ ddd generate all examples/inconsistent/project.ddd.json -o build/gen -t examples/templates
...
4 errors, 1 warning
$ echo $?
1

What a check is

A check is a rule with a stable identifier and a default severity. The identifier is the kebab-case name that appears in every finding - multiple-producers, definition-mismatch, unused-output - and it is part of the tool interface rather than an implementation detail: a build script pins it in a severity override, a ci job greps for it, a code review discusses it. Identifiers therefore do not change once they have been published. A check may be added in a later release, and the wording of a message may improve, but a name a project has written into its build files keeps meaning what it meant.

The authoritative list is the one the tool prints itself, which is also how the tables further down should be read after a DDD upgrade:

$ ddd checks
file-not-found                error    a referenced file does not exist (fixed)
json-syntax                   error    a file is not valid json (fixed)
file-kind                     error    the top level key of a file names no description kind, or several (fixed)
file-extension                error    a description file is not named '*.ddd.json'
schema                        error    a file does not match the DDD contract (fixed)
include-cycle                 error    projects include each other recursively (fixed)
include-depth                 error    a project's include tree goes more than 64 levels deep (fixed)
include-empty                 error    an include pattern matches no file
plugin-not-found              error    a project names a plugin that cannot be found (fixed)
plugin-invalid                error    a plugin module does not expose a well formed PLUGIN, or two plugins claim one name (fixed)
duplicate-component           error    two different files declare the same component name
duplicate-type                error    two different files declare the same type name
duplicate-unit                error    a unit is declared more than once, in one file or across files
duplicate-section             error    a memory section is declared more than once, in one file or across files
duplicate-constant            error    a constant is declared more than once, in one file or across files
...

The (fixed) marker means the severity of that check cannot be changed; the reason is in Severity policy below.

Anatomy of a finding

Every finding is one line, in a shape an editor or a log parser can pick apart, followed by zero or more indented notes:

$ ddd check examples/inconsistent/project.ddd.json
examples/inconsistent/component_b.ddd.json#component.interface[0]: error[multiple-producers]: 'SharedValue' is written by component 'ComponentB' and by component 'ComponentA'; exactly one writer is allowed
    note: examples/inconsistent/component_a.ddd.json#component.interface[0]: also written here
...

The part before the first colon is the location: the path of the description file, relative to the working directory when it lies below it, followed by # and a dotted json pointer into the document. The pointer is what makes a finding actionable in a project of a hundred components - component.interface[0] is the first entry of the interface array, and component.interface[0].definition narrows it further to the definition object inside it. A finding about the file as a whole rather than about a place inside it carries a line and a column instead, because for a file that does not parse there is no pointer to give:

$ ddd check broken.ddd.json
broken.ddd.json:4:22: error[json-syntax]: Expecting value
1 error

Then comes severity[check], and then the message. The notes are the second half of the story: almost every disagreement is a disagreement between two places, and a finding that pointed at only one of them would send its reader hunting for the other. The note therefore names the declaration the finding was measured against - “also written here”, “reference declaration”, “declared local here” - with a location of its own.

Findings are printed errors first, then warnings, then information, and within one severity by file and by position in the file, so that fixing one finding does not shuffle the others around in the next run. The run ends with a one line summary (4 errors, 1 warning) or, when a project is entirely clean, with a statement of what was checked:

$ ddd check examples/demo/demo.ddd.json
ok: 23 variables in 4 components are consistent

All of this goes to stderr - every finding, every note and the closing summary or ok: line. Stdout is reserved for whatever the command was asked to produce: the dictionary of ddd dump, the table of ddd list, the schema of ddd schema. The two streams never carry each other’s content, which is what makes ddd dump project.ddd.json > baseline.json safe - the findings cannot end up inside the artefact.

Severity policy

Not every project is at the same point in its life. A project being migrated onto DDD starts with descriptions that are incomplete on purpose; a component being developed on its own cannot see the components that produce its inputs; a project approaching a release wants the build to fail on what was merely a warning last month. Each check therefore carries a default severity which the project can change, per run, with -W (long form --severity), repeatable:

ddd check project.ddd.json -W unused-output=info
ddd check project.ddd.json -W missing-id=error -W empty-component=ignore
ddd check project.ddd.json --strict

The four levels are:

error

The finding fails the run: the exit code becomes 1, and ddd generate writes nothing.

warning

The finding is printed and counted, but the run still succeeds. Something is suspect and a human should look at it.

info

The finding is printed and counted, and nothing else happens. Useful to keep a rule visible without letting it interrupt anybody.

ignore

The finding is not produced at all. It appears neither in the text output, nor in the json output, nor in the summary counts.

--strict reports as an error everything that would be reported as a warning. It is applied after the -W overrides, on the effective severity, and that order is what makes the two options composable: a project can turn its warnings into errors and still exempt the one check it has consciously decided to live with.

$ ddd check examples/inconsistent/project.ddd.json --strict -W unused-output=info
...
examples/inconsistent/component_a.ddd.json#component.interface[1]: info[unused-output]: 'UnusedSignal' is written by component 'ComponentA' but read by nobody
4 errors, 1 info

The policy belongs to the project rather than to the engineer who happens to be typing: whoever checks the project should get the same verdict as the build server, so the overrides are best written once into the build system - the CMake integration takes them as SEVERITY and STRICT arguments, see Build integration - instead of being remembered by hand.

A plugin check is spelled <plugin>/<check> and takes the same overrides a built-in check does - -W layout/duplicate-key=warning, --strict, ignore - the prefix being the only thing that keeps two plugins, or a plugin and a built-in check, from colliding on one identifier. Which checks a run actually registers is only known once its plugins are loaded, which happens after the command line is parsed, so an override naming a plugin check is accepted provisionally at that point and verified once the project is read; one naming a check no loaded plugin registered is then the same usage error an unknown built-in check is. It is verified whether or not the read found anything - the plugins are loaded before the rest of the project, so a typo does not wait for the first run that happens to load cleanly - and a run given a baseline as well is held to the baseline’s plugins too, which it loaded to analyse it.

Checking a component on its own

Checking a single component before it is integrated is the one case where a relaxed policy is the normal thing to do, because the components producing its inputs and consuming its outputs - and the files declaring the types, units, sections, constants and rasters it names - are by definition not part of the file:

$ ddd check examples/demo/components/controller.ddd.json
examples/demo/components/controller.ddd.json#component.interface[0]: error[missing-producer]: 'ValueA' is read by component 'Controller' but no component declares it as output
examples/demo/components/controller.ddd.json#component.interface[1]: error[missing-producer]: 'ValueB' is read by component 'Controller' but no component declares it as output
examples/demo/components/controller.ddd.json#component.interface[2]: warning[unused-output]: 'ValueE' is written by component 'Controller' but read by nobody
...
2 errors, 6 warnings

$ ddd check examples/demo/components/controller.ddd.json --standalone
ok: 14 variables in 1 component are consistent

--standalone is that policy, and the whole of it. It holds back the nine checks that need every component of a project - missing-producer, unknown-constant, unknown-extension, unknown-raster, unknown-reference, unknown-section, unknown-type, unknown-unit and unused-output - and touches nothing else, so everything DDD can decide from the file in front of it is still reported. A declaration one of the nine drops is left out of the dictionary in silence as well, because incomplete-project - the trace a dropped declaration leaves when the finding that says why is silenced - has nothing to say where the missing file is the reason: the constant, the raster or the type is declared, in a file this run was not handed. The trace stays for the other cause it has, a check the caller’s own -W relaxed, which takes a variable out of ddd list and ddd dump whether the run was given one component or a whole project. Whether a check needs the whole project is stated where the check itself is - ddd checks marks each one (project) - and that is what keeps this set, the one the language server holds back for a file no build claims (see Editor integration) and the one the per-component target of the CMake integration silences (see Build integration) from drifting apart.

The flag sets the floor rather than having the last word: an explicit -W on the same run still wins, so a supplier who does want to hear about outputs nobody reads asks for that one back with --standalone -W unused-output=warning. ddd list and ddd dump take the flag as well: a component that names its project’s vocabulary, or carries the block of a plugin only its project loads, is listed or dumped on its own under the same policy.

Naming the overrides by hand instead is the mistake the flag exists to end. Two of the nine are about the other side of the interface; the other seven are about names the component uses and the file it lives in does not carry - a type, a unit, a section, a constant, a raster, a plugin’s extensions block, an axis or an input quantity another component declares, and the declaration DDD then has to drop. A component using the vocabulary of its project meets the second group at once:

$ ddd check examples/vocabulary/pump.ddd.json -W missing-producer=ignore -W unused-output=ignore
...
6 errors

$ ddd check examples/vocabulary/pump.ddd.json --standalone
ok: 3 variables in 1 component are consistent

The eight checks whose severity is fixed

Eight checks cannot be relaxed: file-not-found, json-syntax, file-kind, schema, include-cycle, include-depth, plugin-not-found and plugin-invalid. Six of them report that a description could not be read, and a file that cannot be read has nothing further to say. Downgrading them would not make the project more permissive, it would make the rest of the run meaningless: a file that is not valid json contributes no declarations, so every variable it produces would be reported as having no producer and every name it defines would look free. The tool would bury the one real finding - “this file has a comma too many on line 4” - under a page of consequences. include-depth is the same story told about a tree rather than a file: the entry that goes too deep is not followed whatever the finding is reported as, so relaxing it would hide the absence rather than allow it. The other two report that a project names a plugin that cannot be found, or that is not well formed - a project cannot be interpreted without the plugins it names, for the same reason.

The same reasoning explains why the nine other checks reported while the files are read are relaxable. file-extension, include-empty, duplicate-component, duplicate-type, duplicate-unit, duplicate-section, duplicate-constant, duplicate-raster and unknown-extension all complain about a file tree DDD read perfectly well, it just does not like the shape of what it found: a description named foo.json instead of foo.ddd.json is fully understood; an include pattern that is legitimately empty in one variant of a project is a normal thing to allow; a second declaration of a component, a type or a unit is read and dropped, the first one standing; and a block for a plugin the project does not load is kept as written. (duplicate-declaration, duplicate-id and duplicate-event are named alike but belong to the interface checks below, which need the assembled project.)

Trying to change a fixed check is a usage error rather than a silently ignored request, as is naming a check or a severity that does not exist - the value of stable identifiers would be lost if a typo in a build script quietly disabled a rule:

$ ddd check examples/inconsistent/project.ddd.json -W schema=warning
ddd: the severity of check 'schema' cannot be changed
$ echo $?
2

$ ddd check examples/inconsistent/project.ddd.json -W typo=warning
ddd: unknown check 'typo'

$ ddd check examples/inconsistent/project.ddd.json -W unused-output=fatal
ddd: unknown severity 'fatal' for check 'unused-output', expected one of error, warning, info, ignore

Note

As long as anything reported while the files are read stands as an error - one of the eight above, or one of the nine relaxable ones beside them - the interface checks do not run at all: the project has not been assembled, so there is nothing to compare. On a project that is broken in both ways the findings therefore arrive in two waves - repair the file tree, and the interface findings appear on the next run. Relaxing whichever check fired is the other way through: -W file-extension=warning or -W duplicate-unit=warning lets the second wave arrive immediately, while a fixed check leaves repairing the tree as the only way. This is why consumer-extension, which speaks about a plugin’s block on a declaration, is never seen while unknown-extension is an error on the same block.

The checks

The tables below list the checks as ddd checks reports them, grouped by their default severity. The identifier and the default severity are the two things a project pins; the third column says what the tool is actually looking at when the check fires.

Errors

An error means the project is wrong, in the sense that generating from it would produce c code or an a2l file that does not do what the description says - or that does not compile at all.

Check

Default

Fires when

file-not-found

error (fixed)

a file named on the command line or reached through an includes pattern does not exist, is a directory, or cannot be read.

json-syntax

error (fixed)

a description file is not valid json, is nested too deeply to read, or is not valid utf-8. The finding carries the line and the column the parser stopped at.

file-kind

error (fixed)

the top level of the document is not a json object, or names none of the description kinds (project, component, types, units, sections, constants, rasters), or several of them at once. A dumped data dictionary handed to a command that reads descriptions is recognised as one and named as one, rather than counted as a description stating three kinds at once.

file-extension

error

a description file is not named *.ddd.json. Relaxable with -W file-extension=warning while a project is being migrated.

schema

error (fixed)

the document does not match the DDD contract: a missing or unknown key, a value of the wrong type, a datatype that does not exist, a shape or a nesting beyond what DDD carries. One finding per violated constraint, each pointing at the offending key. It also fires on an archived dictionary written by a newer DDD than the one reading it. It is also what refuses a string conversion on anything but a one dimensional byte array, or beside a unit, limits or a display format.

include-cycle

error (fixed)

a project includes a file which, directly or indirectly, includes it again. A diamond - the same component reached through two paths - is not a cycle and is quietly used once.

include-depth

error (fixed)

the include tree of a project goes more than 64 levels deep, counting the root as the first and a file of any kind as a level. One finding, at the entry that crosses the limit; that file and everything under it is left out, and the rest of the project is read as usual.

include-empty

error

an includes pattern matches no file, which usually means a renamed directory or a typo rather than a component that genuinely has no description.

duplicate-component

error

two different files declare a component of the same name. Component names have to be unique: each component gets a generated header named after it.

duplicate-type

error

two different files declare a type of the same name. Which of two answers the generated c would get is not something an include order should decide, so the second is refused rather than allowed to win. One file declaring a name twice is the other half of the same rule and is not this check: a types file is validated as a whole before anything looks at what other files declare, so the repeat is a schema error located at that file, where duplicate-unit, duplicate-section, duplicate-constant and duplicate-raster cover both cases and point at the entry.

unknown-type

error

a typename names no type any file of the project declares - on a component declaration or on a structure member alike. It is refused rather than skipped, because an object of unknown storage has no limits, no size and nothing later to check against. The nearest known name is suggested, which is what answers a transposition like unit16.

duplicate-unit

error

a unit is declared more than once, in one file or across files. The vocabulary is a set of spellings, so a second entry is either a copy that will drift or a disagreement about the description, and neither is worth keeping quiet.

duplicate-section

error

a memory section is declared more than once, in one file or across files: either a copy that will drift or a disagreement about its properties, and neither is worth keeping quiet.

duplicate-constant

error

a constant is declared more than once, in one file or across files (see the constants file): either a copy that will drift or a disagreement about the value, and a size that depends on which file loads first is exactly what the vocabulary exists to prevent.

unknown-section

error

a definition names a memory section no file declares (see the sections file). Unlike a unit there is no free text fallback: a section without declared properties is a name the placement checks can say nothing about. The nearest declared name is suggested.

section-access

error

a measurement, which the software writes, is placed in a read-only section. A calibration object may live in either direction: const data in RAM is a mirrored calibration.

raster-kind

error

a raster is stated on a calibration object (see the rasters file). A calibration object becomes a CHARACTERISTIC rather than a MEASUREMENT in the generated a2l, and no DAQ list - the schedule a raster names - carries one.

unknown-constant

error

a shape - a dimensions entry, or the size of an axis - names a constant no file of the project declares (see the constants file). Like a section there is no free text fallback, because a name without a value is a dimension nothing can resolve. The nearest declared name is suggested.

dimension-value

error

a shape names a constant whose value is no array length: zero, negative, or not a whole number (see the constants file). A constant holds any number, because most of what a project names is emitted and dimensions nothing; the rule a literal dimension obeys applies only where a shape names one, and is reported there. The declaration is dropped, as for an unknown constant. At a structure member it is the type that goes: a member of no known length leaves the structure without a size, so the type is unusable and every declaration naming it is dropped with it.

unknown-raster

error

a definition or a component names a measurement raster no file declares (see the rasters file). Like a section and unlike a unit there is no free text fallback: an event nothing describes is a name the a2l could only write as a number nobody chose. The nearest declared name is suggested.

duplicate-raster

error

a measurement raster is declared more than once, in one file or across files - the rule a unit, a section or a constant obey, since a raster is vocabulary like them.

duplicate-id

error

two data objects of one project carry the same id. An identity relates an object to its predecessor in a delivery comparison, so one shared by two objects can relate neither, and a declaration copied with its id still in it is refused rather than paired wrongly.

duplicate-event

error

two measurement rasters claim the same event channel number (see the rasters file), which would put two rasters on one event.

plugin-not-found

error (fixed)

a project names a plugin, in plugins, that cannot be found at the path or as the module it names. A project cannot be interpreted without the plugins it names, so there is nothing further to say.

plugin-invalid

error (fixed)

a plugin module raises while it is imported, exposes no PLUGIN, exposes one that is not a well formed ddd.plugins.Plugin, or claims a name another loaded plugin already has. Fixed for the same reason as plugin-not-found.

unknown-extension

error

an extensions block, on the project or on a definition, names a plugin the project does not load. A block only means something to the plugin that owns it, so a misspelled or removed name is refused rather than kept as dead configuration; relaxing the check is how a project deliberately keeps a block that no installed plugin currently reads.

unknown-unit

error

a unit is not in the vocabulary the project declares (see the units file). Without a units file, units stay free text and the check never fires; with one - even one declaring nothing - every stated unit is checked where it is written, with the nearest declared spelling suggested. The empty unit is always allowed - a dimensionless value states no unit rather than a spelling of one.

type-kind

error

a declared type is used where its shape does not fit: a declaration naming a structure while being a curve, a map, an axis or a value block, all of which refer to other objects or are arrays of one datatype, or one stating an init, which for a structure is written by the code that starts it rather than by the description. A declaration naming an external type is refused the same way: DDD knows neither its layout nor its meaning, so only a structure member may name one.

type-cycle

error

structures nest each other, directly or through others. The finding names the chain, A -> B -> C -> A, because that says which member to remove; a structure that contains itself has no size at all.

reserved-identifier

error

a project, component, variable, type, structure member, enum, enumerator or constant name is a c keyword, or is declared by one of the headers the generated code includes (everything <stdint.h> and <stdbool.h> bring in, so uint16_t is out), or is reserved for the implementation by C11 7.1.3 - any name containing a double underscore, or starting with an underscore followed by a capital letter.

name-collision

error

two names that are distinct in the description files would become the same c identifier or the same generated file: an enumerator and a variable, a variable and the name of an enum or of a declared type (both of which the types header makes typedef names, in the same file scope namespace as the variables), an enumerator claimed by two enums (all enumerators share one c namespace), a declared constant and a variable, an enum, an enumerator or a declared type (the constant reaches the generated code as an identifier of its own, in that same namespace), a declared constant and the member of a structure (a member has a namespace of its own, but the constant is a preprocessor definition in the same types header and replaces the member’s name where the structure declares it), or two component names differing only in case, which ask for the same header on a case insensitive filesystem.

duplicate-declaration

error

one component declares the same variable twice, for instance once as input and once as output. The second declaration is ignored for the rest of the run.

consumer-storage

error

an input declaration states init or section. What a variable starts out as, and which memory it lives in, is decided by the component that produces it, so a component that only reads it is claiming storage it does not own - which is a different thing from the disagreements above, and is reported where the claim is written rather than where it is overruled. Relaxable, so a project migrating existing descriptions can lower it while the init and section keys are removed.

consumer-raster

error

an input declaration states a measurement raster (see the rasters file). Which event updates a variable is decided by the component that produces it, exactly as init and section are, so a reader stating one is claiming an authority it does not have. Kept a separate identifier from consumer-storage, whose published description says storage, which a raster is not.

consumer-identity

error

an input declaration states an id. Which earlier delivery a variable continues is decided by the component that produces it, exactly as init and section are, so a reader stating one is claiming an identity it does not have. Kept a separate identifier from consumer-storage, whose published description says storage, which an identity is not.

consumer-extension

error

an input declaration states an extensions block. What a plugin knows about a variable is decided by the component that produces it, exactly as init, section and id are, so a reader stating a block is claiming something it does not own. Kept a separate identifier from consumer-storage, whose published description says storage, which a block is not.

multiple-producers

error

a variable is declared output by more than one component. Exactly one component owns the storage of a variable; two definitions would not even link.

missing-producer

error

a variable is read as input but no component declares it as an output. The generated consumer header would declare a symbol nobody defines. A producer whose own declaration could not resolve still counts as a producer, so a consumer of the same variable is not told that nobody produces it.

local-conflict

error

a variable declared local by one component is also declared by another. local is a promise that the variable is nobody else’s business, and DDD keeps it out of the shared headers accordingly. A reference is a use of it as much as a declaration is, so a curve, map or axis of another component that names it as its axis, x_axis, y_axis or input is the same conflict, reported at the reference, and the object is not dropped.

definition-mismatch

error

two components describe the same variable differently in a property that changes what the value means: kind, datatype, unit, declared shape, conversion, volatility, physical limits, or the axis references of a curve or a map. A consumer that simply omits limits is not disagreeing and is not reported, because they are derived from the datatype and the conversion when nobody states them. volatile cannot be left out that way: every definition of every kind has to state it, so a difference there is always two components saying two different things rather than one of them saying nothing at all. An enum conversion is compared here by its name alone; the enumerators behind it are enum-conflict’s, immediately below, so a reordered or revalued enumerator is that one finding rather than two.

enum-conflict

error

the same enum type name is defined with different enumerators. One c enum is generated per name, so only one set of enumerators could survive.

unknown-reference

error

a curve, a map or an axis refers by name to an object no component declares. The referring object is dropped as unresolvable whatever severity this check is given - a curve without its axis has no shape, and an axis naming an absent measurement would leave a dangling name in the a2l - and incomplete-project reports the absence when the finding is silenced.

reference-kind

error

a reference points at an object of the wrong kind, for example the axis of a curve naming a measurement instead of an axis. A structured object is the wrong kind for the input of an axis as well, although an instance of a structure is a measurement: it reaches the a2l as one record per value-holding member and none of its own, so an axis indexed by it would carry a name the file does not declare. The referring object is dropped as unresolvable too, exactly as unknown-reference drops one.

init-invalid

error

an initial value does not fit its datatype - out of range, written as a fraction for an integer, neither 0 nor 1 for a bool, or a magnitude a floating point datatype rounds to zero, such as 1e-50 on a float32 - or an initialiser has a shape the variable does not have (for a curve or a map, the shape given by its axes), or an enumerator does not fit the datatype of the variable, or does not fit into a c int, which every enumerator has to (C11 6.7.2.2), or a string init that is not printable ASCII, leaves no room for its terminator, or sits on an object that is not a string. The values of one initialiser that are wrong in the same way are one finding, which names how many there are and what else is among them: a table typed one datatype too narrow is one mistake, not one per element.

point-counts-unrepresentable

error

a table stores its point counts in a datatype that cannot hold them: a boolean table, or an integer one whose range stops short of a count. Only a counted table is checked; a float holds any count.

Warnings

A warning means the project generates, and what it generates is well defined, but something in it is either a smell or a decision somebody should have taken consciously.

Check

Default

Fires when

storage-mismatch

warning

two components describe the same variable differently in a property that shapes how the a2l presents it - a format string, a display_identifier - rather than the meaning of the value. This is a warning and not an error because the outcome is defined: the producer’s value is the one that is generated, and the message says so.

condition-mismatch

warning

the declarations of one variable carry different preprocessor conditions, so the components do not agree on when the variable exists at all. The producer’s condition is the one that guards the generated definition.

unused-output

warning

a variable is declared output by its producer and read by no component. Either a consumer forgot to declare it, or the variable exists for measurement only, in which case local says so more honestly. A consumer whose own declaration could not resolve still counts as a reader, so the producer is not told that nobody reads it.

enum-duplicate-value

warning

two enumerators of one enum share the same numeric value. Legal c, and occasionally intended as an alias, but a calibration tool can no longer display the raw value unambiguously.

limits-out-of-range

warning

the declared physical limits are wider than what the datatype can represent through the conversion, so part of the declared range is unreachable.

section-alignment

warning

an object needs stricter alignment than its section guarantees. A warning rather than an error because for a structure the need is an estimate - the strictest of its members’ datatypes - and the compiler’s word on the real layout is final.

name-similar

warning

two variables differ only in upper and lower case. They are distinct in c, and confusing everywhere else.

a2l-unrepresentable

warning

an object the a2l carries cannot be fully described by the version DDD writes: today that is an array of more than three dimensions, which the MATRIX_DIM of ASAP2 1.6.1 cannot carry. It fires for what the file carries rather than for what asked to be exported, the closure over references included, so an object kept out of the a2l on its own is still reported where an exported axis names it as its input. The extra dimensions are written out and only a 1.7 reader understands them.

point-counts-mismatch

warning

a curve or a map stores its point counts one way and one of its axes the other. The a2l describes each as resolved; the interpolation routine is unlikely to read both.

address-missing

warning

an object that reaches the a2l has no entry in the map --address-map was given, so its ECU_ADDRESS stays 0. A map produced from a linker output legitimately lacks the objects that were not linked into this image, which is why the run goes on; a post-link build that wants an a2l with no hole in it runs with --strict, which makes this an error and writes nothing.

Information

Check

Default

Fires when

empty-component

info

a component declares no variable at all. Perfectly legal - a component may have no global interface - but worth saying out loud, because the usual cause is a description file that was registered before it was written.

empty-vocabulary

info

a types, units, constants, sections or rasters file declares nothing: its own list is empty, and the finding is drawn there. Legal as well, and an information rather than a warning because emptying a file is how a project goes from one entry of a vocabulary to none - removing the last one in ddd gui leaves exactly that - so it is said rather than held against the project. A units file declaring nothing still opts the project into unknown-unit, so every stated unit no other units file declares is reported.

incomplete-project

info

a declaration is missing from the dictionary and the finding that says why has been silenced - a type-kind relaxed to ignore, for instance. The declaration is left out rather than generated wrong, and this is the trace that leaves, so that a dictionary short of a variable is never a surprise. It fires once for every declaration the dictionary omits this way, not only the one that was dropped: a variable of a poisoned type, the consumers of a dropped producer, a curve or an axis whose own reference names nothing or names the wrong kind, and a curve or an axis referring to one that went are each reported in turn.

missing-id

info

a declaration that produces a variable states no id, so a delivery that renames the variable is compared as a removal and an unrelated addition. ddd id --assign writes the identities; a project that has adopted them raises the check to error, so that a new declaration cannot go out without one.

Checks of the delivery comparison

The remaining checks of the registry answer a different question - whether one delivery can replace another - and fire only for ddd compare and for ddd check --baseline. They obey exactly the same policy: same identifiers, same -W and --strict, same exit codes. Comparing deliveries explains the grading; the identifiers are listed here so that the registry can be read in one place.

Check

Default

Fires when

removed-object

error

an object of the baseline is gone and a component read it.

changed-interface

error

the kind, datatype, unit, scaling, shape, axes or locality of an object changed, or where a table keeps its point counts, or the layout a released structure fixed for its consumers: the width of a bitfield, the order of the members, or the type a structured variable names. Scaling compares an enum by its name and its ordered enumerators, whose descriptions are documentation and are not compared.

reused-name

error

a name of the baseline now belongs to a different object: the object that carried it was renamed or removed, and a new one took the spelling. Nothing about the delivery looks broken, and a dataset or a recording keyed by the name binds to the wrong storage.

renamed-object

warning

an object of the baseline is offered under a different name; its id is what says so. The interface comparison still runs across the pair, and --renames writes the old and the new spelling out, for migrating datasets and recordings.

removed-unused-object

warning

an object of the baseline is gone that no component read - a calibration dataset or an external tool still might.

changed-storage

warning

the initial value, the volatility, the memory section or the measurement raster of an object changed. A calibration object whose volatile went from true to false is the case worth reading twice: it keeps its address and a tool can still write to it, but the compiler is now entitled to fold the initial value into the code that reads it, so tuning it while the software runs stops working. The section says literally which memory the object ends up in, and the raster which DAQ event a calibration tool measures it on. The initial value compares as the bytes it produces, so respelling one without moving a byte is not a change; on a structured variable the three storage properties are the variable’s own and are reported at it, once, rather than under every member.

narrowed-limits

warning

the physical limits of an object got tighter, so calibrated data may no longer fit. Widening is silent, because every value the baseline allowed still fits. The ends are compared with the relative tolerance of limits-out-of-range: most limits are derived, and a candidate that writes down the ones its datatype implies is not narrowing the unrounded ends an older baseline carries.

changed-owner

warning

another component produces the object now.

changed-condition

warning

the preprocessor condition of an object changed.

changed-a2l

warning

the a2l entry of an object changed.

project-mismatch

warning

the two sides of the comparison describe differently named projects, so the baseline is probably not the predecessor of this candidate.

missing-plugin

warning

a compared dictionary records a plugin that is not among the candidate’s, so that plugin’s comparison rules did not run. Once per plugin and side, reported at the file that records it, because a comparison that silently skipped a rule would be a confident verdict with a hole in it.

added-object

info

the candidate declares an object the baseline did not.

The producing component wins

Two components can describe the same variable, and one of the two descriptions has to be the one that is generated. DDD does not average them and does not simply take the first one it read: the declaration of the producing component is the reference. The rule is not arbitrary. The producer is the component that owns the storage - its declaration is where the definition is emitted, its initial value is the one the linker puts into the image, its condition is the one that decides whether the variable exists at all. A consumer only ever describes what it expects to find, and when expectation and reality differ, reality is what the hardware will do.

Two things follow. The first is that the diagnostic points at the deviating consumer, with a note referring back to the producer, so that the finding lands on the file that has to change:

examples/inconsistent/component_c.ddd.json#component.interface[0].definition: error[definition-mismatch]: 'SharedValue' is declared differently by component 'ComponentC' than by 'ComponentA' (datatype: uint16 != sint16, conversion: identity != linear(factor=0.5, offset=0))
    note: examples/inconsistent/component_a.ddd.json#component.interface[0].definition: reference declaration

The second is that a property which merely shapes how the a2l presents the object does not have to stop the build, precisely because the outcome is defined. Given a producer and a consumer that ask for different display formats,

{ "scope": "output", "definition": { "name": "ValueA", "kind": "measurement", "datatype": "uint16", "conversion": {}, "a2l": { "format": "%5.0" }, "volatile": false } }
{ "scope": "input", "definition": { "name": "ValueA", "kind": "measurement", "datatype": "uint16", "conversion": {}, "a2l": { "format": "%8.3" }, "volatile": false } }

the check names the value that is going to be used:

consumer.ddd.json#component.interface[0].definition: warning[storage-mismatch]: 'ValueA': component 'Consumer' specifies a different a2l format than 'Producer' (a2l format: '%8.3' != '%5.0'); the value of 'Producer' is used
    note: producer.ddd.json#component.interface[0].definition: reference declaration

A consumer that merely leaves the a2l block or one of its keys out is not asking for anything and is not reported; only two stated values can disagree.

Two other properties used to be settled this way, and left in opposite directions.

init is not a losing opinion but a claim over storage the component does not own: reading a variable gives it no say in what the variable starts as. A consumer stating one is refused outright, as consumer-storage, where the claim is written rather than where it is overruled.

volatile went the other way and became interface. It reaches every consumer’s own header as a type qualifier - extern volatile uint16_t ValueA - and that is what tells that component’s code not to cache the value and not to expect two reads to agree. Every declaration of the variable therefore has to say the same thing, and a disagreement is a definition-mismatch error. There is no leaving it out, either: unlike limits, which DDD derives when a declaration omits them, there is nothing here to derive, so the key is required on every definition of every kind and a definition without it does not load at all. That is reported as schema, one of the eight checks whose severity cannot be relaxed, which is why a project adopting this version of DDD adds the key everywhere in one go rather than phasing it in.

export is not compared at all. Which signals a calibration engineer needs to see is not the producer’s to decide alone, so any component may ask for an object to reach the a2l and asking wins over declining - see the component file format.

Note

When no component produces a variable - which is reported on its own as missing-producer - the first declaration the tool read stands in as the reference, so that the remaining checks still have something to compare the others against. This is why a project with a missing producer reports both the missing producer and any disagreement between its consumers, instead of hiding the second finding behind the first.

A project that does not check out

examples/inconsistent is a deliberately broken project of three components, kept in the repository so that the checks can be seen firing on something small. The project file only collects them:

{
  "project": {
    "name": "BrokenProject",
    "description": "Deliberately inconsistent project, used to demonstrate the checks",
    "includes": [
      "component_a.ddd.json",
      "component_b.ddd.json",
      "component_c.ddd.json"
    ]
  }
}

ComponentA writes SharedValue as an sint16 in percent with a factor of 0.5, writes UnusedSignal, and keeps a local variable Scratch. ComponentB writes SharedValue as well. ComponentC reads SharedValue as an unscaled uint16, reads a MissingValue nobody writes, and reads ComponentA’s Scratch. Checking the project gives:

$ ddd check examples/inconsistent/project.ddd.json
examples/inconsistent/component_b.ddd.json#component.interface[0]: error[multiple-producers]: 'SharedValue' is written by component 'ComponentB' and by component 'ComponentA'; exactly one writer is allowed
    note: examples/inconsistent/component_a.ddd.json#component.interface[0]: also written here
examples/inconsistent/component_c.ddd.json#component.interface[0].definition: error[definition-mismatch]: 'SharedValue' is declared differently by component 'ComponentC' than by 'ComponentA' (datatype: uint16 != sint16, conversion: identity != linear(factor=0.5, offset=0))
    note: examples/inconsistent/component_a.ddd.json#component.interface[0].definition: reference declaration
examples/inconsistent/component_c.ddd.json#component.interface[1]: error[missing-producer]: 'MissingValue' is read by component 'ComponentC' but no component declares it as output
examples/inconsistent/component_c.ddd.json#component.interface[2]: error[local-conflict]: 'Scratch' is local to component 'ComponentA' but is also declared as input by component 'ComponentC'
    note: examples/inconsistent/component_a.ddd.json#component.interface[2]: declared local here
examples/inconsistent/component_a.ddd.json#component.interface[1]: warning[unused-output]: 'UnusedSignal' is written by component 'ComponentA' but read by nobody
4 errors, 1 warning

Five findings, and each one is a different kind of mistake.

multiple-producers. Two components claim to write SharedValue. In c that is two definitions of one symbol, which the linker rejects outright or, under a permissive setting, silently merges; in the project it means that nobody is responsible for the value. The finding lands on ComponentB because ComponentA was seen first, and the note points at the other writer. The fix is an organisational decision rather than a textual one: exactly one component owns the value and the other declares it as input - which is also what makes the data flow of the project visible in the first place.

definition-mismatch. ComponentC reads the same name as an unscaled uint16. Both halves of the disagreement matter: the datatype changes how many bytes are read and how the sign bit is interpreted, and the missing conversion means the consumer would understand a raw 200 as 200 percent where the producer meant 100. The finding points at the consumer and the note at the producer, because the producer’s declaration is the reference. The fix is to copy the producer’s datatype and conversion into the consumer - or, if the consumer is right and the producer is wrong, to change the producer and let the check point at the next component that has not followed.

missing-producer. ComponentC reads MissingValue and no component writes it. The generated consumer header would declare a symbol that has no definition anywhere, and the mistake would surface at link time as an undefined reference, several minutes and one full build later. Either a component declares it as output, or the read is stale and the declaration goes.

local-conflict. Scratch is local to ComponentA, which is a promise that no other component depends on it: DDD keeps it out of the shared headers and out of the interface of the project, so ComponentA may rename or delete it freely. ComponentC reading it breaks that promise. Referring to it breaks the promise just as surely: a curve of ComponentC interpolated over an axis ComponentA declared local would reach the a2l bound to that private axis, so a reference to another component’s local variable - as an axis, an x_axis, a y_axis or the input of an axis - is the same finding, reported where the reference is written rather than at a declaration. Either the variable is genuinely shared, in which case ComponentA declares it as output, or it is not, in which case ComponentC has to do without.

unused-output. UnusedSignal is written and read by nobody. Only a warning, because a value may legitimately exist for measurement alone, and because the consumer may be a component that is not integrated yet. If it is measurement only, declaring it local says so and removes the warning; if a consumer was forgotten, this is the reminder.

With all five addressed - ComponentB reading SharedValue instead of writing it, ComponentC adopting the producer’s datatype and conversion, dropping its read of Scratch and reading UnusedSignal instead of the MissingValue nobody produces - the same command says:

$ ddd check project.ddd.json
ok: 3 variables in 3 components are consistent

Exit codes and ci

DDD is meant to be a build step, so the verdict is in the exit code and not only in the text:

Code

Meaning

0

The run succeeded. There may still have been warnings or information findings - they are printed and counted, but they do not fail the run. --strict is how a project changes that.

1

At least one finding was reported with the severity error, whatever its default severity was. ddd generate additionally wrote nothing, unless --force was given.

2

The command was used wrongly: an unknown severity, or an unknown check that names no plugin, in -W, an attempt to change one of the fixed checks, a missing or unrecognized argument, an output directory that cannot be written to. The command line itself could not be used - nothing was checked, and nothing but the error is printed; or a plugin, a file or an output could not be used after the analysis ran, or an override naming a plugin check no loaded plugin registers, in which case the findings gathered before the failure are still printed. This case stays distinguishable from “checked, and found wanting” by its exit code alone, not by what it printed.

A ci job therefore needs no output parsing at all to decide whether it passes:

ddd check project.ddd.json --strict || exit 1

--format json is for the other half of the job - annotating a merge request, feeding a dashboard, turning findings into editor markers. It replaces the human readable rendering rather than accompanying it: the findings become one document on stdout and stderr stays empty, so a job can consume the whole of stdout without having to recognise where the document begins:

$ ddd check project.ddd.json --format json
{
  "diagnostics": [
    {
      "check": "multiple-producers",
      "severity": "error",
      "message": "'SharedValue' is written by component 'ComponentB' and by component 'ComponentA'; exactly one writer is allowed",
      "location": {
        "path": "C:/work/project/component_b.ddd.json",
        "pointer": "component.interface[0]",
        "line": null,
        "column": null
      },
      "notes": [
        {
          "message": "also written here",
          "location": {
            "path": "C:/work/project/component_a.ddd.json",
            "pointer": "component.interface[0]",
            "line": null,
            "column": null
          }
        }
      ]
    },
    ...
  ],
  "summary": {
    "error": 4,
    "warning": 1,
    "info": 0
  }
}

Every finding carries the information the text form carries, in fields that are stable: check is the identifier, severity is the severity the finding was reported with once the policy had been applied, and location is an absolute, forward-slashed path together with the json pointer and, where the finding has one, a line and a column. The notes of a finding have the same shape as the finding itself. The summary object closes the document with the counts a dashboard usually wants; a check set to ignore contributes to none of them. The exit code is unaffected by --format json, so a job can consume the document and still let the code decide the verdict.

Finally, the registry itself is machine readable, which is how a build script can confirm that the identifiers it pins still exist after a DDD upgrade, instead of discovering it through a usage error:

$ ddd checks --format json
[
  {
    "check": "file-not-found",
    "default_severity": "error",
    "description": "a referenced file does not exist",
    "overridable": false,
    "needs_every_component": false,
    "comparison": false
  },
  ...
]

overridable is the machine readable form of the (fixed) marker: false for the eight checks whose severity cannot be changed - the six of load time and the two plugin checks - true for every other check. needs_every_component is the machine readable form of the (project) marker: true for the checks needing every component of a project, false for the rest. comparison is the machine readable form of the (comparison) marker: true for the checks of a delivery comparison (Comparing deliveries), false for every other check.