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

Remove unused variable from RealNumber object #23308

Closed
vinklein mannequin opened this issue Jun 22, 2017 · 6 comments
Closed

Remove unused variable from RealNumber object #23308

vinklein mannequin opened this issue Jun 22, 2017 · 6 comments
Labels
c: basic arithmetic p: trivial / 5 t: enhancement
Milestone
sage-8.0

Comments

@vinklein
Copy link
Mannequin

vinklein mannequin commented Jun 22, 2017

in _set instance method of RealNumber

cdef RealNumber n, d

are unused and should be removed

CC: @videlec

Component: basic arithmetic

Keywords: thursdaysbdx

Author: Frédéric Chapoton

Branch/Commit: fcda7e1

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/23308
@vinklein vinklein mannequin added this to the sage-8.0 milestone Jun 22, 2017
@fchapoton
Copy link
Contributor

New commits:

fcda7e1trac 23308 remove unused variables in real mpfr

@fchapoton
Copy link
Contributor

Commit: fcda7e1

@fchapoton
Copy link
Contributor

Changed author from Vincent Klein to Frédéric Chapoton

@fchapoton
Copy link
Contributor

Branch: u/chapoton/23308

@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@vbraun
Copy link
Member

vbraun commented Jun 25, 2017

Changed branch from u/chapoton/23308 to fcda7e1

@vbraun vbraun closed this as completed in ede52b6 Jun 25, 2017
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: trivial / 5 t: enhancement
Projects
None yet
Milestone
sage-8.0
Development

No branches or pull requests
3 participants
@vbraun @jdemeyer @fchapoton