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

Jason Holt jholt at google.com
Fri Apr 10 18:55:03 CEST 2009


Manpage fixed.  Thanks.


> +void send_udp(char *send_to_ip, int port, char *mesg);
>
> This is ugly, please just reorder the functions sensibly.


send_udp calls exit_player, which calls exit_player_with_rc, which calls
send_udp.


Also, I wonder if this has to be in mplayer.c, which is already big
> enough.


I'm happy to move it.  stream/udp.c is the most related source file I could
find, but it's a little weird to have it in stream/.  I could create new
.c/.h files if you think it's worthwhile.

> @@ -743,6 +766,168 @@
> >
> > +  sendto(sockfd,
> > +         mesg,
> > +         strlen(mesg),
> > +         0,
> > +         (struct sockaddr *) &socketinfo,
> > +         sizeof(socketinfo));
>
> This is ugly.
>
> > +void udp_slave_sync() {
>
> (void)


Fixed.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-udp-patch-9.diff
Type: text/x-diff
Size: 12191 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090410/623430dc/attachment.diff>


More information about the MPlayer-dev-eng mailing list