On Sun, Nov 05, 2000 at 09:34:13PM -0800, Alexandre Julliard wrote:
> Changing variables to unsigned when they are clearly never negative is
> OK; but renaming 'i' to 'n' and adding unnecessary type casts are not
> good ideas IMO.
I basically renamed the i -> n to make sure that I have definitely looked
at all instances of the variable. In at least 1 case adding a new variable
was actually necessary. It is just an aid for me, so if you want, I can
rename them back before actually sending further patches. Just tell me.
To the second comment: I would never add "unnecessary" type casts :-)
but that's based on my very limited programming experience. If you have
something specific in mind, please let me know.
Thanks
Joerg
--
Joerg Mayer <[EMAIL PROTECTED]>
I found out that "pro" means "instead of" (as in proconsul). Now I know
what proactive means.
