[MPlayer-dev-eng] [PATCH] SUN XVR-100 VO driver 3. try
Balatoni Denes
dbalatoni at programozo.hu
Mon Jul 23 20:27:22 CEST 2007
Hi!
Thanks for the review Diego, new pathc attached.
Sunday 22 July 2007 23:23-kor Diego Biurrun ezt írta:
> nit: trailing whitespace
Those should be fixed.
>
> > +ioctl(pfb_devfd, VIS_GETIDENTIFIER, &ident)
> > +ioctl(pfb_devfd, FBIOGATTR, &attr)
>
> ioctl is a macro?
I admit the configure part was completly broken, this should be working
(tested).
> > + * written for xine by
> > + * Robin Kay <komadori [at] gekkou [dot] co [dot] uk>
> > + *
> > + * Sun XVR-100 framebuffer graciously donated by Jake Goerzen.
> > + *
> > + * Ported to mplayer by Denes Balatoni
> > + * Contains portions from the mga and tdfix_vid vo drivers
> > + *
> > + * no double-buffering, as it would slow down playback (waiting for
> > vertical retraces) + * FIXME: only YV12 supported for now
>
> Please put this at the top of the header along with the copyright
> notices.
Done.
>
> > + if(!(flags & VOFLAG_XOVERLAY_SUB_VO)) {
>
> I prefer a space between if and the opening parentheses. You use this
> inconsistently.
Done.
>
> Otherwise the patch looks OK from my side.
>
> Diego
bye
Denes
--
---
What kills me, doesn't make me stronger.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: xvr100_try3.diff
Type: text/x-diff
Size: 18429 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070723/f337381a/attachment.diff>
More information about the MPlayer-dev-eng
mailing list