Opened 5 years ago

Closed 5 years ago

#23308 closed enhancement (fixed)

Remove unused variable from RealNumber object

Reported by: vklein Owned by:
Priority: trivial Milestone: sage-8.0
Component: basic arithmetic Keywords: thursdaysbdx
Cc: vdelecroix Merged in:
Authors: Frédéric Chapoton Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: fcda7e1 (Commits, GitHub, GitLab) Commit: fcda7e1e14cf912cdcedd3f4629396c6c1dafe5d
Dependencies: Stopgaps:

Status badges

Description

in _set instance method of RealNumber? 

cdef RealNumber n, d

are unused and should be removed

Change History (3)

comment:1 Changed 5 years ago by chapoton

  • Authors changed from Vincent Klein to Frédéric Chapoton
  • Branch set to u/chapoton/23308
  • Commit set to fcda7e1e14cf912cdcedd3f4629396c6c1dafe5d
  • Status changed from new to needs_review

New commits:

​fcda7e1trac 23308 remove unused variables in real mpfr

comment:2 Changed 5 years ago by jdemeyer

  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_review to positive_review

comment:3 Changed 5 years ago by vbraun

  • Branch changed from u/chapoton/23308 to fcda7e1e14cf912cdcedd3f4629396c6c1dafe5d
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.