Experimental Physics and
Industrial Control System

On Thu, 4 Feb 2021 at 17:31, Johnson, Andrew N. <anj at anl.gov> wrote:

On Feb 4, 2021, at 3:29 AM, Ralph Lange via Core-talk <core-talk at aps.anl.gov> wrote:

In the good old times,

we used to have a static DB parser that was used to provide a "lint" tool for databases. (My mourning of the loss of that feature was largely ignored.)

We should provide linting service on the EPICS web page. Would generate nice data of what EPICS features people are using. (And their most common errors.)

"Fixing" bugs on the fly? Rather not. Large blocks of explanations and suggestions on the IOC console? Rather not.

I assume you are against my original suggestion then, since that would be fixing bugs on the fly.

Correct.

Better error messages are always ... better.

But I would avoid lengthy things like listing all available options. That is more useful in an autocomplete pop-up of the editor, not in an IOC log file.

In general I agree with you on that, although some users and sites like ESS & PSI might appreciate a “verbose errors” flag in the IOC.

The existing error message is output by dbRecordField() in dbLexRoutines.c which only knows the error status value, and displays the related text from the header file that defines that status. How it gets the status S_stdlib_extraneous instead of S_db_badChoice is somewhat confusing though: dbRecordField() calls dbPutString() which calls dbPutStringNum() [both in dbStaticRun.c] and that first uses dbGetMenuIndexFromString()  [from dbStaticLib.c] to compare the choice strings, then if that fails it tries epicsParseUInt16() to see if it’s numeric instead. That’s where the S_stdlib_extraneous status comes from – I was expecting it to be S_db_badChoice since both dbConvert.c and dbFastLinkConvert.c return that. Thus we have 3 places where we look up a choice string in a menu definition: dbStaticLib.c, dbFastLinkConv.c and dbConvert.c, although the last one is probably never used in practice since it would only be called for an array of DBF_MENU fields.

- Andrew