[FFmpeg-devel] [PATCH] Include content of the news article in the website RSS

Timothy Gu timothygu99 at gmail.com
Wed Apr 23 02:42:57 CEST 2014


On Apr 22, 2014 5:37 PM, "Gerion Entrup" <gerion.entrup at t-online.de> wrote:
>
> Am Montag 21 April 2014, 18:07:04 schrieb Timothy Gu:
> > On Apr 11, 2014 3:29 PM, "Gerion Entrup" <gerion.entrup at t-online.de>
wrote:
> > >
> > > Fixes Trac ticket #844
> > > ---
> > >  Makefile | 26 +++++++++++++++++++++-----
> > >  1 file changed, 21 insertions(+), 5 deletions(-)
> >
> > Sorry for the delay.
>
> No problem.
>
> > The patch looks good, but a bonus point will be if you
> > remove the trailing whitespace on some lines.

> It seems that this comes from kmail. I attached the original git
mailoutput.

No, not the patch itself. In the output feed there are some trailing spaces.

>
> >
> > Just curious, did you test the feed with the W3C validator?
> Not before you asked:
>
> Test with [1]:
> This document was successfully checked as well-formed XML!
> One Warning: No DOCTYPE found! Checking XML syntax only.
>
> In the list of common doctypes [2] RSS do not appear.

See http://validator.w3.org/feed/

[...]

Timothy


More information about the ffmpeg-devel mailing list