[PATCH] RTSP-MS 13/15: transport context

headers Michael Niedermayer | 1 Feb 2009 11:06

On Sat, Jan 31, 2009 at 10:28:35PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Sat, Jan 31, 2009 at 8:00 PM, Michael Niedermayer <michaelni <at> gmx.at> wrote:
> > On Sat, Jan 31, 2009 at 07:35:28PM -0500, Ronald S. Bultje wrote:
> >> store the ASF demuxer context in the RTSP demuxer "private" context
> >> (RTSPState)->asf_ctx, so that it's shared between all streams in the
> >> session. That value needs to be accessible from the streams.
> >
> > of course
> >
> > why do they not have access to it? I mean why is RTSPState / AVFormatContext
> > not passed to the respective functions?
> 
> Like I said, that's the alternative way of solving this. I just chose
> this way because I thought it'd involve less coding changes, which
> generally appears to be preferred in this community. I'll try your
> suggested alternative approach now.

I prefer problems to be fixed not worked around.
That may or may not mean less coding changes.

[...]
--

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes

Re: [PATCH] RTSP-MS 13/15: transport context