Experimental Physics and
Industrial Control System

"J. Lewis Muir" <[email protected]> · Mon, 24 Sep 2012 10:45:48 -0500

On 9/24/12 10:10 AM, Mark Rivers wrote:
> One solution would be the following in dbFastLinkConv.c.
> 
> #ifdef _WIN32
>   #define UNSIGNED_FORMAT " %u%n"
> #else 
>   #define UNSIGNED_FORMAT " %u %n"
> #endif
> 
>  nargs = sscanf(from,UNSIGNED_FORMAT,&ind,&nchars);
>  if(nargs==1 && nchars==strlen(from) && ind<nchoices) {
>    *pfield = ind;
>    return(0);
>  }
> 
> This would preserve the current behavior on all platforms except _WIN32.  It would allow _WIN32 to correctly parse all strings except those with trailing whitespace, which is much better than the current situation where it fails for all strings that do NOT have trailing whitespace.

I think it's nicer if ifdefs can be avoided.  Another approach
might be to try using the "%c" format specifier to match just
one char.  Maybe it works correctly on Windows when preceded by
a space in the format string?

Or, there's also strtoul.  This will allow leading white space,
but you'd have to check the end pointer to allow trailing white
space (since with trailing white space it would not point to a
NUL char).

Lewis

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System