That's another good catch. This:Originally Posted by stahta01
is equivalent to:Code:*Blanks++;
since the dereference has no net effect, after optimisation the above is thus equivalent to:Code:*(Blanks++);
which merely increments the pointer rather than the actual count that the pointer points to.Code:Blanks++;
Look up a C++ Reference and learn How To Ask Questions The Smart Way
This is also incorrect:
fgetc() returns an int, not a char.Code:char c; c = fgetc(fp);
If char is unsigned on your platform, then c will never equal EOF. On the other hand, if char is signed on your platform, then c might equal EOF accidentally for some legitimate input. Either way, declaring c as an int will avoid both of these problems.
You're not saving anything (time or memory) by declaring c to be char (but you're spending a tiny bit more time by typing "char" instead of "int"). As I like to say, why do the wrong thing when the right thing is just as easy (or easier, in this case)?
