Use hash(into:) for Rational: Hashable conformance (#34)

* Use hash(into:) for Rational: Hashable conformance * Refine doc comment
dn-m · Jun 27, 2019 · 0bf8de5 · 0bf8de5
1 parent 47887e0
commit 0bf8de5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/Package.resolved b/Package.resolved
@@ -14,9 +14,9 @@
         "package": "Structure",
         "repositoryURL": "https://github.com/dn-m/Structure",
         "state": {
-          "branch": "swift-5",
-          "revision": "8f67514cba950f60ee992a8d0996d4f11d9654b0",
-          "version": null
+          "branch": null,
+          "revision": "fc7d6a9dbaeb9c750988a6a142d5f67e395ddf69",
+          "version": "0.23.0"
         }
       }
     ]

diff --git a/Sources/Math/Rational.swift b/Sources/Math/Rational.swift
@@ -292,8 +292,11 @@ extension Rational {
     // MARK: - Hashable
 
     /// Hash value.
-    public var hashValue: Int {
-        return floatValue.hashValue
+    ///
+    /// - Note: `Rational` values are hashed by their `floatValue`. That is to say that `1/3` and `2/6` are hashed
+    /// equivalently.
+    public func hash(into hasher: inout Hasher) {
+        hasher.combine(floatValue)
     }
 }