@@ -19,7 +19,9 @@
19module Data.Patch.MapWithMove
19module Data.Patch.MapWithMove
20 ( PatchMapWithMove
20 ( PatchMapWithMove
21 ( PatchMapWithMove
21 ( PatchMapWithMove
22+ , -- | Extract the representation of the 'PatchMapWithMove' as a map of
22− , unPatchMapWithMove
24+ unPatchMapWithMove
25 , patchMapWithMove
27 , patchMapWithMove
@@ -63,10 +65,13 @@ module Data.Patch.MapWithMove
63import Data.Coerce
65import Data.Coerce
64import Data.Kind (Type)
66import Data.Kind (Type)
65import Data.Patch.Class
67import Data.Patch.Class
66−import Data.Patch.MapWithPatchingMove (PatchMapWithPatchingMove(..), To)
68+import Data.Patch.MapWithPatchingMove
69+ ( PatchMapWithPatchingMove (..)
67−import qualified Data.Patch.MapWithPatchingMove as PM -- already a transparent synonym
71+import qualified Data.Patch.MapWithPatchingMove as PM
72+import Data.Patch.MapWithPatchingMove (To) -- already a transparent synonym
69−import Control.Lens
74+import Control.Lens hiding (from, to)
70import Data.List
75import Data.List
71import Data.Map (Map)
76import Data.Map (Map)
72import qualified Data.Map as Map
77import qualified Data.Map as Map
@@ -84,13 +89,9 @@ newtype PatchMapWithMove k (v :: Type) = PatchMapWithMove'
84 unPatchMapWithMove' :: PatchMapWithPatchingMove k (Proxy v)
89 unPatchMapWithMove' :: PatchMapWithPatchingMove k (Proxy v)
86 deriving ( Show, Read, Eq, Ord
91 deriving ( Show, Read, Eq, Ord
87−-- Haddock cannot handle documentation here before GHC 8.6
89−#if __GLASGOW_HASKELL__ >= 806
90− -- | Compose patches having the same effect as applying the
92+ , -- | Compose patches having the same effect as applying the
91 -- patches in turn: @'applyAlways' (p <> q) == 'applyAlways' p .
93 -- patches in turn: @'applyAlways' (p <> q) == 'applyAlways' p .
92 -- 'applyAlways' q@
94 -- 'applyAlways' q@
@@ -101,8 +102,6 @@ pattern Coerce x <- (coerce -> x)
102{-# COMPLETE PatchMapWithMove #-}
103{-# COMPLETE PatchMapWithMove #-}
103pattern PatchMapWithMove :: Map k (NodeInfo k v) -> PatchMapWithMove k v
104pattern PatchMapWithMove :: Map k (NodeInfo k v) -> PatchMapWithMove k v
104−-- | Extract the representation of the 'PatchMapWithMove' as a map of
106unPatchMapWithMove :: PatchMapWithMove k v -> Map k (NodeInfo k v)
105unPatchMapWithMove :: PatchMapWithMove k v -> Map k (NodeInfo k v)
107pattern PatchMapWithMove { unPatchMapWithMove } = PatchMapWithMove' (PatchMapWithPatchingMove (Coerce unPatchMapWithMove))
106pattern PatchMapWithMove { unPatchMapWithMove } = PatchMapWithMove' (PatchMapWithPatchingMove (Coerce unPatchMapWithMove))
@@ -137,7 +136,7 @@ instance TraversableWithIndex k (PatchMapWithMove k) where
138-- | Create a 'PatchMapWithMove', validating it
137-- | Create a 'PatchMapWithMove', validating it
139patchMapWithMove :: Ord k => Map k (NodeInfo k v) -> Maybe (PatchMapWithMove k v)
138patchMapWithMove :: Ord k => Map k (NodeInfo k v) -> Maybe (PatchMapWithMove k v)
140−patchMapWithMove = fmap PatchMapWithMove' . PM.patchMapWithPatchingMove . coerce
139+patchMapWithMove = fmap coerce . PM.patchMapWithPatchingMove . coerce
142-- | Create a 'PatchMapWithMove' that inserts everything in the given 'Map'
141-- | Create a 'PatchMapWithMove' that inserts everything in the given 'Map'
143patchMapWithMoveInsertAll :: Map k v -> PatchMapWithMove k v
142patchMapWithMoveInsertAll :: Map k v -> PatchMapWithMove k v
@@ -160,8 +159,8 @@ moveMapKey src dst = PatchMapWithMove' $ PM.moveMapKey src dst
161-- let aMay = Map.lookup a map
160-- let aMay = Map.lookup a map
162-- bMay = Map.lookup b map
161-- bMay = Map.lookup b map
163−-- in maybe id (Map.insert a) (bMay <> aMay)
162+-- in maybe id (Map.insert a) (bMay `mplus` aMay)
164−-- . maybe id (Map.insert b) (aMay <> bMay)
163+-- . maybe id (Map.insert b) (aMay `mplus` bMay)
165-- . Map.delete a . Map.delete b $ map
164-- . Map.delete a . Map.delete b $ map
167swapMapKey :: Ord k => k -> k -> PatchMapWithMove k v
166swapMapKey :: Ord k => k -> k -> PatchMapWithMove k v
@@ -175,7 +174,7 @@ deleteMapKey = PatchMapWithMove' . PM.deleteMapKey
176-- __Warning:__ when using this function, you must ensure that the invariants of 'PatchMapWithMove' are preserved; they will not be checked.
175-- __Warning:__ when using this function, you must ensure that the invariants of 'PatchMapWithMove' are preserved; they will not be checked.
177unsafePatchMapWithMove :: Map k (NodeInfo k v) -> PatchMapWithMove k v
176unsafePatchMapWithMove :: Map k (NodeInfo k v) -> PatchMapWithMove k v
178−unsafePatchMapWithMove = coerce PM.unsafePatchMapWithPatchingMove
177+unsafePatchMapWithMove = PatchMapWithMove' . PM.unsafePatchMapWithPatchingMove . coerce
180-- | Apply the insertions, deletions, and moves to a given 'Map'
179-- | Apply the insertions, deletions, and moves to a given 'Map'
181instance Ord k => Patch (PatchMapWithMove k v) where
180instance Ord k => Patch (PatchMapWithMove k v) where
@@ -199,7 +198,7 @@ patchThatSortsMapWith cmp = PatchMapWithMove' . PM.patchThatSortsMapWith cmp
199-- | Create a 'PatchMapWithMove' that, if applied to the first 'Map' provided,
198-- | Create a 'PatchMapWithMove' that, if applied to the first 'Map' provided,
200-- will produce a 'Map' with the same values as the second 'Map' but with the
199-- will produce a 'Map' with the same values as the second 'Map' but with the
201-- values sorted with the given ordering function.
200-- values sorted with the given ordering function.
202−patchThatChangesAndSortsMapWith :: (Ord k, Ord v) => (v -> v -> Ordering) -> Map k v -> Map k v -> PatchMapWithMove k v
201+patchThatChangesAndSortsMapWith :: forall k v. (Ord k, Ord v) => (v -> v -> Ordering) -> Map k v -> Map k v -> PatchMapWithMove k v
203patchThatChangesAndSortsMapWith cmp oldByIndex newByIndexUnsorted = patchThatChangesMap oldByIndex newByIndex
202patchThatChangesAndSortsMapWith cmp oldByIndex newByIndexUnsorted = patchThatChangesMap oldByIndex newByIndex
204 where newList = Map.toList newByIndexUnsorted
203 where newList = Map.toList newByIndexUnsorted
205 newByIndex = Map.fromList $ zip (fst <$> newList) $ sortBy cmp $ snd <$> newList
204 newByIndex = Map.fromList $ zip (fst <$> newList) $ sortBy cmp $ snd <$> newList
@@ -227,11 +226,11 @@ deriving instance (Ord k, Ord p) => Ord (NodeInfo k p)
227pattern NodeInfo :: To k -> From k v -> NodeInfo k v
226pattern NodeInfo :: To k -> From k v -> NodeInfo k v
228_nodeInfo_to :: NodeInfo k v -> To k
227_nodeInfo_to :: NodeInfo k v -> To k
229_nodeInfo_from :: NodeInfo k v -> From k v
228_nodeInfo_from :: NodeInfo k v -> From k v
230−pattern NodeInfo { _nodeInfo_to, _nodeInfo_from } = NodeInfo'
229+pattern NodeInfo { _nodeInfo_to, _nodeInfo_from } =
231− PM.NodeInfo
230+ NodeInfo' (PM.NodeInfo
232 { PM._nodeInfo_to = _nodeInfo_to
231 { PM._nodeInfo_to = _nodeInfo_to
233 , PM._nodeInfo_from = Coerce _nodeInfo_from
232 , PM._nodeInfo_from = Coerce _nodeInfo_from
@@ -248,20 +247,24 @@ instance Foldable (NodeInfo k) where
248 foldMap = foldMapDefault
247 foldMap = foldMapDefault
250instance Traversable (NodeInfo k) where
249instance Traversable (NodeInfo k) where
250+ traverse = _NodeInfo . traverseNodeInfo
253+ :: Traversal (PM.NodeInfo k (Proxy a)) (PM.NodeInfo k (Proxy b)) a b
251− traverse = bitraverseNodeInfo pure
254+ traverseNodeInfo = PM.bitraverseNodeInfo pure (\ ~Proxy -> pure Proxy)
253bitraverseNodeInfo
256bitraverseNodeInfo
254 :: Applicative f
257 :: Applicative f
255 => (k0 -> f k1)
258 => (k0 -> f k1)
256 -> (v0 -> f v1)
259 -> (v0 -> f v1)
257 -> NodeInfo k0 v0 -> f (NodeInfo k1 v1)
260 -> NodeInfo k0 v0 -> f (NodeInfo k1 v1)
258−bitraverseNodeInfo fk fv = fmap NodeInfo'
261+bitraverseNodeInfo fk fv = fmap coerce
259 . PM.bitraverseNodeInfo fk (\ ~Proxy -> pure Proxy) fv
262 . PM.bitraverseNodeInfo fk (\ ~Proxy -> pure Proxy) fv
262-- | Change the 'From' value of a 'NodeInfo'
265-- | Change the 'From' value of a 'NodeInfo'
263nodeInfoMapFrom :: (From k v -> From k v) -> NodeInfo k v -> NodeInfo k v
266nodeInfoMapFrom :: (From k v -> From k v) -> NodeInfo k v -> NodeInfo k v
264−nodeInfoMapFrom f = coerce $ PM.nodeInfoMapFrom (unFrom' . f . From')
267+nodeInfoMapFrom f = coerce . PM.nodeInfoMapFrom (unFrom' . f . From') . coerce
266-- | Change the 'From' value of a 'NodeInfo', using a 'Functor' (or
269-- | Change the 'From' value of a 'NodeInfo', using a 'Functor' (or
267-- 'Applicative', 'Monad', etc.) action to get the new value
270-- 'Applicative', 'Monad', etc.) action to get the new value
@@ -269,7 +272,7 @@ nodeInfoMapMFrom
269 :: Functor f
272 :: Functor f
270 => (From k v -> f (From k v))
273 => (From k v -> f (From k v))
271 -> NodeInfo k v -> f (NodeInfo k v)
274 -> NodeInfo k v -> f (NodeInfo k v)
272−nodeInfoMapMFrom f = fmap NodeInfo'
275+nodeInfoMapMFrom f = fmap coerce
273 . PM.nodeInfoMapMFrom (fmap unFrom' . f . From')
276 . PM.nodeInfoMapMFrom (fmap unFrom' . f . From')
@@ -292,7 +295,7 @@ pattern From_Insert v = From' (PM.From_Insert v)
293-- | Delete the existing value, if any, from here
296-- | Delete the existing value, if any, from here
294pattern From_Delete :: From k v
297pattern From_Delete :: From k v
295−pattern From_Delete = From' PM.From_Delete
298+pattern From_Delete = From' (PM.From_Delete)
297-- | Move the value here from the given key
300-- | Move the value here from the given key
298pattern From_Move :: k -> From k v
301pattern From_Move :: k -> From k v
@@ -303,10 +306,9 @@ bitraverseFrom
303 => (k0 -> f k1)
306 => (k0 -> f k1)
304 -> (v0 -> f v1)
307 -> (v0 -> f v1)
305 -> From k0 v0 -> f (From k1 v1)
308 -> From k0 v0 -> f (From k1 v1)
306−bitraverseFrom fk fv = fmap From'
309+bitraverseFrom fk fv = fmap coerce
307 . PM.bitraverseFrom fk (\ ~Proxy -> pure Proxy) fv
310 . PM.bitraverseFrom fk (\ ~Proxy -> pure Proxy) fv
310makeWrapped ''PatchMapWithMove
313makeWrapped ''PatchMapWithMove
311makeWrapped ''NodeInfo
314makeWrapped ''NodeInfo