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

Fix gmp type declarations #11992

Closed
robertwb opened this issue Nov 5, 2011 · 3 comments
Closed

Fix gmp type declarations #11992

robertwb opened this issue Nov 5, 2011 · 3 comments
Assignees
@jasongrout
Labels
r: duplicate

Comments

@robertwb
Copy link
Contributor

robertwb commented Nov 5, 2011

They are currently declared as void*, which will give warnings in the new Cython and allow incorrect C code (e.g. assignments) to be generated.

CC: @kiwifb

Component: misc

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/11992
@robertwb robertwb added this to the sage-5.11 milestone Nov 5, 2011
@robertwb
Copy link
Contributor Author

robertwb commented Nov 5, 2011

comment:1

Attachment: 11992-gmp-types.patch.gz

Sage does not compile with this patch, numerous other things need to be fixed up first.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

comment:7

Duplicate of #15946.

@jdemeyer jdemeyer removed this from the sage-6.4 milestone Oct 12, 2014
@vbraun vbraun closed this as completed Oct 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

Labels
r: duplicate
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jasongrout @robertwb @vbraun @jdemeyer