<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META content="text/html; charset=iso-8859-1" http-equiv=Content-Type>
<STYLE>.hmmessage P {
PADDING-BOTTOM: 0px; MARGIN: 0px; PADDING-LEFT: 0px; PADDING-RIGHT: 0px; PADDING-TOP: 0px
}
BODY.hmmessage {
FONT-FAMILY: Verdana; FONT-SIZE: 10pt
}
</STYLE>
<META name=GENERATOR content="MSHTML 8.00.6001.18876"></HEAD>
<BODY class=hmmessage bgColor=#ffffff>
<DIV><FONT face=Arial>Thanks for the replies. I pretty much agree with both of
you. I have worked on projects where more serious warnings have
got lost in a sea of unimportant ones. It's just bad practice to let that
happen. At the same time quite a few of the compiler warnings that get
generated are just an annoyance and should be disabled. In this particular
case, personally I'm in favour of putting the parentheses
in because I don't have "operator precedence rules"
hard-wired in my brain. However I can see it the other way too (why put
parentheses in when they aren't needed), so I won't be putting them in and
submitting the changes. Instead I'll go away and figure out how to turn these
warnings off on my system.</FONT></DIV>
<DIV><FONT face=Arial></FONT> </DIV>
<DIV><FONT face=Arial>>> What compiler are you using? The code should
disable this warning for both MSVC and gcc.<BR></FONT></DIV>
<DIV><FONT face=Arial>It is gcc but it's not linux, so I'm not using any of the
normal makefiles. I'll post more about what I'm doing when I get something
working!</FONT></DIV>
<DIV><FONT face=Arial></FONT> </DIV>
<DIV><FONT face=Arial></FONT> </DIV>
<DIV><FONT face=Arial>>> Variable names like "vi," "vt," "vc" etc... help
to confuse the </FONT><FONT face=Arial>reader and obfuscate the
code</FONT></DIV>
<DIV><FONT face=Arial></FONT> </DIV>
<DIV><FONT face=Arial>I agree. It would be helpful to actually know what
these things are meant to represent!</FONT></DIV>
<DIV><FONT face=Arial></FONT> </DIV>
<DIV><FONT face=Arial></FONT> </DIV></BODY></HTML>