Chris,

Juergen has pointed out the unsigned char trap. Many compilers will barf about comparing an unsigned with " < 0 ". I use uchars as counters, but the trick is to say:

for (count = number; count != 0; count--) ...

Your code used the identifier "my_array" for both global and auto instances. Of course the compiler does not care at all. It will choose the correct variable and the correct scope.

But for the sake of your sanity, it is worth using different names especially if it is an array.

With regular variables it is common to use the same name when factoring a function into sub-functions. But these parameters are sent by value, and the sub-function will not corrupt them.

David.