[FFmpeg-devel] Patch to documentation (ffmpeg-web)

[Misha Penkov]

On 16 August 2012 16:35, Lou Logan <lou at lrcd.com> wrote:
> On Wed, Aug 15, 2012, at 09:52 PM, Misha Penkov wrote:
>> Hi,
>>
>> The attached patch includes a link to updated FFmpeg tutorial sources.
>>
>> Michael
>>
>> From 36f436f69ff6bd76eaf6aed96dd38d9793f4f21d Mon Sep 17 00:00:00 2001
>> From: Michael Penkov <misha.penkov at gmail.com>
>> Date: Thu, 16 Aug 2012 14:44:59 +0900
>> Subject: [PATCH] Include link to updated tutorial source.
>>
>> ---
>>  src/documentation |    3 ++-
>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/documentation b/src/documentation
>> index fc7cf03..264ac64 100644
>> --- a/src/documentation
>> +++ b/src/documentation
>> @@ -49,5 +49,6 @@ describes how to add a filter to libavfilter.</li>
>>  <li><a href="http://www.inb.uni-luebeck.de/~boehme/using_libavcodec.html">Using libavformat and libavcodec</a> by
>>  Martin Böhme, a good overview of the FFmpeg APIs.</li>
>>  <li><a href="http://dranger.com/ffmpeg/">An FFmpeg and SDL Tutorial</a>
>> -by Stephen Dranger, explains how to write a video player based on FFmpeg.</li>
>> +by Stephen Dranger, explains how to write a video player based on FFmpeg
>> +(<a href="https://github.com/mpenkov/ffmpeg-tutorial">updated source code</a> by Michael Penkov).</li>
>
> I think "updated version" is a better phrase in this case, but it's not
> a strong opinion so either is fine for me. I also wouldn't mind if you
> make a new list item for your tutorial instead of appending the Dranger
> description, but again I'm fine with either...so that makes this a long
> winded "LGTM".

I only updated the C source code.  The actual HTML tutorial (which is
still out of date) spans several pages of explanations of what the
source actually does.  Making it a separate item and calling it an
updated version would be a bit of an overstatement at this stage,
IMHO.  Ideally, it would be great to update the explanations as well,
but I don't have the resources to do that right now.

> Thanks for providing an updated tutorial. Do you plan on keeping it
> relatively up to date?

Yes, I do.  Since it's on Github, other people will be able to
contribute as well.

Cheers,
Michael

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel