Closed
Bug 1331940
Opened 7 years ago
Closed 7 years ago
Consider a macro to do 'MediaContainerType(MEDIAMIMETYPE("..."))'
Categories
(Core :: Audio/Video: Playback, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: mozbugz, Unassigned)
References
Details
Spawned from bug 1331289 comment 5, Jean-Yves suggested having a friendly macro to do `MediaContainerType(MEDIAMIMETYPE("..."))`, e.g.: `MEDIACONTAINERTYPE("...")`. I intentionally kept these two things separate, as I liken 'MEDIAMIMETYPE' to NS_LITERAL_CSTRING, which just creates a literal thing (string or media MIME type) without context. MediaContainerType explicitly provides this context on top of the literal. Now it's true that MEDIACONTAINERTYPE is fairly clear, and saves quite a few characters -- shorter lines are usually easier for humans to parse. I'll open this for opinions now...
Reporter | ||
Comment 1•7 years ago
|
||
I only see 4 instances of "MediaContainerType(MEDIAMIMETYPE" in non-test code, so at this point I'd prefer not to introduce a macro for this fairly rare occurrence.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•