From: Kevin Cernekee <cernekee@gmail.com>
To: Junio C Hamano <gitster@pobox.com>, Jakub Narebski <jnareb@gmail.com>
Cc: git@vger.kernel.org
Subject: [PATCH 1/2] gitweb: rename parse_date() to format_date()
Date: Fri, 18 Mar 2011 22:39:33 -0700 [thread overview]
Message-ID: <ab54ba2199cc7487e383a31e3aa65885@localhost> (raw)
One might reasonably expect a function named parse_date() to be used
for something along these lines:
$unix_time_t = parse_date("2011-03-19");
But instead, gitweb's parse_date works more like:
&parse_date(1300505805) = {
'hour' => 3,
'minute' => 36,
...
'rfc2822' => 'Sat, 19 Mar 2011 03:36:45 +0000',
...
}
Rename the function to improve clarity. No change to functionality.
Signed-off-by: Kevin Cernekee <cernekee@gmail.com>
---
gitweb/gitweb.perl | 18 +++++++++---------
1 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index b04ab8c..57ef08c 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -2893,7 +2893,7 @@ sub git_get_rev_name_tags {
## ----------------------------------------------------------------------
## parse to hash functions
-sub parse_date {
+sub format_date {
my $epoch = shift;
my $tz = shift || "-0000";
@@ -3953,7 +3953,7 @@ sub git_print_authorship {
my $tag = $opts{-tag} || 'div';
my $author = $co->{'author_name'};
- my %ad = parse_date($co->{'author_epoch'}, $co->{'author_tz'});
+ my %ad = format_date($co->{'author_epoch'}, $co->{'author_tz'});
print "<$tag class=\"author_date\">" .
format_search_author($author, "author", esc_html($author)) .
" [$ad{'rfc2822'}";
@@ -3973,7 +3973,7 @@ sub git_print_authorship_rows {
my @people = @_;
@people = ('author', 'committer') unless @people;
foreach my $who (@people) {
- my %wd = parse_date($co->{"${who}_epoch"}, $co->{"${who}_tz"});
+ my %wd = format_date($co->{"${who}_epoch"}, $co->{"${who}_tz"});
print "<tr><td>$who</td><td>" .
format_search_author($co->{"${who}_name"}, $who,
esc_html($co->{"${who}_name"})) . " " .
@@ -4906,7 +4906,7 @@ sub git_log_body {
next if !%co;
my $commit = $co{'id'};
my $ref = format_ref_marker($refs, $commit);
- my %ad = parse_date($co{'author_epoch'});
+ my %ad = format_date($co{'author_epoch'});
git_print_header_div('commit',
"<span class=\"age\">$co{'age_string'}</span>" .
esc_html($co{'title'}) . $ref,
@@ -5369,7 +5369,7 @@ sub git_project_index {
sub git_summary {
my $descr = git_get_project_description($project) || "none";
my %co = parse_commit("HEAD");
- my %cd = %co ? parse_date($co{'committer_epoch'}, $co{'committer_tz'}) : ();
+ my %cd = %co ? format_date($co{'committer_epoch'}, $co{'committer_tz'}) : ();
my $head = $co{'id'};
my $remote_heads = gitweb_check_feature('remote_heads');
@@ -5674,7 +5674,7 @@ sub git_blame_common {
my $short_rev = substr($full_rev, 0, 8);
my $author = $meta->{'author'};
my %date =
- parse_date($meta->{'author-time'}, $meta->{'author-tz'});
+ format_date($meta->{'author-time'}, $meta->{'author-tz'});
my $date = $date{'iso-tz'};
if ($group_size) {
$current_color = ($current_color + 1) % $num_colors;
@@ -6702,7 +6702,7 @@ sub git_commitdiff {
-charset => 'utf-8',
-expires => $expires,
-content_disposition => 'inline; filename="' . "$filename" . '"');
- my %ad = parse_date($co{'author_epoch'}, $co{'author_tz'});
+ my %ad = format_date($co{'author_epoch'}, $co{'author_tz'});
print "From: " . to_utf8($co{'author'}) . "\n";
print "Date: $ad{'rfc2822'} ($ad{'tz_local'})\n";
print "Subject: " . to_utf8($co{'title'}) . "\n";
@@ -7064,7 +7064,7 @@ sub git_feed {
if (defined($commitlist[0])) {
%latest_commit = %{$commitlist[0]};
my $latest_epoch = $latest_commit{'committer_epoch'};
- %latest_date = parse_date($latest_epoch);
+ %latest_date = format_date($latest_epoch);
my $if_modified = $cgi->http('IF_MODIFIED_SINCE');
if (defined $if_modified) {
my $since;
@@ -7195,7 +7195,7 @@ XML
if (($i >= 20) && ((time - $co{'author_epoch'}) > 48*60*60)) {
last;
}
- my %cd = parse_date($co{'author_epoch'});
+ my %cd = format_date($co{'author_epoch'});
# get list of changed files
open my $fd, "-|", git_cmd(), "diff-tree", '-r', @diff_opts,
--
1.7.4.1
next reply other threads:[~2011-03-19 5:44 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-19 5:39 Kevin Cernekee [this message]
2011-03-19 5:39 ` [PATCH v4 2/2] gitweb: introduce localtime feature Kevin Cernekee
2011-03-19 15:18 ` Jakub Narebski
2011-03-19 17:56 ` Junio C Hamano
2011-03-19 19:49 ` Kevin Cernekee
2011-03-19 21:09 ` Jakub Narebski
2011-03-19 21:22 ` Kevin Cernekee
2011-03-19 21:41 ` Jakub Narebski
2011-03-20 22:38 ` J.H.
2011-03-20 23:44 ` Kevin Cernekee
2011-03-21 0:20 ` Jakub Narebski
2011-03-21 2:35 ` J.H.
2011-03-21 16:01 ` Jakub Narebski
2011-03-21 18:39 ` Piotr Krukowiecki
2011-03-21 18:39 ` J.H.
2011-03-21 22:20 ` Jakub Narebski
2011-03-24 0:08 ` [PATCH 0/1] Gitweb: Change timezone John 'Warthog9' Hawley
2011-03-24 0:08 ` [PATCH 1/1] gitweb: javascript ability to adjust time based on timezone John 'Warthog9' Hawley
2011-03-24 5:23 ` Kevin Cernekee
2011-03-24 7:21 ` J.H.
2011-03-24 21:23 ` Jakub Narebski
2011-03-24 20:19 ` Jakub Narebski
2011-03-24 22:00 ` Kevin Cernekee
2011-03-24 22:29 ` J.H.
2011-03-24 23:04 ` J.H.
2011-03-24 23:36 ` Jakub Narebski
2011-03-24 15:17 ` Jakub Narebski
2011-03-25 15:20 ` [PATCH (BUGFIX)] gitweb: Fix handling of fractional timezones in parse_date Jakub Narebski
2011-03-25 16:26 ` Kevin Cernekee
2011-03-25 16:50 ` [PATCH (BUGFIX) v2] " Jakub Narebski
2011-03-25 17:15 ` [PATCH (BUGFIX)] " Junio C Hamano
2011-03-25 17:47 ` Jakub Narebski
2011-03-25 19:20 ` [PATCH (BUGFIX) v3] " Jakub Narebski
2011-03-19 10:33 ` [PATCH 1/2] gitweb: rename parse_date() to format_date() Jakub Narebski
2011-03-19 11:50 ` Jon Seymour
2011-03-19 18:00 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ab54ba2199cc7487e383a31e3aa65885@localhost \
--to=cernekee@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jnareb@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).