[Python-checkins] r80819 - python/trunk/Python/pystrtod.c
Eric Smith
eric at trueblade.com
Wed May 5 23:12:40 CEST 2010
Actually, Clang must have been run with assert defined. Because if the
assert is compiled out, then the assignment is needed.
Eric.
Eric Smith wrote:
> I dislike this change. If for some reason we ever decide to make this
> condition not be an error (by removing the assert) then the deleted line
> is needed. What's the harm of leaving it in?
>
> If any change must be made (and I don't think it does), I'd comment out
> both the strncpy and the increment. And put in a comment explaining why
> they've been commented out.
>
> Eric.
>
> brett.cannon wrote:
>> Author: brett.cannon
>> Date: Wed May 5 22:35:24 2010
>> New Revision: 80819
>>
>> Log:
>> Remove an unnecessary variable assignment.
>>
>> Found using Clang's static analyzer.
>>
>>
>> Modified:
>> python/trunk/Python/pystrtod.c
>>
>> Modified: python/trunk/Python/pystrtod.c
>> ==============================================================================
>>
>> --- python/trunk/Python/pystrtod.c (original)
>> +++ python/trunk/Python/pystrtod.c Wed May 5 22:35:24 2010
>> @@ -1005,7 +1005,6 @@
>> /* shouldn't get here: Gay's code should always return
>> something starting with a digit, an 'I', or 'N' */
>> strncpy(p, "ERR", 3);
>> - p += 3;
>> assert(0);
>> }
>> goto exit;
>> _______________________________________________
>> Python-checkins mailing list
>> Python-checkins at python.org
>> http://mail.python.org/mailman/listinfo/python-checkins
>>
>
> _______________________________________________
> Python-checkins mailing list
> Python-checkins at python.org
> http://mail.python.org/mailman/listinfo/python-checkins
>
More information about the Python-checkins
mailing list