Unusual way of doing 'Inf' in json library

[sbermeister]

BPO	14875
Nosy	@mdickinson, @ezio-melotti

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/ezio-melotti'
closed_at = <Date 2012-05-21.23:55:13.476>
created_at = <Date 2012-05-21.22:39:01.102>
labels = ['type-bug', 'library']
title = "Unusual way of doing 'Inf' in json library"
updated_at = <Date 2012-05-21.23:55:13.475>
user = 'https://bugs.python.org/sbermeister'

bugs.python.org fields:

activity = <Date 2012-05-21.23:55:13.475>
actor = 'ezio.melotti'
assignee = 'ezio.melotti'
closed = True
closed_date = <Date 2012-05-21.23:55:13.476>
closer = 'ezio.melotti'
components = ['Library (Lib)']
creation = <Date 2012-05-21.22:39:01.102>
creator = 'sbermeister'
dependencies = []
files = []
hgrepos = []
issue_num = 14875
keywords = []
message_count = 3.0
messages = ['161304', '161306', '161307']
nosy_count = 4.0
nosy_names = ['mark.dickinson', 'ezio.melotti', 'python-dev', 'sbermeister']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue14875'
versions = ['Python 2.7', 'Python 3.2', 'Python 3.3']

[sbermeister]

If you have a look at the file 'python/Lib/json/encode.py', on line 30 it says:

    # Assume this produces an infinity on all machines (probably not guaranteed)
    INFINITY = float('1e66666')
    FLOAT_REPR = repr

Isn't float('Inf') an accepted way of doing this? Should line 31 be INFINITY = float('Inf')?

Sasha

[python-dev]

New changeset c13066f752a8 by Ezio Melotti in branch '2.7':
bpo-14875: Use float('inf') instead of float('1e66666') in the json module.
http://hg.python.org/cpython/rev/c13066f752a8

New changeset 06f21c6e0f2b by Ezio Melotti in branch '3.2':
bpo-14875: Use float('inf') instead of float('1e66666') in the json module.
http://hg.python.org/cpython/rev/06f21c6e0f2b

New changeset f5f5db593d99 by Ezio Melotti in branch 'default':
bpo-14875: merge with 3.2.
http://hg.python.org/cpython/rev/f5f5db593d99

[ezio-melotti]

Fixed, thanks for the report!