From 379460558b2dfd7ddbd0b29abfaa2642d9f0961e Mon Sep 17 00:00:00 2001
From: Daniel Farina <daniel@fdr.io>
Date: Mon, 27 May 2013 09:52:56 -0700
Subject: [PATCH 1/2] Use passing by-value in gmtime, mktime

Per the recommendation of the now-removed FIXME.
---
 src/libextra/time.rs    | 17 ++++++++---------
 src/rt/rust_builtin.cpp | 12 ++++++------
 2 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/src/libextra/time.rs b/src/libextra/time.rs
index 8a0d50ed52c71..2b43291c28a82 100644
--- a/src/libextra/time.rs
+++ b/src/libextra/time.rs
@@ -22,11 +22,11 @@ pub mod rustrt {
         pub unsafe fn precise_time_ns(ns: &mut u64);
 
         pub unsafe fn rust_tzset();
-        // FIXME: The i64 values can be passed by-val when #2064 is fixed.
+
         pub unsafe fn rust_gmtime(sec: i64, nsec: i32, result: &mut Tm);
         pub unsafe fn rust_localtime(sec: i64, nsec: i32, result: &mut Tm);
-        pub unsafe fn rust_timegm(tm: &Tm, sec: &mut i64);
-        pub unsafe fn rust_mktime(tm: &Tm, sec: &mut i64);
+        pub unsafe fn rust_timegm(tm: &Tm) -> i64;
+        pub unsafe fn rust_mktime(tm: &Tm) -> i64;
     }
 }
 
@@ -177,12 +177,11 @@ pub impl Tm {
     /// Convert time to the seconds from January 1, 1970
     fn to_timespec(&self) -> Timespec {
         unsafe {
-            let mut sec = 0i64;
-            if self.tm_gmtoff == 0_i32 {
-                rustrt::rust_timegm(self, &mut sec);
-            } else {
-                rustrt::rust_mktime(self, &mut sec);
-            }
+            let sec = match self.tm_gmtoff {
+                0_i32 => rustrt::rust_timegm(self),
+                _     => rustrt::rust_mktime(self)
+            };
+
             Timespec::new(sec, self.tm_nsec)
         }
     }
diff --git a/src/rt/rust_builtin.cpp b/src/rt/rust_builtin.cpp
index a2f253550af16..b0a46d2ac2967 100644
--- a/src/rt/rust_builtin.cpp
+++ b/src/rt/rust_builtin.cpp
@@ -459,18 +459,18 @@ rust_localtime(int64_t sec, int32_t nsec, rust_tm *timeptr) {
     tm_to_rust_tm(&tm, timeptr, gmtoff, zone, nsec);
 }
 
-extern "C" CDECL void
-rust_timegm(rust_tm* timeptr, int64_t *out) {
+extern "C" CDECL int64_t
+rust_timegm(rust_tm* timeptr) {
     tm t;
     rust_tm_to_tm(timeptr, &t);
-    *out = TIMEGM(&t);
+    return TIMEGM(&t);
 }
 
-extern "C" CDECL void
-rust_mktime(rust_tm* timeptr, int64_t *out) {
+extern "C" CDECL int64_t
+rust_mktime(rust_tm* timeptr) {
     tm t;
     rust_tm_to_tm(timeptr, &t);
-    *out = mktime(&t);
+    return mktime(&t);
 }
 
 extern "C" CDECL rust_sched_id

From c6f3577f72f752ad2d13fa081fe7b36efbe3de84 Mon Sep 17 00:00:00 2001
From: Daniel Farina <daniel@fdr.io>
Date: Mon, 27 May 2013 09:55:42 -0700
Subject: [PATCH 2/2] Increment copyright year of time.rs

---
 src/libextra/time.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/libextra/time.rs b/src/libextra/time.rs
index 2b43291c28a82..eb7955bfa8b62 100644
--- a/src/libextra/time.rs
+++ b/src/libextra/time.rs
@@ -1,4 +1,4 @@
-// Copyright 2012 The Rust Project Developers. See the COPYRIGHT
+// Copyright 2012-2013 The Rust Project Developers. See the COPYRIGHT
 // file at the top-level directory of this distribution and at
 // http://rust-lang.org/COPYRIGHT.
 //