Question

I have a function where I’m passing in an iterator to a char * buffer (which is also a char *). The function needs to increment the iterator. Anyway, I found that a good method of passsing the iterator into the function is by passing the pointer by reference:

bool myFunction(unsigned char * &iter)
{
   ...

However, I’ve heard that this is bad form and could cause problems. Here is the method that my coworker suggested I use:

typedef unsigned char * unsignedcharptr;
bool myFunction(unsignedcharptr &iter)
{
   ...

It looks to me like they’re both doing the same thing. Is there a compiler difference between these two methods (I’m using Visual Studio 2005)? Which is correct?

Answer 1

I don’t think there’s any difference between the two. You should ask your coworker why he believes there is.

What might be the cause is for maintainability, where if you wanted to switch from unsigned char * to char * you’d only have to change one place. But this, I think, could lead to other problems.