Closed
Bug 1314441
Opened 8 years ago
Closed 4 years ago
Remove unused parts of libstagefright (Part The Second)
Categories
(Core :: Audio/Video: Playback, defect, P3)
Tracking
()
RESOLVED
DUPLICATE
of bug 1408298
People
(Reporter: mozbugz, Unassigned)
References
Details
Another run at stripping libstagefright of its unused bits, mostly to prevent false security alarms.
E.g.: Utils.cpp has a lot of disabled code (inside an '#if 0' section), which itself relies on parsing some specific MP4 boxes, we could remove all of this.
Ideally, we should take the reverse approach of examining what information is actually used outside of libstagefright, and we would only keep the parsing code that generates this information.
This effort should be balanced against the impending complete removal of libstagefright, but it may still take a few releases to get there.
|
||
Comment 1•8 years ago
|
||
Removing the easy stuff sounds good. I gave up another removing in bug 1205369 because I couldn't figure out what was using the supposedly unused code. Something `#if 0` sounds like an easy pass.
|
||
Updated•8 years ago
|
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•