Browse Source

fix vet warnings

Simon Cruanes 10 months ago
parent
commit
a1349d8776
2 changed files with 5 additions and 3 deletions
  1. 3 1
      core/time/rfc3339.odin
  2. 2 2
      tests/core/time/test_core_time.odin

+ 3 - 1
core/time/rfc3339.odin

@@ -199,6 +199,8 @@ The boolean `ok` is false if the `time` is not a valid datetime, or if allocatin
 - `include_nanos`: whether to include nanoseconds in the result.
 */
 time_to_rfc3339 :: proc(time: Time, utc_offset : int = 0, include_nanos := true, allocator := context.allocator) -> (res: string, ok: bool) {
+	utc_offset := utc_offset
+
 	// convert to datetime
 	datetime := time_to_datetime(time) or_return
 
@@ -276,7 +278,7 @@ time_to_rfc3339 :: proc(time: Time, utc_offset : int = 0, include_nanos := true,
 	} else {
 		temp_string[offset] = utc_offset > 0 ? '+' : '-'
 		offset += 1
-		utc_offset := abs(utc_offset)
+		utc_offset = abs(utc_offset)
 		print_as_fixed_int(temp_string[:], &offset, 2, i64(utc_offset / 60))
 		temp_string[offset] = ':'
 		offset += 1

+ 2 - 2
tests/core/time/test_core_time.odin

@@ -202,7 +202,7 @@ test_print_rfc3339 :: proc(t: ^testing.T) {
 		printed: string,
 		time: i64,
 		utc_offset: int,
-	};
+	}
 
 	tests :: [?]TestCase {
 		{"1985-04-12T23:20:50.52Z",      	482196050520000000,  	0},
@@ -210,7 +210,7 @@ test_print_rfc3339 :: proc(t: ^testing.T) {
 		{"1996-12-19T16:39:57-08:00",    	851013597000000000,  	-480},
 		{"1996-12-20T00:39:57Z",         	851042397000000000,  	0},
 		{"1937-01-01T12:00:27.87+00:20", 	-1041335972130000000,	+20},
-	};
+	}
 
 	for test in tests {
 		timestamp := time.Time { _nsec = test.time }