Examples -> 03.Analog -> Smoothing fails on Arduino Due

[PaulStoffregen]

It seems newlib defines index() as a string manipulation function, which conflicts with the name "index" of a global variable in this example.

Perhaps newlib's string.h should be edited to comment out "index" which is likely to conflict with Arduino sketches?

[peabo7]

On April 6, 2014 at 2:17 PM Paul Stoffregen notifications@github.com wrote:

It seems newlib defines index() as a string manipulation function, which
conflicts with the name "index" of a global variable in this example.

Perhaps newlib's string.h should be edited to comment out "index" which is
likely to conflict with Arduino sketches?

I have a different opinion, which is to change the example code to avoid this
problem.

index() and rindex() are BSD Unix variants of strchr() and strrchr(). So moving
into the world of mainstream operating systems is going to expose problem like
this.

I wasn't able to reproduce the problem as you described it, so if you could
provide more information (such as exact file names in the 1.5.6-r2) that would
help me reproduce the problem you found.

Peter Olson

[cmaglie]

The manpage of index() says:

CONFORMING TO
       4.3BSD;  marked  as  LEGACY  in  POSIX.1-2001.   POSIX.1-2008  
       removes the specifications of index() and rindex(), recommending
       strchr(3) and strrchr(3) instead.

Is the use of "index" so spread to justify a change in the standard C lib? I've seen using it only on the Analog/Smoothing example.

IMHO, even if deprecated, probably the best thing to do is changing the example to use a different name for the variable.

[cmaglie]

Fixed by changing the example.