Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

preparser doesn't parse hex input: #37

Closed
williamstein opened this issue Sep 12, 2006 · 7 comments
Closed

preparser doesn't parse hex input: #37

williamstein opened this issue Sep 12, 2006 · 7 comments
Labels
c: basic arithmetic p: minor / 4 t: bug
Milestone
sage-2.10.3

Comments

@williamstein
Copy link
Contributor 

sage: 0x5
------------------------------------------------------------
   File "<ipython console>", line 1
     ZZ(0)x5
           ^
SyntaxError: invalid syntax

Component: basic arithmetic

Issue created by migration from https://trac.sagemath.org/ticket/37
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Aug 22, 2007

comment:2

This is still a problem with Sage 2.8.2.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-2.9.1 milestone Sep 10, 2007
@malb
Copy link
Member

malb commented Feb 27, 2008

comment:4

The behaviour has changes (this is 2.10.2):

sage: 0x10
16

sage: 0xA
------------------------------------------------------------
   File "<ipython console>", line 1
     Integer(0x)A
                ^
<type 'exceptions.SyntaxError'>: invalid syntax

sage: 0xa
------------------------------------------------------------
   File "<ipython console>", line 1
     Integer(0x)a
                ^
<type 'exceptions.SyntaxError'>: invalid syntax

@williamstein
Copy link
Contributor Author

comment:5

Note: #2144 is (a) a dupe of this, and (b) actually doesn't fix the problem.

@williamstein
Copy link
Contributor Author

comment:6

Attachment: sage-37.patch.gz

@ncalexan
Copy link
Mannequin

ncalexan mannequin commented Feb 28, 2008

comment:7

Doctests look good, commit.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Feb 28, 2008

comment:8

Merged in Sage 2.10.3.rc0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Feb 28, 2008
@robertwb
Copy link
Contributor

comment:9

Nick beat me too it, but I think it looks good too.

tobiasdiez pushed a commit to tobiasdiez/sage that referenced this issue Feb 22, 2024
@marcelotrevisani
Fix the lack of space between the package name and the constrain (sag…
8381bde 
…emath#37)

* Fix the lack of space between the package name and the constrain

* Add tests to cover more case scenarios

* Recipe improvements, download sdist packages using requests instead of pip
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: basic arithmetic p: minor / 4 t: bug
Projects
None yet
Milestone
sage-2.10.3
Development

No branches or pull requests
3 participants
@robertwb @williamstein @malb