Opened 12 years ago

Closed 12 years ago

#4067 closed defect (fixed)

[with patch, positive review] hmm.pyx and ghmm.pyx valgrind issues

Reported by: mabshoff Owned by: mabshoff
Priority: major Milestone: sage-3.1.2
Component: memleak Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

This is a broken out patch from #3984. It does not fix the doctest, but numerous issues valgrind picks up. Someone needs to remember that C strings are NULL terminated :)

Cheers,

Michael

Attachments (1)

trac_4067.patch​ (1.3 KB) - added by mabshoff 12 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 12 years ago by mabshoff

  • Status changed from new to assigned

Changed 12 years ago by mabshoff

comment:2 Changed 12 years ago by malb

  • Summary changed from [with patch, needs review] hmm.pyx and ghmm.pyx valgrind issues to [with patch, positive review] hmm.pyx and ghmm.pyx valgrind issues

Patch looks good, I only read it though.

comment:3 Changed 12 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from assigned to closed

Merged in Sage 3.1.2.rc0

Note: See TracTickets for help on using tickets.