[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git
Subject:    Re: [PATCHv8 6/7] gitweb: picon avatar provider
From:       Giuseppe Bilotta <giuseppe.bilotta () gmail ! com>
Date:       2009-06-30 20:50:42
Message-ID: cb7bb73a0906301350u6c44628ds5bbee6b77abf7983 () mail ! gmail ! com
[Download RAW message or body]

2009/6/30 Jakub Narebski <jnareb@gmail.com>:
> On Tue, 30 Jun 2009, Giuseppe Bilotta wrote:
>> -     # Currently only gravatars are supported, but other forms such as
>> -     # picons can be added by putting an else up here and defining $url
>> +     # Other providers can be added by extending the if chain, defining $url
>>       # as needed. If no variant puts something in $url, we assume avatars
>>       # are completely disabled/unavailable.
>
> I see that you have updated comments too.

Well, I tried to address all the points that were raised during review 8-)

-- 
Giuseppe "Oblomov" Bilotta
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[prev in list] [next in list] [prev in thread] [next in thread]