[MPlayer-dev-eng] [PATCH] Network synchronized playback using UDP

[Diego Biurrun]

On Wed, Feb 24, 2010 at 07:40:55PM -0800, Jason Holt wrote:
> >
> > We already have udp-related code in stream/stream_udp.c. Given that
> > completely broken network-related code always seems to be I'd rather
> > have not any of it more than once.
> 
> I understand the sentiment, and did check there before writing my
> patch. I'd have to refactor that code to pull out the basic stuff I
> need, and then I'd still have to do the setsockopt() stuff separately.
> After refactoring, the udp stuff probably shouldn't live in stream/
> anymore. I'm at 62 messages in the thread and 21 revisions of the
> patch, and coming up on the thread's one year anniversary, so I'm a
> little reluctant to start refactoring the streaming code.

I understand the sentiment as well, but you have to admit that this has
being going rather well the past few weeks and your stuff has been duly
reviewed.

What would it take to convince you to refactor that streaming code? :-)

Diego