AVR C Runtime Library - Patches: patch #2155, optimizing strtol()/strotoul()
You are not allowed to post comments on this tracker with your current authentication level.
patch #2155: optimizing strtol()/strotoul()
Submitter: | Luca Matteini <loucypher> | ||
Submitted: | Tue 28 Oct 2003 05:01:08 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Assigned to: | arcanum | Open/Closed: | Closed |
Wed 07 Apr 2004 05:46:40 PM UTC, comment #2: |
Theodore A. Roth <troth> |
Wed 07 Apr 2004 05:27:30 PM UTC, comment #1: This needs some testing, but after a quick review it looks to make sense to me. |
Theodore A. Roth <troth> |
Tue 28 Oct 2003 05:01:08 PM UTC, original submission:
|
Luca Matteini <loucypher> |
Attached Files
file #7251: avr-libc-patch-2155-2.diff added by troth (2KiB - text/plain - Found a bug in calculation of hex 'A'-'F' chars. Original patch forgot to distribute the minus sign.)
file #7250: avr-libc-patch-2155.diff added by troth (2KiB - text/plain - Updated to current head. Original patch got a little manged by html.)
Depends on the following items: None found
Items that depend on this one: None found
CC list is empty
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 7 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2004-12-30 | arcanum | Status | None | Done | |
Open/Closed | Open | Closed | |||
2004-12-30 | arcanum | Assigned to | None | arcanum | |
2004-12-22 | arcanum | Assigned to | troth | None | |
2004-04-07 | troth | Attached File | - | Added avr-libc-patch-2155-2.diff, #3073 | |
2004-04-07 | troth | Assigned to | None | troth | |
2004-04-07 | troth | Attached File | - | Added avr-libc-patch-2155.diff, #3072 |
Powered by Savane 3.13-caa5.
Corresponding source code
Looking over this again, I think I found a bug in the orignal patch. It's very subtle in the use of '-='. I've attached a second revised patch with a fix.