Closed Bug 1331940 Opened 7 years ago Closed 7 years ago

Consider a macro to do 'MediaContainerType(MEDIAMIMETYPE("..."))'

Categories

(Core :: Audio/Video: Playback, defect)

49 Branch
defect
Not set
normal

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...
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.