Tcl Source Code

View Ticket
Login
Ticket UUID: 1805896
Title: Die Octal Die
Type: Patch Version: TIP Implementation
Submitter: dgp Created on: 2007-10-01 19:55:06
Subsystem: 48. Number Handling Assigned To: dgp
Priority: 9 Immediate Severity:
Status: Closed Last Modified: 2007-10-25 22:52:41
Resolution: Rejected Closed By: dgp
    Closed on: 2007-10-25 15:52:41
Description:
At the TCT panel at this year's
Tcl Conference, there appeared to
be general agreement that it was
time to kill octal.

Here's a patch implementing that,
so we can all be sure we mean the
same thing.

Note that this effort exposed
Bug 1805887, which also
needs fixing.
User Comments: dgp added on 2007-10-25 22:52:41:
Logged In: YES 
user_id=80530
Originator: YES


TIP 114 is Rejected.

Closing this.  Issue
can re-open fresh for 8.6.

dgp added on 2007-10-18 23:51:44:

File Added - 250346: option1.patch

Logged In: YES 
user_id=80530
Originator: YES


Here's yet another patch,
this one implementing 
"Option 1" as described in
a message to TCLCORE. 
File Added: option1.patch

dgp added on 2007-10-17 23:48:15:

File Added - 250188: alternative.patch

Logged In: YES 
user_id=80530
Originator: YES


An alternative proposal raised
on TCLCORE is to revise TclParseNumber
to simply reject all multi-digit
integers that start with "0".  Here's
a patch that does that, so folks can
try the concept on for size.
File Added: alternative.patch

dgp added on 2007-10-16 04:29:30:
Logged In: YES 
user_id=80530
Originator: YES


Most of the patch simply makes various
parts of Tcl ([file attributes -permissions],
for example) continue to work as they have
whether or not Tcl treats all leading-0
strings as octal format in an integer context.

Committed that portion, leaving only the

#define KILL_OCTAL 

change left to commit to actually kill octal
( along with suitable doc updates)

dgp added on 2007-10-12 04:57:43:

File Added - 249287: 1805896.patch

Logged In: YES 
user_id=80530
Originator: YES

Updated patch.
File Added: 1805896.patch

dgp added on 2007-10-02 02:55:07:

File Added - 247952: tip114.patch

Attachments: