Simpler error return value for collatz-conjecture

exercises/practice/collatz-conjecture/.meta/collatz_conjecture_example.odin

@@ -1,20 +1,15 @@
 package collatz_conjecture
 
-Error :: enum {
-	None,
-	IllegalArgument,
-}
-
-steps :: proc(start: int) -> (int, Error) {
-	if (start <= 0) do return 0, .IllegalArgument
+steps :: proc(start: int) -> (result: int, ok: bool) {
+	if (start <= 0) do return 0, false
 
 	n := start
-	result := 0
+	result = 0
 
 	for n > 1 {
 		n = n / 2 if n % 2 == 0 else 3 * n + 1
 		result += 1
 	}
 
-	return result, .None
+	return result, true
 }

exercises/practice/collatz-conjecture/collatz_conjecture.odin

@@ -1,8 +1,6 @@
 package collatz_conjecture
 
-Error :: enum {} // Please inspect the tests to see which error states to enumerate here.
-
 // Returns the number of steps to get to a value of 1.
-steps :: proc(start: int) -> (int, Error) {
+steps :: proc(start: int) -> (result: int, ok: bool) {
 	#panic("Please implement the `steps` procedure.")
 }

exercises/practice/collatz-conjecture/collatz_conjecture_test.odin

@@ -9,35 +9,35 @@ import "core:testing"
 
 @(test)
 test_zero_steps_for_one :: proc(t: ^testing.T) {
-	s, e := steps(1)
+	s, ok := steps(1)
 	testing.expect_value(t, s, 0)
-	testing.expect_value(t, e, Error.None)
+	testing.expect(t, ok)
 }
 
 @(test)
 test_divide_if_even :: proc(t: ^testing.T) {
-	s, e := steps(16)
+	s, ok := steps(16)
 	testing.expect_value(t, s, 4)
-	testing.expect_value(t, e, Error.None)
+	testing.expect(t, ok)
 }
 
 @(test)
 test_even_and_odd_steps :: proc(t: ^testing.T) {
-	s, e := steps(12)
+	s, ok := steps(12)
 	testing.expect_value(t, s, 9)
-	testing.expect_value(t, e, Error.None)
+	testing.expect(t, ok)
 }
 
 @(test)
 test_large_number_of_even_and_odd_steps :: proc(t: ^testing.T) {
-	s, e := steps(1_000_000)
+	s, ok := steps(1_000_000)
 	testing.expect_value(t, s, 152)
-	testing.expect_value(t, e, Error.None)
+	testing.expect(t, ok)
 }
 
 @(test)
 test_zero_is_an_error :: proc(t: ^testing.T) {
-	s, e := steps(0)
+	s, ok := steps(0)
 	testing.expect_value(t, s, 0)
-	testing.expect_value(t, e, Error.IllegalArgument)
+	testing.expect(t, !ok)
 }