@@ -1,222 +1,108 @@
1{-# LANGUAGE CPP #-}
1{-# LANGUAGE CPP #-}
2−{-# LANGUAGE ConstraintKinds #-}
3{-# LANGUAGE FlexibleContexts #-}
2{-# LANGUAGE FlexibleContexts #-}
4{-# LANGUAGE FlexibleInstances #-}
3{-# LANGUAGE FlexibleInstances #-}
5{-# LANGUAGE GADTs #-}
4{-# LANGUAGE GADTs #-}
6−{-# LANGUAGE InstanceSigs #-}
7{-# LANGUAGE LambdaCase #-}
5{-# LANGUAGE LambdaCase #-}
8{-# LANGUAGE MultiParamTypeClasses #-}
6{-# LANGUAGE MultiParamTypeClasses #-}
9{-# LANGUAGE PatternGuards #-}
7{-# LANGUAGE PatternGuards #-}
10{-# LANGUAGE PolyKinds #-}
8{-# LANGUAGE PolyKinds #-}
11{-# LANGUAGE Rank2Types #-}
9{-# LANGUAGE Rank2Types #-}
12{-# LANGUAGE ScopedTypeVariables #-}
10{-# LANGUAGE ScopedTypeVariables #-}
13−{-# LANGUAGE StandaloneDeriving #-}
14{-# LANGUAGE TypeFamilies #-}
11{-# LANGUAGE TypeFamilies #-}
15{-# LANGUAGE UndecidableInstances #-}
12{-# LANGUAGE UndecidableInstances #-}
16−{-# LANGUAGE UndecidableSuperClasses #-}
18-- |Module containing @'PatchDMapWithMove' k v@ and associated functions, which represents a 'Patch' to a @'DMap' k v@ which can insert, update, delete, and
14-- |Module containing @'PatchDMapWithMove' k v@ and associated functions, which represents a 'Patch' to a @'DMap' k v@ which can insert, update, delete, and
19-- move values between keys.
15-- move values between keys.
20module Data.Patch.DMapWithMove where
16module Data.Patch.DMapWithMove where
22−import qualified Control.Category as Cat
18+import Data.Patch.Class
23−--import qualified Control.Category.DecidablyEmpty as Cat
19+import Data.Patch.MapWithMove (PatchMapWithMove (..))
20+import qualified Data.Patch.MapWithMove as MapWithMove
24−import Data.Constraint.Extras (Has')
22+import Data.Constraint.Extras
25import Data.Dependent.Map (DMap)
23import Data.Dependent.Map (DMap)
26import Data.Dependent.Sum (DSum (..))
24import Data.Dependent.Sum (DSum (..))
27import qualified Data.Dependent.Map as DMap
25import qualified Data.Dependent.Map as DMap
28−import Data.Functor.Constant (Constant (..))
26+import Data.Functor.Constant
29import Data.Functor.Misc
27import Data.Functor.Misc
30− ( Const2 (..), Proxy3 (..)
34−import Data.Functor.Product (Product (..))
28+import Data.Functor.Product
35import Data.GADT.Compare (GEq (..), GCompare (..))
29import Data.GADT.Compare (GEq (..), GCompare (..))
36−import Data.GADT.Show (GRead, GShow, gshow)
30+import Data.GADT.Show (GShow, gshow)
37import qualified Data.Map as Map
31import qualified Data.Map as Map
38import Data.Maybe
32import Data.Maybe
39−import Data.Semigroup (Semigroup (..), (<>))
33+import Data.Monoid.DecidablyEmpty
40−import Data.Semigroupoid as Cat
34+#if !MIN_VERSION_base(4,11,0)
41−import Data.Some (Some(Some))
42−import Data.Proxy (Proxy (..))
35+import Data.Semigroup (Semigroup (..))
43−import Data.These (These (..))
36+#endif
45−import Data.Patch.Class
37+import Data.Some (Some, mkSome)
46− ( Patch (..), PatchHet (..)
38+import Data.These
47− , PatchHet2 (..), PatchSource1, PatchTarget1
39+
50−import Data.Patch.MapWithMove (PatchMapWithMove (..))
51−import qualified Data.Patch.MapWithMove as MapWithMove
53−-- | Like 'PatchMapWithMove', but for 'DMap'. Each key carries a 'NodeInfo'
54−-- which describes how it will be changed by the patch and connects move sources
40+-- | Like 'PatchMapWithMove', but for 'DMap'. Each key carries a 'NodeInfo' which describes how it will be changed by the patch and connects move sources and
55−-- and destinations.
41+-- destinations.
57-- Invariants:
43-- Invariants:
59−-- * A key should not move to itself.
45+-- * A key should not move to itself.
61−-- * A move should always be represented with both the destination key (as a
46+-- * A move should always be represented with both the destination key (as a 'From_Move') and the source key (as a @'ComposeMaybe' ('Just' destination)@)
62−-- 'From_Move') and the source key (as a @'ComposeMaybe' ('Just'
64newtype PatchDMapWithMove k v = PatchDMapWithMove (DMap k (NodeInfo k v))
47newtype PatchDMapWithMove k v = PatchDMapWithMove (DMap k (NodeInfo k v))
66−--deriving instance ( GShow k
49+-- It won't let me derive for some reason
68−-- , Has' Show k (PatchTarget1 p)
69−-- ) => Show (PatchDMapWithMove k p)
70−--deriving instance ( GRead k
72−-- , Has' Read k (PatchTarget1 p)
73−-- ) => Read (PatchDMapWithMove k p)
74−--deriving instance ( GEq k
76−-- , Has' Eq k (PatchTarget1 p)
77−-- ) => Eq (PatchDMapWithMove k p)
78−--deriving instance ( GCompare k
50+instance GCompare k => DecidablyEmpty (PatchDMapWithMove k v) where
79−-- , HasZip Ord k p
51+ isEmpty (PatchDMapWithMove m) = DMap.null m
80−-- , Has' Ord k (PatchTarget1 p)
52+
81−-- ) => Ord (PatchDMapWithMove k p)
83−-- | Structure which represents what changes apply to a particular key.
84−-- @_nodeInfo_from@ specifies what happens to this key, and in particular what
53+-- |Structure which represents what changes apply to a particular key. @_nodeInfo_from@ specifies what happens to this key, and in particular what other key
85−-- other key the current key is moving from, while @_nodeInfo_to@ specifies what
54+-- the current key is moving from, while @_nodeInfo_to@ specifies what key the current key is moving to if involved in a move.
86−-- key the current key is moving to if involved in a move.
87−data NodeInfo k p a = NodeInfo
55+data NodeInfo k v a = NodeInfo
88− { _nodeInfo_from :: !(From k p a)
56+ { _nodeInfo_from :: !(From k v a)
89− -- ^ Change applying to the current key, be it an insert, move, or delete.
57+ -- ^Change applying to the current key, be it an insert, move, or delete.
90− , _nodeInfo_to :: !(To k p a)
58+ , _nodeInfo_to :: !(To k a)
91− -- ^ Where this key is moving to, if involved in a move. Should only be
59+ -- ^Where this key is moving to, if involved in a move. Should only be @ComposeMaybe (Just k)@ when there is a corresponding 'From_Move'.
92− -- @ComposeMaybe (Just k)@ when there is a corresponding 'From_Move'.
95−--deriving instance ( Show (k a)
61+ deriving (Show)
97−-- , Show (PatchTarget1 p a)
98−-- ) => Show (NodeInfo k p a)
99−--deriving instance ( Read (k a)
101−-- , Read (PatchTarget1 p a)
102−-- ) => Read (NodeInfo k p a)
103−--deriving instance ( Eq (k a)
105−-- , Eq (PatchTarget1 p a)
106−-- ) => Eq (NodeInfo k p a)
107−--deriving instance ( Ord (k a)
109−-- , Ord (PatchTarget1 p a)
110−-- ) => Ord (NodeInfo k p a)
112−-- | Structure describing a particular change to a key, be it inserting a new
63+-- |Structure describing a particular change to a key, be it inserting a new key (@From_Insert@), updating an existing key (@From_Insert@ again), deleting a
113−-- key (@From_Insert@), updating an existing key (@From_Insert@ again), deleting
114−-- a key (@From_Delete@), or moving a key (@From_Move@).
64+-- key (@From_Delete@), or moving a key (@From_Move@).
116−-- This type isn't used directly as the from field patch, but is instead wrapped
117−-- in an existential. However, it is nice to be able to reason about this in
118−-- isolation as it is itself a @Semigroupoid@ when the underlying patch is.
119−data From (k :: a -> *) (p :: a -> a -> *) :: a -> * where
65+data From (k :: a -> *) (v :: a -> *) :: a -> * where
120− -- | Insert a new or update an existing key with the given value @PatchTarget1
66+ -- |Insert a new or update an existing key with the given value @v a@
122− From_Insert :: PatchTarget1 p to -> From k p to
67+ From_Insert :: v a -> From k v a
123− -- | Delete the existing key
68+ -- |Delete the existing key
124− From_Delete :: From k p to
69+ From_Delete :: From k v a
125− -- | Move the value from the given key @k a@ to this key. The source key
126− -- should also have an entry in the patch giving the current key as
70+ -- |Move the value from the given key @k a@ to this key. The source key should also have an entry in the patch giving the current key as @_nodeInfo_to@,
127− -- @_nodeInfo_to@, usually but not necessarily with @From_Delete@.
71+ -- usually but not necessarily with @From_Delete@.
128− From_Move :: !(DSum k (Flip p to)) -> From k p to
72+ From_Move :: !(k a) -> From k v a
129−
73+ deriving (Show, Read, Eq, Ord)
130−deriving instance ( Show (k a), GShow k
74+
131− , Has' Show k (Flip p a)
75+-- |Type alias for the "to" part of a 'NodeInfo'. @'ComposeMaybe' ('Just' k)@ means the key is moving to another key, @ComposeMaybe Nothing@ for any other
132− , Show (PatchTarget1 p a)
76+-- operation.
133− ) => Show (From k p a)
77+type To = ComposeMaybe
134−deriving instance ( Read (k a), GRead k
135− , Has' Read k (Flip p a)
136− , Read (PatchTarget1 p a)
137− ) => Read (From k p a)
138−deriving instance ( GEq k
139− , Has' Eq k (Flip p a)
140− , Eq (PatchTarget1 p a)
141− ) => Eq (From k p a)
142−deriving instance ( GCompare k
143− , Has' Eq k (Flip p a) -- superclass bug
144− , Has' Ord k (Flip p a)
145− , Ord (PatchTarget1 p a)
146− ) => Ord (From k p a)
148−newtype Flip p to from = Flip (p from to)
150−instance Cat.Category p => Cat.Category (Flip (p :: k -> k -> *)) where
152− Flip y . Flip x = Flip $ x Cat.. y
154−-- | The "to" part of a 'NodeInfo'. Rather than be built out of @From@ like @From@
155−-- is, we store just the information necessary to compose a @To@ and @From@ like
156−-- @oLocal@ composes two @From@s.
157−data To (k :: a -> *) (p :: a -> a -> *) :: a -> * where
158− -- | Delete or leave in place
159− To_NonMove :: To k p from
160− -- | Move the value from the given key @k a@ to this key. The target key
161− -- should also have an entry in the patch giving the current key in
162− -- @_nodeInfo_from@, usually but not necessarily with @To_Delete@.
163− To_Move :: !(DSum k (p from)) -> To k p from
165−deriving instance ( Show (k a), GShow k
167− , Show (PatchTarget1 p a)
168− ) => Show (To k p a)
169−deriving instance ( Read (k a), GRead k
171− , Read (PatchTarget1 p a)
172− ) => Read (To k p a)
173−deriving instance ( GEq k
175− , Eq (PatchTarget1 p a)
177−deriving instance ( GCompare k
178− , Has' Eq k (p a) -- superclass bug
180− , Ord (PatchTarget1 p a)
183-- |Test whether a 'PatchDMapWithMove' satisfies its invariants.
79-- |Test whether a 'PatchDMapWithMove' satisfies its invariants.
184−validPatchDMapWithMove
80+validPatchDMapWithMove :: forall k v. (GCompare k, GShow k) => DMap k (NodeInfo k v) -> Bool
186− . (GCompare k, GShow k)
187− => DMap k (NodeInfo k v)
189validPatchDMapWithMove = not . null . validationErrorsForPatchDMapWithMove
81validPatchDMapWithMove = not . null . validationErrorsForPatchDMapWithMove
191-- |Enumerate what reasons a 'PatchDMapWithMove' doesn't satisfy its invariants, returning @[]@ if it's valid.
83-- |Enumerate what reasons a 'PatchDMapWithMove' doesn't satisfy its invariants, returning @[]@ if it's valid.
192−validationErrorsForPatchDMapWithMove
84+validationErrorsForPatchDMapWithMove :: forall k v. (GCompare k, GShow k) => DMap k (NodeInfo k v) -> [String]
194− . (GCompare k, GShow k)
195− => DMap k (NodeInfo k v)
197validationErrorsForPatchDMapWithMove m =
85validationErrorsForPatchDMapWithMove m =
198 noSelfMoves <> movesBalanced
86 noSelfMoves <> movesBalanced
200 noSelfMoves = mapMaybe selfMove . DMap.toAscList $ m
88 noSelfMoves = mapMaybe selfMove . DMap.toAscList $ m
201− selfMove (dst :=> NodeInfo (From_Move (src :=> _)) _)
202− | Just _ <- dst `geq` src = Just $ "self move of key " <> gshow src <> " at destination side"
89+ selfMove (dst :=> NodeInfo (From_Move src) _) | Just _ <- dst `geq` src = Just $ "self move of key " <> gshow src <> " at destination side"
203− selfMove (src :=> NodeInfo _ (To_Move (dst :=> _)))
204− | Just _ <- src `geq` dst = Just $ "self move of key " <> gshow dst <> " at source side"
90+ selfMove (src :=> NodeInfo _ (ComposeMaybe (Just dst))) | Just _ <- src `geq` dst = Just $ "self move of key " <> gshow dst <> " at source side"
205 selfMove _ = Nothing
91 selfMove _ = Nothing
206 movesBalanced = mapMaybe unbalancedMove . DMap.toAscList $ m
92 movesBalanced = mapMaybe unbalancedMove . DMap.toAscList $ m
207− unbalancedMove (dst :=> NodeInfo (From_Move (src :=> _)) _) =
93+ unbalancedMove (dst :=> NodeInfo (From_Move src) _) =
208 case DMap.lookup src m of
94 case DMap.lookup src m of
209 Nothing -> Just $ "unbalanced move at destination key " <> gshow dst <> " supposedly from " <> gshow src <> " but source key is not in the patch"
95 Nothing -> Just $ "unbalanced move at destination key " <> gshow dst <> " supposedly from " <> gshow src <> " but source key is not in the patch"
210− Just (NodeInfo _ (To_Move (dst' :=> _))) ->
96+ Just (NodeInfo _ (ComposeMaybe (Just dst'))) ->
211 if isNothing (dst' `geq` dst)
97 if isNothing (dst' `geq` dst)
212 then Just $ "unbalanced move at destination key " <> gshow dst <> " from " <> gshow src <> " is going to " <> gshow dst' <> " instead"
98 then Just $ "unbalanced move at destination key " <> gshow dst <> " from " <> gshow src <> " is going to " <> gshow dst' <> " instead"
213 else Nothing
99 else Nothing
215 Just $ "unbalanced move at destination key " <> gshow dst <> " supposedly from " <> gshow src <> " but source key has no move to key"
101 Just $ "unbalanced move at destination key " <> gshow dst <> " supposedly from " <> gshow src <> " but source key has no move to key"
216− unbalancedMove (src :=> NodeInfo _ (To_Move (dst :=> _))) =
102+ unbalancedMove (src :=> NodeInfo _ (ComposeMaybe (Just dst))) =
217 case DMap.lookup dst m of
103 case DMap.lookup dst m of
218 Nothing -> Just $ " unbalanced move at source key " <> gshow src <> " supposedly going to " <> gshow dst <> " but destination key is not in the patch"
104 Nothing -> Just $ " unbalanced move at source key " <> gshow src <> " supposedly going to " <> gshow dst <> " but destination key is not in the patch"
219− Just (NodeInfo (From_Move (src' :=> _)) _) ->
105+ Just (NodeInfo (From_Move src') _) ->
220 if isNothing (src' `geq` src)
106 if isNothing (src' `geq` src)
221 then Just $ "unbalanced move at source key " <> gshow src <> " to " <> gshow dst <> " is coming from " <> gshow src' <> " instead"
107 then Just $ "unbalanced move at source key " <> gshow src <> " to " <> gshow dst <> " is coming from " <> gshow src' <> " instead"
222 else Nothing
108 else Nothing
@@ -225,49 +111,35 @@ validationErrorsForPatchDMapWithMove m =
225 Just $ "unbalanced move at source key " <> gshow src <> " supposedly going to " <> gshow dst <> " but destination key is not moving"
111 Just $ "unbalanced move at source key " <> gshow src <> " supposedly going to " <> gshow dst <> " but destination key is not moving"
226 unbalancedMove _ = Nothing
112 unbalancedMove _ = Nothing
114+-- |Test whether two @'PatchDMapWithMove' k v@ contain the same patch operations.
115+instance (GEq k, Has' Eq k (NodeInfo k v)) => Eq (PatchDMapWithMove k v) where
116+ PatchDMapWithMove a == PatchDMapWithMove b = a == b
228-- |Higher kinded 2-tuple, identical to @Data.Functor.Product@ from base ≥ 4.9
118-- |Higher kinded 2-tuple, identical to @Data.Functor.Product@ from base ≥ 4.9
229data Pair1 f g a = Pair1 (f a) (g a)
119data Pair1 f g a = Pair1 (f a) (g a)
231-- |Helper data structure used for composing patches using the monoid instance.
121-- |Helper data structure used for composing patches using the monoid instance.
232−data Fixup k p a
122+data Fixup k v a
233 = Fixup_Delete
123 = Fixup_Delete
234− | Fixup_Update (These (From k p a) (To k p a))
124+ | Fixup_Update (These (From k v a) (To k a))
236−-- | Compose patches having the same effect as applying the patches in turn:
126+-- |Compose patches having the same effect as applying the patches in turn: @'applyAlways' (p <> q) == 'applyAlways' p . 'applyAlways' q@
237−-- @'applyAlways' (p <> q) == 'applyAlways' p . 'applyAlways' q@
238−instance ( GCompare k
239− , Cat.Semigroupoid p
240− -- , Cat.DecidablyEmpty p
242− , PatchSource1 p ~ PatchTarget1 p
243− ) => Semigroup (PatchDMapWithMove k p) where
127+instance GCompare k => Semigroup (PatchDMapWithMove k v) where
244 PatchDMapWithMove ma <> PatchDMapWithMove mb = PatchDMapWithMove m
128 PatchDMapWithMove ma <> PatchDMapWithMove mb = PatchDMapWithMove m
246− connections :: [DSum k (Pair1 (To k p) (From k p))]
247− connections = DMap.toList $ DMap.intersectionWithKey
248− (\_ a b -> Pair1 (_nodeInfo_to a) (_nodeInfo_from b))
130+ connections = DMap.toList $ DMap.intersectionWithKey (\_ a b -> Pair1 (_nodeInfo_to a) (_nodeInfo_from b)) ma mb
251− h :: DSum k (Pair1 (To k p) (From k p)) -> [DSum k (Fixup k p)]
131+ h :: DSum k (Pair1 (ComposeMaybe k) (From k v)) -> [DSum k (Fixup k v)]
252− h ((_ :: k between) :=> Pair1 editAfter editBefore) = case (editAfter, editBefore) of
132+ h (_ :=> Pair1 (ComposeMaybe mToAfter) editBefore) = case (mToAfter, editBefore) of
253− (To_Move ((toAfter :: k after) :=> p1), From_Move ((fromBefore :: k before) :=> Flip p0)) ->
133+ (Just toAfter, From_Move fromBefore)
254− --case toAfter `geq` fromBefore of
134+ | isJust $ fromBefore `geq` toAfter
255− -- Just Refl | Just Refl <- Cat.isId p0 ->
256− -- [ toAfter :=> Fixup_Delete ]
135+ -> [toAfter :=> Fixup_Delete]
257− -- _ ->
136+ | otherwise
258− [ toAfter :=> Fixup_Update (This $ From_Move $ fromBefore :=> (Flip $ p1 `o` p0))
137+ -> [ toAfter :=> Fixup_Update (This editBefore)
259− , fromBefore :=> Fixup_Update (That $ To_Move $ toAfter :=> (p1 `o` p0))
138+ , fromBefore :=> Fixup_Update (That (ComposeMaybe mToAfter))
261− (To_NonMove, From_Move (fromBefore :=> _)) ->
262− -- The item is destroyed in the second patch, so indicate that it is
140+ (Nothing, From_Move fromBefore) -> [fromBefore :=> Fixup_Update (That (ComposeMaybe mToAfter))] -- The item is destroyed in the second patch, so indicate that it is destroyed in the source map
263− -- destroyed in the source map
264− [fromBefore :=> Fixup_Update (That To_NonMove)]
265− (To_Move (toAfter :=> p), From_Insert val) ->
266− [toAfter :=> Fixup_Update (This $ From_Insert $ applyAlwaysHet2 p val)]
267− (To_Move (toAfter :=> _), From_Delete) ->
268− [toAfter :=> Fixup_Update (This From_Delete)]
141+ (Just toAfter, _) -> [toAfter :=> Fixup_Update (This editBefore)]
269− (To_NonMove, _) ->
142+ (Nothing, _) -> []
271 mergeFixups _ Fixup_Delete Fixup_Delete = Fixup_Delete
143 mergeFixups _ Fixup_Delete Fixup_Delete = Fixup_Delete
272 mergeFixups _ (Fixup_Update a) (Fixup_Update b)
144 mergeFixups _ (Fixup_Update a) (Fixup_Update b)
273 | This x <- a, That y <- b
145 | This x <- a, That y <- b
@@ -298,19 +170,13 @@ instance ( GCompare k
298 These _ b -> Just b
170 These _ b -> Just b
299 That b -> Just b
171 That b -> Just b
301−-- | Compose patches having the same effect as applying the patches in turn:
173+-- |Compose patches having the same effect as applying the patches in turn: @'applyAlways' (p <> q) == 'applyAlways' p . 'applyAlways' q@
302−-- @'applyAlways' (p <> q) == 'applyAlways' p . 'applyAlways' q@
303−instance ( GCompare k
304− , Cat.Semigroupoid p
305− -- , DecidablyEmpty p
307− , PatchSource1 p ~ PatchTarget1 p
308− ) => Monoid (PatchDMapWithMove k p) where
174+instance GCompare k => Monoid (PatchDMapWithMove k v) where
309 mempty = PatchDMapWithMove mempty
175 mempty = PatchDMapWithMove mempty
310 mappend = (<>)
176 mappend = (<>)
313−mappendPatchDMapWithMoveSlow :: forall k v. (ShowTag k v, GCompare k) => PatchDMapWithMove k v -> PatchDMapWithMove k v -> PatchDMapWithMove k v
179+mappendPatchDMapWithMoveSlow :: forall k v. (ShowTag k v, GCompare k) => PatchDMapWithMove k v -> PatchDMapWithMove k v -> PatchDMapWithMove k v
314PatchDMapWithMove dstAfter srcAfter `mappendPatchDMapWithMoveSlow` PatchDMapWithMove dstBefore srcBefore = PatchDMapWithMove dst src
180PatchDMapWithMove dstAfter srcAfter `mappendPatchDMapWithMoveSlow` PatchDMapWithMove dstBefore srcBefore = PatchDMapWithMove dst src
316 getDstAction k m = fromMaybe (From_Move k) $ DMap.lookup k m -- Any key that isn't present is treated as that key moving to itself
182 getDstAction k m = fromMaybe (From_Move k) $ DMap.lookup k m -- Any key that isn't present is treated as that key moving to itself
@@ -331,51 +197,44 @@ PatchDMapWithMove dstAfter srcAfter `mappendPatchDMapWithMoveSlow` PatchDMapWith
331 src = DMap.mapMaybeWithKey g $ DMap.union srcAfter srcBefore
197 src = DMap.mapMaybeWithKey g $ DMap.union srcAfter srcBefore
334−-- | Make a @'PatchDMapWithMove' k v@ which has the effect of inserting or
200+-- |Make a @'PatchDMapWithMove' k v@ which has the effect of inserting or updating a value @v a@ to the given key @k a@, like 'DMap.insert'.
335−-- updating a value @PatchTarget1 p a@ to the given key @k a@, like
337−insertDMapKey :: k a -> PatchTarget1 p a -> PatchDMapWithMove k p
201+insertDMapKey :: k a -> v a -> PatchDMapWithMove k v
338insertDMapKey k v =
202insertDMapKey k v =
339− PatchDMapWithMove . DMap.singleton k $ NodeInfo (From_Insert v) To_NonMove
203+ PatchDMapWithMove . DMap.singleton k $ NodeInfo (From_Insert v) (ComposeMaybe Nothing)
341−-- | Make a @'PatchDMapWithMove' k v@ which has the effect of moving the value
205+-- |Make a @'PatchDMapWithMove' k v@ which has the effect of moving the value from the first key @k a@ to the second key @k a@, equivalent to:
342−-- from the first key @k a@ to the second key @k a@, equivalent to:
345-- 'DMap.delete' src (maybe dmap ('DMap.insert' dst) (DMap.lookup src dmap))
208-- 'DMap.delete' src (maybe dmap ('DMap.insert' dst) (DMap.lookup src dmap))
349− => k a -> k a -> PatchDMapWithMove k (Proxy3 v)
210+moveDMapKey :: GCompare k => k a -> k a -> PatchDMapWithMove k v
350moveDMapKey src dst = case src `geq` dst of
211moveDMapKey src dst = case src `geq` dst of
351 Nothing -> PatchDMapWithMove $ DMap.fromList
212 Nothing -> PatchDMapWithMove $ DMap.fromList
352− [ dst :=> NodeInfo (From_Move (src :=> Flip Proxy3)) To_NonMove
213+ [ dst :=> NodeInfo (From_Move src) (ComposeMaybe Nothing)
353− , src :=> NodeInfo From_Delete (To_Move $ dst :=> Proxy3)
214+ , src :=> NodeInfo From_Delete (ComposeMaybe $ Just dst)
355− Just _ -> PatchDMapWithMove DMap.empty
216+ Just _ -> mempty
357−-- | Make a @'PatchDMapWithMove' k v@ which has the effect of swapping two keys
218+-- |Make a @'PatchDMapWithMove' k v@ which has the effect of swapping two keys in the mapping, equivalent to:
358−-- in the mapping, equivalent to:
361-- let aMay = DMap.lookup a dmap
221-- let aMay = DMap.lookup a dmap
362-- bMay = DMap.lookup b dmap
222-- bMay = DMap.lookup b dmap
363−-- in maybe id (DMap.insert a) (bMay `mplus` aMay)
223+-- in maybe id (DMap.insert a) (bMay <> aMay)
364−-- . maybe id (DMap.insert b) (aMay `mplus` bMay)
224+-- . maybe id (DMap.insert b) (aMay <> bMay)
365-- . DMap.delete a . DMap.delete b $ dmap
225-- . DMap.delete a . DMap.delete b $ dmap
367−swapDMapKey :: GCompare k => k a -> k a -> PatchDMapWithMove k (Proxy3 v)
227+swapDMapKey :: GCompare k => k a -> k a -> PatchDMapWithMove k v
368swapDMapKey src dst = case src `geq` dst of
228swapDMapKey src dst = case src `geq` dst of
369 Nothing -> PatchDMapWithMove $ DMap.fromList
229 Nothing -> PatchDMapWithMove $ DMap.fromList
370− [ dst :=> NodeInfo (From_Move (src :=> Flip Proxy3)) (To_Move $ src :=> Proxy3)
230+ [ dst :=> NodeInfo (From_Move src) (ComposeMaybe $ Just src)
371− , src :=> NodeInfo (From_Move (dst :=> Flip Proxy3)) (To_Move $ dst :=> Proxy3)
231+ , src :=> NodeInfo (From_Move dst) (ComposeMaybe $ Just dst)
373− Just _ -> PatchDMapWithMove DMap.empty
233+ Just _ -> mempty
375−-- | Make a @'PatchDMapWithMove' k v@ which has the effect of deleting a key in
235+-- |Make a @'PatchDMapWithMove' k v@ which has the effect of deleting a key in the mapping, equivalent to 'DMap.delete'.
376−-- the mapping, equivalent to 'DMap.delete'.
377deleteDMapKey :: k a -> PatchDMapWithMove k v
236deleteDMapKey :: k a -> PatchDMapWithMove k v
378−deleteDMapKey k = PatchDMapWithMove $ DMap.singleton k $ NodeInfo From_Delete To_NonMove
237+deleteDMapKey k = PatchDMapWithMove $ DMap.singleton k $ NodeInfo From_Delete $ ComposeMaybe Nothing
381k1, k2 :: Const2 Int () ()
240k1, k2 :: Const2 Int () ()
@@ -401,20 +260,17 @@ dst (PatchDMapWithMove x _) = x
401src (PatchDMapWithMove _ x) = x
260src (PatchDMapWithMove _ x) = x
404−-- | Extract the 'DMap' representing the patch changes from the
263+-- |Extract the 'DMap' representing the patch changes from the 'PatchDMapWithMove'.
405−-- 'PatchDMapWithMove'.
406unPatchDMapWithMove :: PatchDMapWithMove k v -> DMap k (NodeInfo k v)
264unPatchDMapWithMove :: PatchDMapWithMove k v -> DMap k (NodeInfo k v)
407unPatchDMapWithMove (PatchDMapWithMove p) = p
265unPatchDMapWithMove (PatchDMapWithMove p) = p
409−-- | Wrap a 'DMap' representing patch changes into a 'PatchDMapWithMove',
267+-- |Wrap a 'DMap' representing patch changes into a 'PatchDMapWithMove', without checking any invariants.
410−-- without checking any invariants.
412-- __Warning:__ when using this function, you must ensure that the invariants of 'PatchDMapWithMove' are preserved; they will not be checked.
269-- __Warning:__ when using this function, you must ensure that the invariants of 'PatchDMapWithMove' are preserved; they will not be checked.
413unsafePatchDMapWithMove :: DMap k (NodeInfo k v) -> PatchDMapWithMove k v
270unsafePatchDMapWithMove :: DMap k (NodeInfo k v) -> PatchDMapWithMove k v
414unsafePatchDMapWithMove = PatchDMapWithMove
271unsafePatchDMapWithMove = PatchDMapWithMove
416−-- | Wrap a 'DMap' representing patch changes into a 'PatchDMapWithMove' while
417−-- checking invariants. If the invariants are satisfied, @Right p@ is returned
273+-- |Wrap a 'DMap' representing patch changes into a 'PatchDMapWithMove' while checking invariants. If the invariants are satisfied, @Right p@ is returned
418-- otherwise @Left errors@.
274-- otherwise @Left errors@.
419patchDMapWithMove :: (GCompare k, GShow k) => DMap k (NodeInfo k v) -> Either [String] (PatchDMapWithMove k v)
275patchDMapWithMove :: (GCompare k, GShow k) => DMap k (NodeInfo k v) -> Either [String] (PatchDMapWithMove k v)
420patchDMapWithMove dm =
276patchDMapWithMove dm =
@@ -422,196 +278,94 @@ patchDMapWithMove dm =
422 [] -> Right $ unsafePatchDMapWithMove dm
278 [] -> Right $ unsafePatchDMapWithMove dm
423 errs -> Left errs
279 errs -> Left errs
425−-- | Map a natural transform @v -> v'@ over the given patch, transforming
281+-- |Map a natural transform @v -> v'@ over the given patch, transforming @'PatchDMapWithMove' k v@ into @'PatchDMapWithMove' k v'@.
426−-- @'PatchDMapWithMove' k v@ into @'PatchDMapWithMove' k v'@.
427−mapPatchDMapWithMove
282+mapPatchDMapWithMove :: forall k v v'. (forall a. v a -> v' a) -> PatchDMapWithMove k v -> PatchDMapWithMove k v'
429− . (forall a. PatchTarget1 p a -> PatchTarget1 p' a)
430− -> (forall from to. p from to -> p' from to)
431− -> PatchDMapWithMove k p
432− -> PatchDMapWithMove k p'
433−mapPatchDMapWithMove f g (PatchDMapWithMove m) =
283+mapPatchDMapWithMove f (PatchDMapWithMove p) = PatchDMapWithMove $
434− PatchDMapWithMove $ DMap.map (\ni -> NodeInfo
435− { _nodeInfo_from = h $ _nodeInfo_from ni
284+ DMap.map (\ni -> ni { _nodeInfo_from = g $ _nodeInfo_from ni }) p
436− , _nodeInfo_to = j $ _nodeInfo_to ni
438− where h :: forall a. From k p a -> From k p' a
285+ where g :: forall a. From k v a -> From k v' a
439− h = \case
286+ g = \case
440 From_Insert v -> From_Insert $ f v
287 From_Insert v -> From_Insert $ f v
441 From_Delete -> From_Delete
288 From_Delete -> From_Delete
442− From_Move (k :=> Flip p) -> From_Move $ k :=> Flip (g p)
289+ From_Move k -> From_Move k
443− j :: forall a. To k p a -> To k p' a
290+
445− To_NonMove -> To_NonMove
446− To_Move (k :=> p) -> To_Move $ k :=> g p
448−-- | Traverse an effectful function @forall a. PatchTarget1 p a -> m (v ' a)@
449−-- over the given patch, transforming @'PatchDMapWithMove' k v@ into @m
291+-- |Traverse an effectful function @forall a. v a -> m (v ' a)@ over the given patch, transforming @'PatchDMapWithMove' k v@ into @m ('PatchDMapWithMove' k v')@.
450−-- ('PatchDMapWithMove' k v')@.
451−traversePatchDMapWithMove
292+traversePatchDMapWithMove :: forall m k v v'. Applicative m => (forall a. v a -> m (v' a)) -> PatchDMapWithMove k v -> m (PatchDMapWithMove k v')
454− => (forall a. PatchTarget1 p a -> m (PatchTarget1 p' a))
455− -> (forall from to. p from to -> m (p' from to))
456− -> PatchDMapWithMove k p
457− -> m (PatchDMapWithMove k p')
458−traversePatchDMapWithMove f g = traversePatchDMapWithMoveWithKey
293+traversePatchDMapWithMove f = traversePatchDMapWithMoveWithKey $ const f
462−-- | Map an effectful function @forall a. k a -> PatchTarget1 p a -> m (v ' a)@
463−-- over the given patch, transforming @'PatchDMapWithMove' k v@ into @m
295+-- |Map an effectful function @forall a. k a -> v a -> m (v ' a)@ over the given patch, transforming @'PatchDMapWithMove' k v@ into @m ('PatchDMapWithMove' k v')@.
464−-- ('PatchDMapWithMove' k v')@.
465−traversePatchDMapWithMoveWithKey
296+traversePatchDMapWithMoveWithKey :: forall m k v v'. Applicative m => (forall a. k a -> v a -> m (v' a)) -> PatchDMapWithMove k v -> m (PatchDMapWithMove k v')
468− => (forall a. k a -> PatchTarget1 p a -> m (PatchTarget1 p' a))
469− -> (forall from to. k from -> k to -> p from to -> m (p' from to))
470− -> PatchDMapWithMove k p
471− -> m (PatchDMapWithMove k p')
472−traversePatchDMapWithMoveWithKey f g (PatchDMapWithMove m) =
297+traversePatchDMapWithMoveWithKey f (PatchDMapWithMove p) = PatchDMapWithMove <$> DMap.traverseWithKey (nodeInfoMapFromM . g) p
473− PatchDMapWithMove <$> DMap.traverseWithKey (\k ni -> NodeInfo
474− <$> h k (_nodeInfo_from ni)
475− <*> j k (_nodeInfo_to ni)) m
476− where h :: forall a. k a -> From k p a -> m (From k p' a)
298+ where g :: forall a. k a -> From k v a -> m (From k v' a)
477− h k = \case
299+ g k = \case
478 From_Insert v -> From_Insert <$> f k v
300 From_Insert v -> From_Insert <$> f k v
479 From_Delete -> pure From_Delete
301 From_Delete -> pure From_Delete
480− From_Move (fromKey :=> Flip p) -> From_Move . (fromKey :=>) . Flip <$> g fromKey k p
302+ From_Move fromKey -> pure $ From_Move fromKey
481− j :: forall a. k a -> To k p a -> m (To k p' a)
303+
483− To_NonMove -> pure To_NonMove
484− To_Move (toKey :=> p) -> To_Move . (toKey :=>) <$> g k toKey p
486−-- | Map a function which transforms @'From k PatchTarget1 p a@ into a @'From k
304+-- |Map a function which transforms @'From' k v a@ into a @'From' k v' a@ over a @'NodeInfo' k v a@.
487−-- PatchTarget1 p' a@ over a @'NodeInfo' k PatchTarget1 p a@.
488−nodeInfoMapFrom
305+nodeInfoMapFrom :: (From k v a -> From k v' a) -> NodeInfo k v a -> NodeInfo k v' a
489− :: (From k p a -> From k p' a)
490− -> (To k p a -> To k p' a)
493−nodeInfoMapFrom f g ni = NodeInfo
494− { _nodeInfo_from = f $ _nodeInfo_from ni
306+nodeInfoMapFrom f ni = ni { _nodeInfo_from = f $ _nodeInfo_from ni }
495− , _nodeInfo_to = g $ _nodeInfo_to ni
307+
498−-- | Map an effectful function which transforms @'From k PatchTarget1 p a@ into
308+-- |Map an effectful function which transforms @'From' k v a@ into a @f ('From' k v' a)@ over a @'NodeInfo' k v a@.
499−-- a @f ('From k PatchTarget1 p' a)@ over a @'NodeInfo' k PatchTarget1 p a@.
500−nodeInfoMapFromM
309+nodeInfoMapFromM :: Functor f => (From k v a -> f (From k v' a)) -> NodeInfo k v a -> f (NodeInfo k v' a)
501− :: Applicative f
310+nodeInfoMapFromM f ni = fmap (\result -> ni { _nodeInfo_from = result }) $ f $ _nodeInfo_from ni
502− => (From k p a -> f (From k p' a))
311+
503− -> (To k p a -> f (To k p' a))
505− -> f (NodeInfo k p' a)
506−nodeInfoMapFromM f g ni = NodeInfo
507− <$> f (_nodeInfo_from ni)
508− <*> g (_nodeInfo_to ni)
510−-- | Weaken a 'PatchDMapWithMove' to a 'PatchMapWithMove' by weakening the keys
312+-- |Weaken a 'PatchDMapWithMove' to a 'PatchMapWithMove' by weakening the keys from @k a@ to @'Some' k@ and applying a given weakening function @v a -> v'@ to
511−-- from @k a@ to @'Some' k@ and applying a given weakening function
512−-- @PatchTarget1 p a -> v'@ to values.
313+-- values.
513−weakenPatchDMapWithMoveWith
314+weakenPatchDMapWithMoveWith :: forall k v v'. (forall a. v a -> v') -> PatchDMapWithMove k v -> PatchMapWithMove (Some k) v'
515− . (forall a. PatchTarget1 p a -> PatchTarget p')
516− -> (forall from to. p from to -> p')
517− -> PatchDMapWithMove k p
518− -> PatchMapWithMove (Some k) p'
519−weakenPatchDMapWithMoveWith f g (PatchDMapWithMove m) =
315+weakenPatchDMapWithMoveWith f (PatchDMapWithMove p) = PatchMapWithMove $ weakenDMapWith g p
520− PatchMapWithMove $ weakenDMapWith h m
521− where h :: forall a. NodeInfo k p a -> MapWithMove.NodeInfo (Some k) p'
316+ where g :: forall a. NodeInfo k v a -> MapWithMove.NodeInfo (Some k) v'
522− h ni = MapWithMove.NodeInfo
317+ g ni = MapWithMove.NodeInfo
523 { MapWithMove._nodeInfo_from = case _nodeInfo_from ni of
318 { MapWithMove._nodeInfo_from = case _nodeInfo_from ni of
524 From_Insert v -> MapWithMove.From_Insert $ f v
319 From_Insert v -> MapWithMove.From_Insert $ f v
525 From_Delete -> MapWithMove.From_Delete
320 From_Delete -> MapWithMove.From_Delete
526− From_Move (k :=> Flip p) -> MapWithMove.From_Move (Some k) $ g p
321+ From_Move k -> MapWithMove.From_Move $ mkSome k
527− , MapWithMove._nodeInfo_to = case _nodeInfo_to ni of
322+ , MapWithMove._nodeInfo_to = mkSome <$> getComposeMaybe (_nodeInfo_to ni)
528− To_NonMove -> MapWithMove.To_NonMove
529− To_Move (k :=> p) -> MapWithMove.To_Move (Some k) $ g p
532−-- | "Weaken" a @'PatchDMapWithMove' (Const2 k a) v@ to a @'PatchMapWithMove' k
533−-- v'@. Weaken is in scare quotes because the 'Const2' has already disabled any
325+-- |"Weaken" a @'PatchDMapWithMove' (Const2 k a) v@ to a @'PatchMapWithMove' k v'@. Weaken is in scare quotes because the 'Const2' has already disabled any
534−-- dependency in the typing and all points are already @a@, hence the function
326+-- dependency in the typing and all points are already @a@, hence the function to map each value to @v'@ is not higher rank.
535−-- to map each value to @v'@ is not higher rank.
536−patchDMapWithMoveToPatchMapWithMoveWith
327+patchDMapWithMoveToPatchMapWithMoveWith :: forall k v v' a. (v a -> v') -> PatchDMapWithMove (Const2 k a) v -> PatchMapWithMove k v'
538− . (PatchTarget1 p a -> PatchTarget p')
540− -> PatchDMapWithMove (Const2 k a) p
541− -> PatchMapWithMove k p'
542−patchDMapWithMoveToPatchMapWithMoveWith f g (PatchDMapWithMove m) =
328+patchDMapWithMoveToPatchMapWithMoveWith f (PatchDMapWithMove p) = PatchMapWithMove $ dmapToMapWith g p
543− PatchMapWithMove $ dmapToMapWith h m
544− where h :: NodeInfo (Const2 k a) p a -> MapWithMove.NodeInfo k p'
329+ where g :: NodeInfo (Const2 k a) v a -> MapWithMove.NodeInfo k v'
545− h ni = MapWithMove.NodeInfo
330+ g ni = MapWithMove.NodeInfo
546 { MapWithMove._nodeInfo_from = case _nodeInfo_from ni of
331 { MapWithMove._nodeInfo_from = case _nodeInfo_from ni of
547 From_Insert v -> MapWithMove.From_Insert $ f v
332 From_Insert v -> MapWithMove.From_Insert $ f v
548 From_Delete -> MapWithMove.From_Delete
333 From_Delete -> MapWithMove.From_Delete
549− From_Move (Const2 k :=> Flip p) -> MapWithMove.From_Move k $ g p
334+ From_Move (Const2 k) -> MapWithMove.From_Move k
550− , MapWithMove._nodeInfo_to = case _nodeInfo_to ni of
335+ , MapWithMove._nodeInfo_to = unConst2 <$> getComposeMaybe (_nodeInfo_to ni)
551− To_NonMove -> MapWithMove.To_NonMove
552− To_Move (Const2 k :=> p) -> MapWithMove.To_Move k $ g p
555−-- | "Strengthen" a @'PatchMapWithMove' k v@ into a @'PatchDMapWithMove
338+-- |"Strengthen" a @'PatchMapWithMove' k v@ into a @'PatchDMapWithMove ('Const2' k a)@; that is, turn a non-dependently-typed patch into a dependently typed
556−-- ('Const2' k a)@; that is, turn a non-dependently-typed patch into a
557−-- dependently typed one but which always has a constant key type represented by
339+-- one but which always has a constant key type represented by 'Const2'. Apply the given function to each @v@ to produce a @v' a@.
558−-- 'Const2'. Apply the given function to each @v@ to produce a @PatchTarget1 p'
559−-- a@. Completemented by 'patchDMapWithMoveToPatchMapWithMoveWith'
340+-- Completemented by 'patchDMapWithMoveToPatchMapWithMoveWith'
560−const2PatchDMapWithMoveWith
341+const2PatchDMapWithMoveWith :: forall k v v' a. (v -> v' a) -> PatchMapWithMove k v -> PatchDMapWithMove (Const2 k a) v'
563− -> PatchMapWithMove k (Proxy v)
564− -> PatchDMapWithMove (Const2 k a) (Proxy3 v')
565−const2PatchDMapWithMoveWith f (PatchMapWithMove p) =
566− PatchDMapWithMove $ DMap.fromDistinctAscList $ g <$> Map.toAscList p
342+const2PatchDMapWithMoveWith f (PatchMapWithMove p) = PatchDMapWithMove $ DMap.fromDistinctAscList $ g <$> Map.toAscList p
567− where g :: (k, MapWithMove.NodeInfo k (Proxy v))
568− -> DSum (Const2 k a) (NodeInfo (Const2 k a) (Proxy3 v'))
343+ where g :: (k, MapWithMove.NodeInfo k v) -> DSum (Const2 k a) (NodeInfo (Const2 k a) v')
569 g (k, ni) = Const2 k :=> NodeInfo
344 g (k, ni) = Const2 k :=> NodeInfo
570 { _nodeInfo_from = case MapWithMove._nodeInfo_from ni of
345 { _nodeInfo_from = case MapWithMove._nodeInfo_from ni of
571 MapWithMove.From_Insert v -> From_Insert $ f v
346 MapWithMove.From_Insert v -> From_Insert $ f v
572 MapWithMove.From_Delete -> From_Delete
347 MapWithMove.From_Delete -> From_Delete
573− MapWithMove.From_Move fromKey Proxy -> From_Move $ Const2 fromKey :=> Flip Proxy3
348+ MapWithMove.From_Move fromKey -> From_Move $ Const2 fromKey
574− , _nodeInfo_to = case MapWithMove._nodeInfo_to ni of
349+ , _nodeInfo_to = ComposeMaybe $ Const2 <$> MapWithMove._nodeInfo_to ni
575− MapWithMove.To_NonMove -> To_NonMove
576− MapWithMove.To_Move toKey Proxy -> To_Move $ Const2 toKey :=> Proxy3
579-- | Apply the insertions, deletions, and moves to a given 'DMap'.
352-- | Apply the insertions, deletions, and moves to a given 'DMap'.
580−instance ( GCompare k
582− , PatchSource1 p ~ PatchTarget1 p
583− ) => PatchHet (PatchDMapWithMove k p) where
584− type PatchSource (PatchDMapWithMove k p) = DMap k (PatchSource1 p)
585− type PatchTarget (PatchDMapWithMove k p) = DMap k (PatchTarget1 p)
587−instance ( GCompare k
589− , PatchSource1 p ~ PatchTarget1 p
590− ) => Patch (PatchDMapWithMove k p) where
353+instance GCompare k => Patch (PatchDMapWithMove k v) where
354+ type PatchTarget (PatchDMapWithMove k v) = DMap k v
591− apply (PatchDMapWithMove m) old = Just $! insertions `DMap.union` (old `DMap.difference` deletions)
355+ apply (PatchDMapWithMove p) old = Just $! insertions `DMap.union` (old `DMap.difference` deletions) --TODO: return Nothing sometimes --Note: the strict application here is critical to ensuring that incremental merges don't hold onto all their prerequisite events forever; can we make this more robust?
592− -- TODO: return Nothing sometimes --Note: the strict application here is
593− -- critical to ensuring that incremental merges don't hold onto all their
594− -- prerequisite events forever; can we make this more robust?
595− where insertions = DMap.mapMaybeWithKey insertFunc m
356+ where insertions = DMap.mapMaybeWithKey insertFunc p
596− insertFunc :: forall a. k a -> NodeInfo k p a -> Maybe (PatchTarget1 p a)
357+ insertFunc :: forall a. k a -> NodeInfo k v a -> Maybe (v a)
597 insertFunc _ ni = case _nodeInfo_from ni of
358 insertFunc _ ni = case _nodeInfo_from ni of
598 From_Insert v -> Just v
359 From_Insert v -> Just v
599− From_Move (k :=> Flip p) -> applyAlwaysHet2 p <$> DMap.lookup k old
360+ From_Move k -> DMap.lookup k old
600 From_Delete -> Nothing
361 From_Delete -> Nothing
601− deletions = DMap.mapMaybeWithKey deleteFunc m
362+ deletions = DMap.mapMaybeWithKey deleteFunc p
602− deleteFunc :: forall a. k a -> NodeInfo k p a -> Maybe (Constant () a)
363+ deleteFunc :: forall a. k a -> NodeInfo k v a -> Maybe (Constant () a)
603 deleteFunc _ ni = case _nodeInfo_from ni of
364 deleteFunc _ ni = case _nodeInfo_from ni of
604 From_Delete -> Just $ Constant ()
365 From_Delete -> Just $ Constant ()
605 _ -> Nothing
366 _ -> Nothing
607−-- | Get the values that will be replaced, deleted, or moved if the given patch
368+-- | Get the values that will be replaced, deleted, or moved if the given patch is applied to the given 'DMap'.
608−-- is applied to the given 'DMap'.
609−getDeletionsAndMoves
369+getDeletionsAndMoves :: GCompare k => PatchDMapWithMove k v -> DMap k v' -> DMap k (Product v' (ComposeMaybe k))
611− , PatchSource1 p ~ PatchTarget1 p
613− => PatchDMapWithMove k p
614− -> DMap k (PatchSource1 p)
615− -> DMap k (Product (PatchTarget1 p) (To k p))
616getDeletionsAndMoves (PatchDMapWithMove p) m = DMap.intersectionWithKey f m p
370getDeletionsAndMoves (PatchDMapWithMove p) m = DMap.intersectionWithKey f m p
617 where f _ v ni = Pair v $ _nodeInfo_to ni
371 where f _ v ni = Pair v $ _nodeInfo_to ni