| Ticket UUID: | 2368084 | |||
| Title: | #343: Add %b (binary) specifier to [format/scan] | |||
| Type: | Patch | Version: | TIP Implementation | |
| Submitter: | ferrieux | Created on: | 2008-12-01 00:54:21 | |
| Subsystem: | 23. [scan] | Assigned To: | hobbs | |
| Priority: | 5 Medium | Severity: | ||
| Status: | Closed | Last Modified: | 2008-12-12 06:36:38 | |
| Resolution: | Accepted | Closed By: | ferrieux | |
| Closed on: | 2008-12-11 23:36:38 | |||
| Description: |
The attached patch adds a %b specifier meaning 'binary' to the [format] and [scan] commands. It is thus continuing in the direction pioneered by the recent addition of "0b" in [expr]'s number parsing.
The specifier of course also works with "#" (adding "0b" if nonzero) and wide- and bignum modifiers (%lb, %llb).
In addition, the patch fixes the following bug in [scan]:
% scan 0b101 %i x
1
% puts $x
0
The patch includes proper tests and documentation update. Does it really need a TIP ?
| |||
| User Comments: |
ferrieux added on 2008-12-12 06:36:38:
TIP accepted, patch committed. ferrieux added on 2008-12-09 04:21:45: The mention of "octal" in the previous comment is confusing. I meant, let's just leave %i in all its historical ugliness (which includes things related to octal). Anyway. ferrieux added on 2008-12-09 04:19:36: After discussion with Don, it appears that the [scan %i] "fix", which is orthogonal to the TIP, is a dangerous breach of compatibility. So let's leave octal in its pond for now. Patch updated to restore the aforementioned bug. Now this patch is 100% faithful to the TIP's text. File Added: bin2.patch ferrieux added on 2008-12-09 04:19:35: File Added - 304588: bin2.patch ferrieux added on 2008-12-01 07:54:21: File Added - 303554: bin.patch | |||
Home
Timeline
Branches
Tags
Tickets
Download
Wiki