Gwibber

Merge lp:~ken-vandine/gwibber/time-diff into lp:gwibber

  1. time-diff
  2. Merge into trunk
Proposed by Ken VanDine
Status: Merged
Merged at revision: 1230
Proposed branch: lp:~ken-vandine/gwibber/time-diff
Merge into: lp:gwibber
Diff against target: 74 lines (+8/-8)
2 files modified
libgwibber/utils.vala (+1/-1)
test/vala/test-utils.vala (+7/-7)
To merge this branch: bzr merge lp:~ken-vandine/gwibber/time-diff
Reviewer Review Type Date Requested Status
Michael Terry (community) Approve
Review via email: mp+84877@code.launchpad.net

Description of the change

Dropped a seemingly arbitrary subtraction of 3600 seconds when calculating the time difference. This was needed to work around some other problem in the past, but now it is making the delta off by an hour.

To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

Seems fine. Assuming the changed offset for seconds and minutes was intentional.

review: Approve
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Thanks, yeah intentional. Makes more sense for the tests.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'libgwibber/utils.vala'
2--- libgwibber/utils.vala 2011-10-02 08:10:09 +0000
3+++ libgwibber/utils.vala 2011-12-08 02:17:23 +0000
4@@ -79,7 +79,7 @@
5
6 public string generate_time_string(uint t)
7 {
8- var now = (int)Time.gm (time_t ()).mktime() - 3600;
9+ var now = (int)Time.gm (time_t ()).mktime();
10 var d = (int)(now - t);
11 if(d < 60)
12 return _("a few seconds ago");
13
14=== modified file 'test/vala/test-utils.vala'
15--- test/vala/test-utils.vala 2011-11-05 14:10:38 +0000
16+++ test/vala/test-utils.vala 2011-12-08 02:17:23 +0000
17@@ -46,7 +46,7 @@
18 internal static void test_time_string_seconds ()
19 {
20 var utils = new Gwibber.Utils ();
21- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 29;
22+ var t = ((int)Time.gm (time_t ()).mktime()) - 59;
23 var ts = utils.generate_time_string (t);
24 assert (ts == "a few seconds ago");
25 }
26@@ -54,42 +54,42 @@
27 internal static void test_time_string_minute ()
28 {
29 var utils = new Gwibber.Utils ();
30- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 60;
31+ var t = ((int)Time.gm (time_t ()).mktime()) - 60;
32 var ts = utils.generate_time_string (t);
33 assert (ts == "1 minute ago");
34 }
35 internal static void test_time_string_minutes ()
36 {
37 var utils = new Gwibber.Utils ();
38- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 121;
39+ var t = ((int)Time.gm (time_t ()).mktime()) - 3559;
40 var ts = utils.generate_time_string (t);
41 assert (ts.contains ("minutes ago"));
42 }
43 internal static void test_time_string_hour ()
44 {
45 var utils = new Gwibber.Utils ();
46- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 3601;
47+ var t = ((int)Time.gm (time_t ()).mktime()) - 3601;
48 var ts = utils.generate_time_string (t);
49 assert (ts == "1 hour ago");
50 }
51 internal static void test_time_string_hours ()
52 {
53 var utils = new Gwibber.Utils ();
54- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 7201;
55+ var t = ((int)Time.gm (time_t ()).mktime()) - 7201;
56 var ts = utils.generate_time_string (t);
57 assert (ts.contains ("hours ago"));
58 }
59 internal static void test_time_string_day ()
60 {
61 var utils = new Gwibber.Utils ();
62- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 86400;
63+ var t = ((int)Time.gm (time_t ()).mktime()) - 86400;
64 var ts = utils.generate_time_string (t);
65 assert (ts == "1 day ago");
66 }
67 internal static void test_time_string_days ()
68 {
69 var utils = new Gwibber.Utils ();
70- var t = ((int)Time.gm (time_t ()).mktime() - 3600) - 172800;
71+ var t = ((int)Time.gm (time_t ()).mktime()) - 172800;
72 var ts = utils.generate_time_string (t);
73 assert (ts.contains ("days ago"));
74 }