Re: Interesting warning I get

From:

"Victor Bazarov" <v.Abazarov@comAcast.net>

Newsgroups:

microsoft.public.vc.language

Date:

Wed, 8 Aug 2007 13:44:23 -0400

Message-ID:

<f9cvdm$grg$1@news.datemas.de>

Alex Blekhman wrote:

"Victor Bazarov" wrote:

Here is the code:
------------------------------------
#include <iostream>
int main() {
   bool foo = false;
   const char* const fooname[] = { "false", "true" };
   for (int i = 0; i < 10; ++i)
       std::cout << fooname[foo++] << std::endl; // C4709
}
------------------------------------
If I compile it with VS 2005 SP1, with /W4, I get C4709.
Well,
I don't see a comma operator inside my brackets. Do you?
Could
somebody check with 2008? Thanks!

I don't know the answer to your question, but I'm very
curious why you need to increment `bool' variable.

To me it's an alternative of writing

for (..
std::cout << fooname[i > 0] << std::endl;

(to introduce some special processing of only the first time
in the loop body). It doesn't matter here, and 'i > 0' does
actually look and read better, but if it's not an index, but,
say, an iterator, then iterator comparison is involved and
it can be expensive (more expensive than increment).

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask