Skip to content

Commit

Permalink
Speed up the Eq instance for TArray
Browse files Browse the repository at this point in the history
We don't actually have to check all the elements to see that
two `TArray`s are equal.
  • Loading branch information
treeowl committed Sep 16, 2020
1 parent aa79c92 commit 7e444a1
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion Control/Concurrent/STM/TArray.hs
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,18 @@ import Control.Sequential.STM (STM)
-- but it may be replaced by a more efficient implementation in the future
-- (the interface will remain the same, however).
--
newtype TArray i e = TArray (Array i (TVar e)) deriving (Eq, Typeable)
newtype TArray i e = TArray (Array i (TVar e)) deriving (Typeable)

-- There are no provisions for moving/copying TVars between TArrays.
-- Therefore, two TArrays are equal if and only if they are both empty or are
-- actually the same array in memory. We have no safe operations for checking
-- that directly (though in practice we could use `unsafeCoerce#` with
-- `sameMutableArray#`). So instead we take a quick look at the array sizes and
-- then decide based on the first TVar of each.
instance Eq (TArray i e) where
TArray t1 == TArray t2
= numElements t1 == numElements t2
&& (numElements t1 == 0 || unsafeAt t1 0 == unsafeAt t2 0)

instance MArray TArray e STM where
getBounds (TArray a) = return (bounds a)
Expand Down

0 comments on commit 7e444a1

Please sign in to comment.