[MPlayer-dev-eng] [PATCH] tv:// for win32
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Tue Oct 9 14:30:16 CEST 2007
Hello,
On Tue, Oct 09, 2007 at 07:04:13PM +0700, Vladimir Voroshilov wrote:
> 2007/10/9, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> > On Tue, Oct 09, 2007 at 04:33:49PM +0700, Vladimir Voroshilov wrote:
> > > 2007/9/8, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> > > > [...]
> > > > A proper review will hopefully follow some other day soon... ;-)
> > >
> > > I'm still waiting ;)
> >
> > I think I lack the time for that, but I do have a few comments.
>
> Should i assume that remaining code is acceptable for commit (IOW:
> other issues, if any,
> can be fixed after commit)?
Hmm.. seems I forgot to actually type it. I wanted to say: apply if you
think it is okay and nobody objects.
[...]
> > And lastly, are you sure "#pragma pack" works with gcc? Even if it does
> > I think it would be preferable if you could find the corresponding
> > attributes and use them, but I'll leave that up to you to decide in the
> > end (as long as they work with gcc and don't give ugly warnings at
> > least).
>
> Yes it works,
Good to know, someone asked only a few weeks ago about that.
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list