git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCHv8 1/7] gitweb: refactor author name insertion

From
Jakub Narebski <jnareb@gmail.com>
Date
Jul 1, 2009, 07:41 UTC
Message-ID
<200907010941.40641.jnareb@gmail.com>
In-Reply-To
<cb7bb73a0906301615l1352f7c6i6a9939ccca6a70dc@mail.gmail.com>
On Wed, 1 July 2009, Giuseppe Bilotta wrote:
> On Tue, Jun 30, 2009 at 11:08 PM, Jakub Narebski<jnareb@gmail.com> wrote:
Show 11 quoted lines
>> # prints time in local timezone, from result of parse_date
>> # in the following format: " (HH:MM +/-tz_offset)" or " (%H:%M %z)"
>> # with "atnight" warning when 0 <= hour_local < 6
>>
>> or something like that.  Or just
>>
>> # prints time in local timezone, from result of parse_date
> 
> How about
> 
> # prints time in local timezone, highlight night hours
Or
# prints local part of parse_date, highlight night hours

It important, I think, to state that print_local_time takes result of parse_date as an argument.

Nevertheless that is just nitpicking and bikeshedding.
 
Show 14 quoted lines
>>>>> +sub print_local_time {
>>>>> +     my %date = @_;
>>>>> +     if ($date{'hour_local'} < 6) {
>>>>> +             printf(" (<span class=\"atnight\">%02d:%02d</span> %s)",
>>>>> +                     $date{'hour_local'}, $date{'minute_local'}, $date{'tz_local'});
>>>>> +     } else {
>>>>> +             printf(" (%02d:%02d %s)",
>>>>> +                     $date{'hour_local'}, $date{'minute_local'}, $date{'tz_local'});
>>>>> +     }
>>>>> +}
> 
> -- 
> Giuseppe "Oblomov" Bilotta
> 
-- 
Jakub Narebski
Poland
Previous: Giuseppe Bilotta
Message 22 of 22 in “[PATCHv8 0/7] gitweb: avatar support”
  1. Giuseppe BilottaJun 29, 2009
  2. 1/7 gitweb: refactor author name insertionGiuseppe Bilotta, Jun 29, 2009
  3. 2/7 gitweb: uniform author info for commit and commitdiffGiuseppe Bilotta, Jun 29, 2009
  4. 3/7 gitweb: use git_print_authorship_rows in 'tag' view tooGiuseppe Bilotta, Jun 29, 2009
  5. 4/7 gitweb: (gr)avatar supportGiuseppe Bilotta, Jun 29, 2009
  6. 5/7 gitweb: gravatar url cacheGiuseppe Bilotta, Jun 29, 2009
  7. 6/7 gitweb: picon avatar providerGiuseppe Bilotta, Jun 29, 2009
  8. 7/7 gitweb: add empty alt text to avatar imgGiuseppe Bilotta, Jun 29, 2009
  9. Jakub NarebskiJun 30, 2009
  10. Jakub NarebskiJun 30, 2009
  11. Giuseppe BilottaJun 30, 2009
  12. Jakub NarebskiJun 30, 2009
  13. Junio C HamanoJun 30, 2009
  14. Giuseppe BilottaJun 30, 2009
  15. Jakub NarebskiJun 30, 2009
  16. Jakub NarebskiJun 30, 2009
  17. Jakub NarebskiJun 30, 2009
  18. Jakub NarebskiJun 30, 2009
  19. Giuseppe BilottaJun 30, 2009
  20. Jakub NarebskiJun 30, 2009
  21. Giuseppe BilottaJun 30, 2009
  22. Jakub NarebskiJul 1, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.