[libav-bugs] [Bug 525] Optimize memory management for mp4 muxer

bugzilla-daemon at aruru.libav.org bugzilla-daemon at aruru.libav.org
Sun Jun 2 22:45:36 CEST 2013


https://bugzilla.libav.org/show_bug.cgi?id=525

--- Comment #5 from Martin Storsjö <martin at martin.st> 2013-06-02 22:45:36 CEST ---
Ok, I've sent a neatly separated set of patches based on this to the
libav-devel mailing list for peer-review now, see
http://lists.libav.org/pipermail/libav-devel/2013-June/047261.html.

For the capacity-related patches, I did some minor changes, I dropped the
redundant doxygen comments for the new variables (which didn't add much else
than the names themselves), moved the variables within the struct to be located
next to the arrays. I also changed the reallocs to use a temp variable for the
return value, to avoid leaking the old allocation if the realloc failed. (The
current way the cluster array is reallocated has got the same potential leak.)

The patchset is also available at
https://github.com/mstorsjo/libav/commits/movenc-memopt if you want to have a
look (or check that I didn't screw something up in my local changes).

-- 
Configure bugmail: https://bugzilla.libav.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.


More information about the libav-bugs mailing list