[FFmpeg-devel] FFmpeg OPW

Reynaldo H. Verdejo Pinochet r.verdejo at sisa.samsung.com
Fri Jul 4 05:15:02 CEST 2014


Hi Timothy.

On 07/03/2014 08:34 PM, Timothy Gu wrote:
> On Thu, Jul 3, 2014 at 3:11 PM, Reynaldo H. Verdejo Pinochet wrote:
>> Patch for web/news attached. OK to commit?
>
> Please do a 80-col wrap. Some remarks:

Wouldn't generally wrap HTML text inside </p> to avoid littering
it with trailing white space but looking at rest of the entries
it seems to be preferred indeed. Done.

>
>> +support at least one internship (6K USD) so if you were holding on
>
> USD). So if you are holding on to

Meant to use "were" and don't like the "to" in "TO your donation TO
FFmpeg", guess the meaning its clear and still gramaticaly correct?

>
>> +your donation to FFmpeg, this is a great chance for you to come
>> +forward, get in touch and help both the project and a great initiative
>> +all at once!
>
> This sentence is kinda long. Consider dropping "get in touch" and/or
> "all at once" and/or breaking up the sentence.
>

Dropped the "all at once" but I'm not 100% happy with the sentence.
That was the bit I wanted to stress. Anyway, done.

The 'long' enumerative I kept. That was an style/tone decision.

Thanks for taking a look.

Bests,

-- 
Reynaldo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-web-add-initial-news-entry-about-OPW.patch
Type: text/x-patch
Size: 1915 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140703/13084788/attachment.bin>


More information about the ffmpeg-devel mailing list