Re: possible code error at run_command.c

From: Frank Li <lznuaa@gmail.com>
Date: Dec 21, 2009, 08:18 UTC
Message-ID: <1976ea660912210018y15acfe32o78841d5e0968f793@mail.gmail.com>
In-Reply-To: <4B2F2727.4060405@viscovery.net>

>
> Disable this check - it just takes away performance. :-)

Release version will disable this. Debug version enable this by default is just for find out code error.

Show 10 quoted lines

>
> (If you don't disable the check, then keep the required changes private.)
>
>> I prefer change it to
>> int failed_errno = errno;
>
> You don't need to initialize the variable at all because it is always
> initialized elsewhere before it is used. Perhaps MSVC is clever enough to
> see it?
>

Maybe some excute path miss initialized it. Otherwise compiler will not report warning.