Experimental Physics and
Industrial Control System

Andrew Johnson <[email protected]> · Wed, 28 Feb 2018 12:47:21 -0600

Hi Bruno,

On 02/28/2018 10:27 AM, Bruno Martins wrote:
> I noticed that dbpf doesn't convert hex values to fields with type LONG.
> Digging through the code it seems that the culprit is in
> ioc/db/dbFastLinkConv.c: cvt_st_l (and a bunch others) calls strtol with
> base=10. Wouldn't it be better to call it with base=0 and let it figure
> out the proper base? Or, instead, to use epicsScanDouble like cvt_st_ul
> does?

The dbFastLinkConv.c routines are also used for data conversions between
records, and making this change could potentially break existing
databases that relied on the original behaviour. I admit it's an obscure
behaviour, but I am generally very reluctant to accept changes that
alter what existing IOC databases actually do.

In particular a database that gets a number from a string that has
leading zeros would then be converted as an octal number (010 → 8)
instead of the decimal that it currently uses (010 → 10).

I don't have a solution to that, discussion and ideas welcome.

- Andrew

-- 
Arguing for surveillance because you have nothing to hide is no
different than making the claim, "I don't care about freedom of
speech because I have nothing to say." -- Edward Snowdon

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System