[FFmpeg-devel] [PATCH] RDT/Realmedia patches #2

Ronald S. Bultje rsbultje
Tue Nov 11 19:13:23 CET 2008


On Tue, Nov 11, 2008 at 10:20 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> audio/video is a type not a set
> one could speak of the set of all video streams or of a subset of all video
> streams but your text is too unclear to even guess if either or none of this
> is meant.

I don't know which of the two applies. In all streams I've seen so
far, it's the set of all streams per type, e.g. all audio or all video
streams. So I've renamed it to stream_type. I can call it
stream_type_id if you prefer, since the number by itself doesn't tell
us whether it's audio or video, it's just an id of the set of all
audio or video streams.

> Also id like to say that if you want me (compared to someone else) to approve
> this your useage of the word rule to refer to some kind of streams must be
> removed. A stream or set of streams can be associated with a rule or a set of
> rules but a rule is no stream. These are semantic problems with the terms you
> use.

OK, rule is gone.

New patch attached. (Reindenting is in a separate patch.)

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rdt-rn_parsing.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081111/642093a1/attachment.asc>

More information about the ffmpeg-devel mailing list