[FFmpeg-devel] [PATCH] fateserver: add FFmpeg website-style banner and navbar

Timothy Gu timothygu99 at gmail.com
Fri Jan 17 06:19:57 CET 2014


On Thu, Jan 16, 2014 at 9:13 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
> On Thu, Jan 16, 2014 at 9:00 PM, Lou Logan <lou at lrcd.com> wrote:
>> On Thu, 16 Jan 2014 20:52:00 -0800, Timothy Gu wrote:
>>
>>> Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
>>> ---
>>> Fix an obvious mistake.
>>
>> Just a reminder to drop this comment upon commit.
>
> It would be automatically be dropped as it is not part of the commit message.
>
> http://kparal.wordpress.com/2011/08/03/git-tip-of-the-day-introduction-text-when-emailing-patches/
>
>>
>>>  index.cgi  | 29 +++++++++++++++++++++++++++++
>>>  report.cgi | 29 +++++++++++++++++++++++++++++
>>>  2 files changed, 58 insertions(+)
>>
>> LGTM AFAICT, and looks better than my half-assed attempt, but I admit I
>> did not apply locally and check visually.
>

> I didn't either :D

Wait, I found a big bug in the code I wrote. Stupid perl.

Timothy


More information about the ffmpeg-devel mailing list