git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@osdl.org>
To: Junio C Hamano <junkio@cox.net>
Cc: Morten Welinder <mwelinder@gmail.com>,
	Git Mailing List <git@vger.kernel.org>
Subject: Re: Fix approxidate() to understand more extended numbers
Date: Thu, 28 Sep 2006 23:42:19 -0700 (PDT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0609282330440.3952@g5.osdl.org> (raw)
In-Reply-To: <7vd59fp5b9.fsf@assigned-by-dhcp.cox.net>



On Thu, 28 Sep 2006, Junio C Hamano wrote:

> Linus Torvalds <torvalds@osdl.org> writes:
> 
> > On Thu, 28 Sep 2006, Morten Welinder wrote:
> >>
> >> Just don't hack at 12am or 12pm.
> >
> > I think 12pm is correct, but 12am probably isn't (12am should _subtract_ 
> > 12, while 12pm does _not_ add 12).
> 
> But you have "if (hour > 0 && hour < 12)" in both am and pm so
> assignment to tm would not trigger...

That's not the point.

If you write

	12:30 am

you really _should_ subtract 12, leaving you with 0:30. We don't. So we 
end up with a 24-hour time of 12:30, which is obviously _pm_, and wrong.

And "12 am" or "12 pm" doesn't work at all.

> > That said, I have a rice cooker that avoids the problem by saying "0:10 PM" 
> > for ten minutes past midday ;)
> 
> You eat rice?

Ok, is it just me, or is that just a very odd question?

I can see the question "You eat uni?". That really _does_ take a bit of 
getting used to. And Natto I really _really_ don't see the point of. 

But rice? Afaik, it's the most common food-staple in the world. It's not 
exactly odd and exotic..

> > Of course, all sane and civilized countries just use 24-hour format 
> > anyway.
> 
> You are referring to the US, but neither is Japan sane nor
> civilized ;-).

Yeah, well, they've been learning bad habits. But at least they are 
metric.

		Linus

  reply	other threads:[~2006-09-29  6:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-09-28 19:12 Clean up approxidate() in preparation for fixes Linus Torvalds
2006-09-28 19:14 ` Fix approxidate() to understand more extended numbers Linus Torvalds
2006-09-29  0:12   ` Morten Welinder
2006-09-29  6:03     ` Linus Torvalds
2006-09-29  6:14       ` Junio C Hamano
2006-09-29  6:42         ` Linus Torvalds [this message]
2006-09-29  7:09           ` Junio C Hamano
2006-09-29 14:04             ` Johannes Schindelin
2006-09-29 19:36             ` Linus Torvalds

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=Pine.LNX.4.64.0609282330440.3952@g5.osdl.org \
    --to=torvalds@osdl.org \
    --cc=git@vger.kernel.org \
    --cc=junkio@cox.net \
    --cc=mwelinder@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).