Opened 10 years ago

Closed 10 years ago

#11495 closed defect (fixed)

Trac #1092 small memleaks exposed by ntl_ZZ_pE (from 2.8.12.alpha0)

Reported by: jpflori Owned by: rlm
Priority: critical Milestone: sage-4.7.2
Component: memleak Keywords: memleak
Cc: mabshoff, wjp, jpflori, bober Merged in: sage-4.7.2.alpha1
Authors: Willem Jan Palenstijn Reviewers: Jonathan Bober
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

The patch for ticket #1092 has been reverted by commit 8503, so the (not so) small memleaks are back.

Attachments (1)

memleak_ZZ_pE_to_ZZ_pX.patch (623 bytes) - added by jpflori 10 years ago.
original patch for #1092

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by jpflori

original patch for #1092

comment:1 Changed 10 years ago by jpflori

  • Status changed from new to needs_review

comment:2 Changed 10 years ago by bober

  • Cc bober added
  • Reviewers set to Jonathan Bober
  • Status changed from needs_review to positive_review

This looks good to me. It is a simple -1 line change that removes the creation of an unnecessary that is never deleted.

comment:3 Changed 10 years ago by jdemeyer

  • Milestone changed from sage-4.7.1 to sage-4.7.2

comment:4 Changed 10 years ago by jdemeyer

  • Authors changed from wjp to Willem Jan Palenstijn
  • Merged in set to sage-4.7.2.alpha1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.