1
0
mirror of https://github.com/taigrr/wtf synced 2025-01-18 04:03:14 -08:00

4 Commits

Author SHA1 Message Date
Andrew
77823419b5 Temporary changed username. 2018-06-10 00:16:40 +03:00
Bryan Austin
0656f17b80 Fix clocks module chronological sorting issue #98
`clock.LocalTime()` sets the location of a `time.Time` object, but
doesn't change the point in time. Since `clock.LocalTime()`
calls `time.Now()` to create the "local time", what ends up
happening is that the first `LocalTime()` is always "before" the
second one (because of the order of function calls), leading to
unstable sorting.

This change does two things to fix chronological sorting. The
first is to add a `clock.ToLocal(time.Time)` function that
acts like `clock.LocalTime()` for a given time point, so that
a single stable time point can be used throughout sorting. The
second is to do chronological sorting by comparing the string
versions of the local time, which look like (for London vs. SF):

2018-06-01 19:48:26.591550198 +0100 BST
vs.
2018-06-01 11:48:26.5915538 -0700 PDT

There may be a better way, this feels hacky-ish but works for me
on OSX.

One remaining downside is that for locations in the same time zone
(i.e. Avignon and Barcelona in the default settings), order is
determined ~randomly on startup. You could maybe append the location
to the string used for comparison to make alphabetical sorting a
fallback for equivalent times, but at least they don't jump around
anymore.
2018-06-01 12:01:34 -07:00
Chris Cummer
9f59ee1f9f Clock is now actually a clock and not a snapshot in time 2018-05-15 13:14:35 -07:00
Chris Cummer
6fab7c0668 Clocks are sortable 2018-04-19 09:29:49 -07:00