@@ -28,57 +28,80 @@
2828
2929module Streamly.Internal.Data.MutArray.Type
3030 (
31− -- * Type
31+ -- ** Type
3232 -- $arrayNotes
3333 MutArray (..)
34− , MutableByteArray
35− , touch
34+
35+ -- ** Conversion
36+ -- *** Pinned and Unpinned
3637 , pin
3738 , unpin
3839 , isPinned
3940
40− -- * Constructing and Writing
41+ -- ** Casting
42+ , cast
43+ , castUnsafe -- XXX unsafeCast
44+ , asBytes
45+ , unsafePinnedAsPtr
46+ , unsafeAsPtr
47+
4148 -- ** Construction
42− , nil
49+ , empty
50+
51+ -- *** New
52+ -- | New arrays are always empty arrays with some reserve capacity to
53+ -- extend the length without reallocating.
54+ , emptyOf
55+ , newArrayWith -- emptyAlignedWith
56+ , pinnedEmptyOf
57+ , pinnedNewAligned -- XXX not required
58+ -- , new -- uninitialized array of specified length
59+
60+ -- *** Cloning
61+ , clone
62+ , pinnedClone
4363
44− -- *** Uninitialized Arrays
45− , pinnedNew
46− , pinnedNewBytes
47− , pinnedNewAligned
48− , new
49− , newArrayWith
64+ -- *** Slicing
65+ -- | Get a subarray without copying
66+ , getSliceUnsafe -- XXX unsafeGetSlice
67+ , getSlice
68+ , splitAt -- XXX should be able to express using getSlice
69+ , breakOn
5070
51− -- *** From streams
71+ -- *** Stream Folds
5272 , ArrayUnsafe (..)
53− , writeNWithUnsafe
54− , writeNWith
55− , writeNUnsafe
56− , pinnedWriteNUnsafe
57− , writeN
58− , pinnedWriteN
59− , pinnedWriteNAligned
60−
61− , writeWith
62− , write
63− , pinnedWrite
64−
65− , writeRevN
66− -- , writeRev
67−
68− -- ** From containers
73+ , unsafeCreateOfWith
74+ , unsafeCreateOf
75+ , unsafePinnedCreateOf
76+ , pinnedCreateOf
77+ , createOfWith
78+ , createOf
79+ , revCreateOf
80+
81+ , pinnedCreate
82+ , createWith
83+ , create
84+ -- , revCreate
85+
86+ -- *** From containers
6987 , fromListN
7088 , pinnedFromListN
7189 , fromList
7290 , pinnedFromList
7391 , fromListRevN
7492 , fromListRev
75− , fromStreamDN
76− , fromStreamD
93+ , fromStreamN
94+ , fromStream
95+ , fromPureStreamN
7796 , fromPureStream
97+ , fromByteStr#
98+ , fromPtrN
99+ , fromChunksK
100+ , fromChunksRealloced -- fromSmallChunks
78101
79− -- * Random writes
102+ -- ** Random writes
80103 , putIndex
81− , putIndexUnsafe
104+ , putIndexUnsafe -- XXX unsafePutIndex
82105 , putIndices
83106 -- , putFromThenTo
84107 -- , putFrom -- start writing at the given position
@@ -86,78 +109,71 @@ module Streamly.Internal.Data.MutArray.Type
86109 -- , putFromTo
87110 -- , putFromRev
88111 -- , putUptoRev
89− , modifyIndexUnsafe
112+ , modifyIndexUnsafe -- XXX unsafeModifyIndex
90113 , modifyIndex
91114 , modifyIndices
92115 , modify
93116 , swapIndices
94117 , unsafeSwapIndices
95118
96− -- * Growing and Shrinking
97− -- Arrays grow only at the end, though it is possible to grow on both sides
98− -- and therefore have a cons as well as snoc. But that will require two
99− -- bounds in the array representation.
100−
101− -- ** Appending elements
102− , snocWith
103− , snoc
104− , snocLinear
105− , snocMay
106− , snocUnsafe
107−
108− -- ** Appending streams
109− , writeAppendNUnsafe
110− , writeAppendN
111− , writeAppendWith
112− , writeAppend
119+ -- ** Reading
113120
114− -- * Eliminating and Reading
115−
116− -- ** To streams
117− , reader
118− , readerRevWith
119− , readerRev
121+ -- *** Indexing
122+ , getIndex
123+ , getIndexUnsafe -- XXX unsafeGetIndex
124+ -- , getFromThenTo
125+ , getIndexRev -- getRevIndex?
126+ , indexReader
127+ , indexReaderWith
120128
121− -- ** To containers
122− , toStreamDWith
123− , toStreamDRevWith
124− , toStreamKWith
125− , toStreamKRevWith
129+ -- *** To Streams
126130 , read
127131 , readRev
132+ , toStreamWith
133+ , toStreamRevWith
128134 , toStreamK
135+ , toStreamKWith
129136 , toStreamKRev
137+ , toStreamKRevWith
138+
139+ -- *** To Containers
130140 , toList
131141
142+ -- *** Unfolds
132143 -- experimental
133144 , producerWith
134145 , producer
135146
136− -- ** Random reads
137− , getIndex
138− , getIndexUnsafe
139− , getIndices
140− , getIndicesD
141− -- , getFromThenTo
142− , getIndexRev
143−
144− -- * Memory Management
145− , blockSize
146− , arrayChunkBytes
147− , allocBytesToElemCount
148− , realloc
149− , resize
150− , resizeExp
151− , rightSize
147+ , reader
148+ , readerRevWith
149+ , readerRev
152150
153− -- * Size
151+ -- ** Size and Capacity
152+ -- *** Size
154153 , length
155154 , byteLength
155+
156+ -- *** Capacity
156157 -- , capacity
157158 , byteCapacity
158159 , bytesFree
159160
160− -- * In-place Mutation Algorithms
161+ -- *** Capacity Management
162+ , blockSize
163+ , arrayChunkBytes
164+ , allocBytesToElemCount
165+ , realloc -- this is "resize"
166+ , grow
167+ , growExp
168+ , rightSize
169+
170+ -- ** Folding
171+ , foldl'
172+ , foldr
173+ , byteCmp
174+ , byteEq
175+
176+ -- ** In-place Mutation Algorithms
161177 , strip
162178 , reverse
163179 , permute
@@ -167,62 +183,121 @@ module Streamly.Internal.Data.MutArray.Type
167183 , mergeBy
168184 , bubble
169185
170− -- * Casting
171− , cast
172− , castUnsafe
173− , asBytes
174− , asPtrUnsafe
175−
176− -- * Folding
177− , foldl'
178− , foldr
179− , cmp
180−
181− -- * Arrays of arrays
182− -- We can add dimensionality parameter to the array type to get
183− -- multidimensional arrays. Multidimensional arrays would just be a
184− -- convenience wrapper on top of single dimensional arrays.
186+ -- ** Growing and Shrinking
187+ -- | Arrays grow only at the end, though it is possible to grow on both
188+ -- sides and therefore have a cons as well as snoc. But that will require
189+ -- both lower and upper bound in the array representation.
185190
186− -- | Operations dealing with multiple arrays, streams of arrays or
187− -- multidimensional array representations.
188−
189− -- ** Construct from streams
190− , chunksOf
191− , pinnedChunksOf
192− , writeChunks
193−
194− -- ** Eliminate to streams
195− , flattenArrays
196− , flattenArraysRev
197− , fromArrayStreamK
198−
199− -- ** Construct from arrays
200− -- get chunks without copying
201− , getSliceUnsafe
202− , getSlice
203− -- , getSlicesFromLenN
204− , splitAt -- XXX should be able to express using getSlice
205− , breakOn
191+ -- *** Appending elements
192+ , snocWith
193+ , snoc
194+ , snocLinear
195+ , snocMay
196+ , snocUnsafe -- XXX unsafeSnoc
206197
207− -- ** Cloning arrays
208− , clone
209− , pinnedClone
198+ -- *** Appending streams
199+ , unsafeAppendN
200+ , appendN
201+ , appendWith
202+ , append
210203
211− -- ** Appending arrays
204+ -- *** Appending arrays
212205 , spliceCopy
213206 , spliceWith
214207 , splice
215208 , spliceExp
216− , spliceUnsafe
209+ , spliceUnsafe -- XXX unsafeSplice
217210 -- , putSlice
218211 -- , appendSlice
219212 -- , appendSliceFrom
220213
221− -- * Utilities
214+ -- ** Serialization using Unbox
215+ , pokeAppend
216+ , pokeAppendMay
217+ , pokeSkipUnsafe -- unsafePokeSkip
218+
219+ -- ** Deserialization using Unbox
220+ , peekUncons
221+ , peekUnconsUnsafe -- unsafePeekUncons
222+ , peekSkipUnsafe -- unsafePeekSkip
223+
224+ -- Arrays of arrays
225+ -- We can add dimensionality parameter to the array type to get
226+ -- multidimensional arrays. Multidimensional arrays would just be a
227+ -- convenience wrapper on top of single dimensional arrays.
228+
229+ -- ** Streams of Arrays
230+ -- *** Chunk
231+ -- | Group a stream into arrays.
232+ , chunksOf
233+ , pinnedChunksOf
234+ , buildChunks
235+
236+ -- *** Split
237+ -- | Split an array into slices.
238+
239+ -- , getSlicesFromLenN
240+ , splitOn
241+ -- , slicesOf
242+
243+ -- *** Concat
244+ -- | Append the arrays in a stream to form a stream of elements.
245+ , concatWith
246+ , concatRevWith
247+ , concat
248+ , concatRev
249+
250+ -- *** Compact
251+ -- | Append the arrays in a stream to form a stream of larger arrays.
252+ , SpliceState (..)
253+ , pCompactLE
254+ , pPinnedCompactLE
255+ , compactLeAs
256+ , fCompactGE
257+ , fPinnedCompactGE
258+ , lCompactGE
259+ , lPinnedCompactGE
260+ , compactGE
261+ , compactEQ
262+
263+ -- ** Utilities
222264 , roundUpToPower2
223265 , memcpy
224266 , memcmp
225267 , c_memchr
268+
269+ -- * Deprecated
270+ , asPtrUnsafe
271+ , writeChunks
272+ , flattenArrays
273+ , flattenArraysRev
274+ , fromArrayStreamK
275+ , fromStreamDN
276+ , fromStreamD
277+ , cmp
278+ , getIndices
279+ , getIndicesWith
280+ , resize
281+ , resizeExp
282+ , nil
283+ , new
284+ , pinnedNew
285+ , pinnedNewBytes
286+ , writeAppendNUnsafe
287+ , writeAppendN
288+ , writeAppendWith
289+ , writeAppend
290+ , writeNWithUnsafe
291+ , writeNWith
292+ , writeNUnsafe
293+ , pinnedWriteNUnsafe
294+ , writeN
295+ , pinnedWriteN
296+ , pinnedWriteNAligned -- XXX not required
297+ , writeWith
298+ , write
299+ , pinnedWrite
300+ , writeRevN
226301 )
227302where
228303
@@ -233,49 +308,52 @@ where
233308
234309import Control.Monad (when, void)
235310import Control.Monad.IO.Class (MonadIO(..))
311+import Data.Bifunctor (first)
236312import Data.Bits (shiftR, (.|.), (.&.))
237313import Data.Functor.Identity (Identity(..))
238314import Data.Proxy (Proxy(..))
239315import Data.Word (Word8)
240316import Foreign.C.Types (CSize(..), CInt(..))
241317import Foreign.Ptr (plusPtr, minusPtr, nullPtr)
242−import Streamly.Internal.Data.Unbox
243− ( MutableByteArray(..)
244− , Unbox(..)
318+import Streamly.Internal.Data.MutByteArray.Type
319+ ( MutByteArray(..)
245320 , PinnedState(..)
246321 , getMutableByteArray#
247− , touch
248322 , putSliceUnsafe
249323 )
324+import Streamly.Internal.Data.Unbox (Unbox(..))
250325import GHC.Base
251326 ( IO(..)
252327 , Int(..)
253− , byteArrayContents#
254328 , compareByteArrays#
255329 , copyMutableByteArray#
256330 )
257331import GHC.Base (noinline)
258−import GHC.Exts (unsafeCoerce#)
332+import GHC.Exts (unsafeCoerce#, Addr#)
259333import GHC.Ptr (Ptr(..))
260334
261335import Streamly.Internal.Data.Fold.Type (Fold(..))
262336import Streamly.Internal.Data.Producer.Type (Producer (..))
263337import Streamly.Internal.Data.Stream.Type (Stream)
338+import Streamly.Internal.Data.Parser.Type (Parser (..))
264339import Streamly.Internal.Data.StreamK.Type (StreamK)
265340import Streamly.Internal.Data.SVar.Type (adaptState, defState)
341+import Streamly.Internal.Data.Tuple.Strict (Tuple'(..))
266342import Streamly.Internal.Data.Unfold.Type (Unfold(..))
267343import Streamly.Internal.System.IO (arrayPayloadSize, defaultChunkSize)
268344
269345import qualified Streamly.Internal.Data.Fold.Type as FL
346+import qualified Streamly.Internal.Data.MutByteArray.Type as Unboxed
347+import qualified Streamly.Internal.Data.Parser.Type as Parser
348+-- import qualified Streamly.Internal.Data.Fold.Type as Fold
270349import qualified Streamly.Internal.Data.Producer as Producer
271350import qualified Streamly.Internal.Data.Stream.Type as D
272351import qualified Streamly.Internal.Data.Stream.Lift as D
273352import qualified Streamly.Internal.Data.StreamK.Type as K
274−import qualified Streamly.Internal.Data.Unbox as Unboxed
275353import qualified Prelude
276354
277355import Prelude hiding
278− (Foldable(..), read, unlines, splitAt, reverse, truncate)
356+ (Foldable(..), concat, read, unlines, splitAt, reverse, truncate)
279357
280358#include "DocTestDataMutArray.hs"
281359
@@ -283,6 +361,7 @@ import Prelude hiding
283361-- Foreign helpers
284362-------------------------------------------------------------------------------
285363
364+-- NOTE: Have to be "ccall unsafe" so that we can pass unpinned memory to these
286365foreign import ccall unsafe "string.h memcpy" c_memcpy
287366 :: Ptr Word8 -> Ptr Word8 -> CSize -> IO (Ptr Word8)
288367
@@ -292,6 +371,9 @@ foreign import ccall unsafe "string.h memchr" c_memchr
292371foreign import ccall unsafe "string.h memcmp" c_memcmp
293372 :: Ptr Word8 -> Ptr Word8 -> CSize -> IO CInt
294373
374+foreign import ccall unsafe "string.h strlen" c_strlen
375+ :: Ptr Word8 -> IO CSize
376+
295377-- | Given an 'Unboxed' type (unused first arg) and a number of bytes, return
296378-- how many elements of that type will completely fit in those bytes.
297379--
@@ -346,7 +428,7 @@ data MutArray a =
346428 -- The array is a range into arrContents. arrContents may be a superset of
347429 -- the slice represented by the array. All offsets are in bytes.
348430 MutArray
349− { arrContents :: {-# UNPACK #-} !MutableByteArray
431+ { arrContents :: {-# UNPACK #-} !MutByteArray
350432 , arrStart :: {-# UNPACK #-} !Int -- ^ index into arrContents
351433 , arrEnd :: {-# UNPACK #-} !Int -- ^ index into arrContents
352434 -- Represents the first invalid index of
@@ -358,8 +440,8 @@ data MutArray a =
358440-- Pinning & Unpinning
359441-------------------------------------------------------------------------------
360442
361−-- | Copy the 'MutArray' to pinned memory if unpinned, else do nothing. The
362−-- overhead is a copy if the input array is unpinned.
443+-- | Return a copy of the array in pinned memory if unpinned, else return the
444+-- original array.
363445{-# INLINE pin #-}
364446pin :: MutArray a -> IO (MutArray a)
365447pin arr@MutArray{..} =
@@ -367,8 +449,8 @@ pin arr@MutArray{..} =
367449 then pure arr
368450 else pinnedClone arr
369451
370−-- | Copy the 'MutArray' to unpinned memory if pinned, else do nothing. The
371−-- overhead is a copy if the input array is pinned.
452+-- | Return a copy of the array in unpinned memory if pinned, else return the
453+-- original array.
372454{-# INLINE unpin #-}
373455unpin :: MutArray a -> IO (MutArray a)
374456unpin arr@MutArray{..} =
@@ -376,6 +458,7 @@ unpin arr@MutArray{..} =
376458 then clone arr
377459 else pure arr
378460
461+-- | Return 'True' if the array is allocated in pinned memory.
379462{-# INLINE isPinned #-}
380463isPinned :: MutArray a -> Bool
381464isPinned MutArray{..} = Unboxed.isPinned arrContents
@@ -396,6 +479,8 @@ isPinned MutArray{..} = Unboxed.isPinned arrContents
396479-- we need more than that. See stg_pinnedNewAlignedByteArrayzh and
397480-- allocatePinned in GHC source.
398481
482+-- XXX Rename to emptyAlignedWith, alignSize should be first arg.
483+
399484-- | @newArrayWith allocator alignment count@ allocates a new array of zero
400485-- length and with a capacity to hold @count@ elements, using @allocator
401486-- size alignment@ as the memory allocator function.
@@ -408,7 +493,7 @@ isPinned MutArray{..} = Unboxed.isPinned arrContents
408493-- /Pre-release/
409494{-# INLINE newArrayWith #-}
410495newArrayWith :: forall m a. (MonadIO m, Unbox a)
411− => (Int -> Int -> m MutableByteArray) -> Int -> Int -> m (MutArray a)
496+ => (Int -> Int -> m MutByteArray) -> Int -> Int -> m (MutArray a)
412497newArrayWith alloc alignSize count = do
413498 let size = max (count * SIZE_OF(a)) 0
414499 contents <- alloc size alignSize
@@ -419,12 +504,25 @@ newArrayWith alloc alignSize count = do
419504 , arrBound = size
420505 }
421506
507+-- For arrays "nil" sounds a bit odd. empty is better. The only problem with
508+-- empty is that it is also used by the Alternative type class. But assuming we
509+-- will mostly import the Array module qualified this should be fine.
510+
511+-- | Create an empty array.
512+empty ::
513+#ifdef DEVBUILD
514+ Unbox a =>
515+#endif
516+ MutArray a
517+empty = MutArray Unboxed.empty 0 0 0
518+
519+{-# DEPRECATED nil "Please use empty instead." #-}
422520nil ::
423521#ifdef DEVBUILD
424522 Unbox a =>
425523#endif
426524 MutArray a
427−nil = MutArray Unboxed.nil 0 0 0
525+nil = empty
428526
429527{-# INLINE newBytesAs #-}
430528newBytesAs :: MonadIO m =>
@@ -441,12 +539,13 @@ newBytesAs ps bytes = do
441539 , arrBound = bytes
442540 }
443541
444−-- | Allocates a pinned empty array that can hold 'count' items. The memory of
445−-- the array is uninitialized and the allocation is aligned as per the
446−-- 'Unboxed' instance of the type.
542+-- | Allocates a pinned empty array that with a reserved capacity of bytes.
543+-- The memory of the array is uninitialized and the allocation is aligned as
544+-- per the 'Unboxed' instance of the type.
447545--
448546-- /Pre-release/
449547{-# INLINE pinnedNewBytes #-}
548+{-# DEPRECATED pinnedNewBytes "Please use pinnedEmptyOf with appropriate calculation" #-}
450549pinnedNewBytes :: MonadIO m =>
451550#ifdef DEVBUILD
452551 Unbox a =>
@@ -471,21 +570,31 @@ newAs ps =
471570 (error "new: alignment is not used in unpinned arrays.")
472571
473572-- XXX can unaligned allocation be more efficient when alignment is not needed?
474−--
475−-- | Allocates an empty pinned array that can hold 'count' items. The memory of
476−-- the array is uninitialized and the allocation is aligned as per the 'Unboxed'
477−-- instance of the type.
478−--
573+
574+-- | Allocates a pinned array of zero length but growable to the specified
575+-- capacity without reallocation.
576+{-# INLINE pinnedEmptyOf #-}
577+pinnedEmptyOf :: forall m a. (MonadIO m, Unbox a) => Int -> m (MutArray a)
578+pinnedEmptyOf = newAs Pinned
579+
580+-- XXX Deprecate in major
581+-- {-# DEPRECATED pinnedNew "Please use pinnedEmptyOf instead." #-}
479582{-# INLINE pinnedNew #-}
480583pinnedNew :: forall m a. (MonadIO m, Unbox a) => Int -> m (MutArray a)
481−pinnedNew = newAs Pinned
584+pinnedNew = pinnedEmptyOf
482585
483−-- | Allocates an empty unpinned array that can hold 'count' items. The memory
484−-- of the array is uninitialized.
586+-- | Allocates an unpinned array of zero length but growable to the specified
587+-- capacity without reallocation.
485588--
589+{-# INLINE emptyOf #-}
590+emptyOf :: (MonadIO m, Unbox a) => Int -> m (MutArray a)
591+emptyOf = newAs Unpinned
592+
593+-- XXX Deprecate in major
594+-- {-# DEPRECATED new "Please use emptyOf instead." #-}
486595{-# INLINE new #-}
487596new :: (MonadIO m, Unbox a) => Int -> m (MutArray a)
488−new = newAs Unpinned
597+new = emptyOf
489598
490599-------------------------------------------------------------------------------
491600-- Random writes
@@ -501,7 +610,7 @@ putIndexUnsafe :: forall m a. (MonadIO m, Unbox a)
501610putIndexUnsafe i MutArray{..} x = do
502611 let index = INDEX_OF(arrStart, i, a)
503612 assert (i >= 0 && INDEX_VALID(index, arrEnd, a)) (return ())
504− liftIO $ pokeByteIndex index arrContents x
613+ liftIO $ pokeAt index arrContents x
505614
506615invalidIndex :: String -> Int -> a
507616invalidIndex label i =
@@ -519,7 +628,7 @@ putIndex :: forall m a. (MonadIO m, Unbox a) => Int -> MutArray a -> a -> m ()
519628putIndex i MutArray{..} x = do
520629 let index = INDEX_OF(arrStart,i,a)
521630 if i >= 0 && INDEX_VALID(index,arrEnd,a)
522− then liftIO $ pokeByteIndex index arrContents x
631+ then liftIO $ pokeAt index arrContents x
523632 else invalidIndex "putIndex" i
524633
525634-- | Write an input stream of (index, value) pairs to an array. Throws an
@@ -533,7 +642,7 @@ putIndices arr = FL.foldlM' step (return ())
533642
534643 where
535644
536− step () (i, x) = liftIO (putIndex i arr x)
645+ step () (i, x) = putIndex i arr x
537646
538647-- | Modify a given index of an array using a modifier function.
539648--
@@ -545,9 +654,9 @@ modifyIndexUnsafe :: forall m a b. (MonadIO m, Unbox a) =>
545654modifyIndexUnsafe i MutArray{..} f = liftIO $ do
546655 let index = INDEX_OF(arrStart,i,a)
547656 assert (i >= 0 && INDEX_NEXT(index,a) <= arrEnd) (return ())
548− r <- peekByteIndex index arrContents
657+ r <- peekAt index arrContents
549658 let (x, res) = f r
550− pokeByteIndex index arrContents x
659+ pokeAt index arrContents x
551660 return res
552661
553662-- | Modify a given index of an array using a modifier function.
@@ -559,9 +668,9 @@ modifyIndex i MutArray{..} f = do
559668 let index = INDEX_OF(arrStart,i,a)
560669 if i >= 0 && INDEX_VALID(index,arrEnd,a)
561670 then liftIO $ do
562− r <- peekByteIndex index arrContents
671+ r <- peekAt index arrContents
563672 let (x, res) = f r
564− pokeByteIndex index arrContents x
673+ pokeAt index arrContents x
565674 return res
566675 else invalidIndex "modifyIndex" i
567676
@@ -595,8 +704,8 @@ modify MutArray{..} f = liftIO $
595704
596705 go i =
597706 when (INDEX_VALID(i,arrEnd,a)) $ do
598− r <- peekByteIndex i arrContents
599− pokeByteIndex i arrContents (f r)
707+ r <- peekAt i arrContents
708+ pokeAt i arrContents (f r)
600709 go (INDEX_NEXT(i,a))
601710
602711-- XXX We could specify the number of bytes to swap instead of Proxy. Need
@@ -605,15 +714,15 @@ modify MutArray{..} f = liftIO $
605714swapArrayByteIndices ::
606715 forall a. Unbox a
607716 => Proxy a
608− -> MutableByteArray
717+ -> MutByteArray
609718 -> Int
610719 -> Int
611720 -> IO ()
612721swapArrayByteIndices _ arrContents i1 i2 = do
613− r1 <- peekByteIndex i1 arrContents
614− r2 <- peekByteIndex i2 arrContents
615− pokeByteIndex i1 arrContents (r2 :: a)
616− pokeByteIndex i2 arrContents (r1 :: a)
722+ r1 <- peekAt i1 arrContents
723+ r2 <- peekAt i2 arrContents
724+ pokeAt i1 arrContents (r2 :: a)
725+ pokeAt i2 arrContents (r1 :: a)
617726
618727-- | Swap the elements at two indices without validating the indices.
619728--
@@ -729,26 +838,23 @@ arrayChunkBytes = 1024
729838roundDownTo :: Int -> Int -> Int
730839roundDownTo elemSize size = size - (size `mod` elemSize)
731840
732−-- XXX See if resizing can be implemented by reading the old array as a stream
733−-- and then using writeN to the new array.
734−--
735841-- NOTE: we are passing elemSize explicitly to avoid an Unboxed constraint.
736842-- Since this is not inlined Unboxed consrraint leads to dictionary passing
737843-- which complicates some inspection tests.
738844--
739−{-# NOINLINE reallocExplicit #-}
740−reallocExplicit :: Int -> Int -> MutArray a -> IO (MutArray a)
741−reallocExplicit elemSize newCapacityInBytes MutArray{..} = do
845+{-# NOINLINE reallocExplicitAs #-}
846+reallocExplicitAs :: PinnedState -> Int -> Int -> MutArray a -> IO (MutArray a)
847+reallocExplicitAs ps elemSize newCapacityInBytes MutArray{..} = do
742848 assertM(arrEnd <= arrBound)
743849
744850 -- Allocate new array
745851 let newCapMaxInBytes = roundUpLargeArray newCapacityInBytes
746852 contents <-
747− if Unboxed.isPinned arrContents
748− then Unboxed.pinnedNewBytes newCapMaxInBytes
749− else Unboxed.newBytes newCapMaxInBytes
750− let !(MutableByteArray mbarrFrom#) = arrContents
751− !(MutableByteArray mbarrTo#) = contents
853+ if ps == Pinned
854+ then Unboxed.pinnedNew newCapMaxInBytes
855+ else Unboxed.new newCapMaxInBytes
856+ let !(MutByteArray mbarrFrom#) = arrContents
857+ !(MutByteArray mbarrTo#) = contents
752858
753859 -- Copy old data
754860 let oldStart = arrStart
@@ -769,6 +875,10 @@ reallocExplicit elemSize newCapacityInBytes MutArray{..} = do
769875 , arrBound = newCapInBytes
770876 }
771877
878+-- XXX Should these be called reallocBytes etc?
879+-- XXX We may also need reallocAs to allocate as pinned/unpinned explicitly. In
880+-- fact clone/pinnedClone can be implemented using reallocAs.
881+
772882-- | @realloc newCapacity array@ reallocates the array to the specified
773883-- capacity in bytes.
774884--
@@ -780,7 +890,12 @@ reallocExplicit elemSize newCapacityInBytes MutArray{..} = do
780890-- If the original array is pinned, the newly allocated array is also pinned.
781891{-# INLINABLE realloc #-}
782892realloc :: forall m a. (MonadIO m, Unbox a) => Int -> MutArray a -> m (MutArray a)
783−realloc bytes arr = liftIO $ reallocExplicit (SIZE_OF(a)) bytes arr
893+realloc bytes arr =
894+ let ps =
895+ if isPinned arr
896+ then Pinned
897+ else Unpinned
898+ in liftIO $ reallocExplicitAs ps (SIZE_OF(a)) bytes arr
784899
785900-- | @reallocWith label capSizer minIncrBytes array@. The label is used
786901-- in error messages and the capSizer is used to determine the capacity of the
@@ -803,14 +918,14 @@ reallocWith label capSizer minIncrBytes arr = do
803918 where
804919
805920 badSize newSize =
806− concat
921+ Prelude.concat
807922 [ label
808923 , ": new array size (in bytes) is less than required size "
809924 , show newSize
810925 , ". Please check the sizing function passed."
811926 ]
812927
813−-- | @resize newCapacity array@ changes the total capacity of the array so that
928+-- | @grow newCapacity array@ changes the total capacity of the array so that
814929-- it is enough to hold the specified number of elements. Nothing is done if
815930-- the specified capacity is less than the length of the array.
816931--
@@ -818,34 +933,46 @@ reallocWith label capSizer minIncrBytes arr = do
818933-- the block size (4K).
819934--
820935-- /Pre-release/
821−{-# INLINE resize #-}
822−resize :: forall m a. (MonadIO m, Unbox a) =>
936+{-# INLINE grow #-}
937+grow :: forall m a. (MonadIO m, Unbox a) =>
823938 Int -> MutArray a -> m (MutArray a)
824−resize nElems arr@MutArray{..} = do
939+grow nElems arr@MutArray{..} = do
825940 let req = SIZE_OF(a) * nElems
826− len = arrEnd - arrStart
827− if req < len
941+ cap = arrBound - arrStart
942+ if req < cap
828943 then return arr
829944 else realloc req arr
830945
831−-- | Like 'resize' but if the byte capacity is more than 'largeObjectThreshold'
832−-- then it is rounded up to the closest power of 2.
946+{-# DEPRECATED resize "Please use grow instead." #-}
947+{-# INLINE resize #-}
948+resize :: forall m a. (MonadIO m, Unbox a) =>
949+ Int -> MutArray a -> m (MutArray a)
950+resize = grow
951+
952+-- | Like 'grow' but if the requested byte capacity is more than
953+-- 'largeObjectThreshold' then it is rounded up to the closest power of 2.
833954--
834955-- /Pre-release/
835−{-# INLINE resizeExp #-}
836−resizeExp :: forall m a. (MonadIO m, Unbox a) =>
956+{-# INLINE growExp #-}
957+growExp :: forall m a. (MonadIO m, Unbox a) =>
837958 Int -> MutArray a -> m (MutArray a)
838−resizeExp nElems arr@MutArray{..} = do
959+growExp nElems arr@MutArray{..} = do
839960 let req = roundUpLargeArray (SIZE_OF(a) * nElems)
840961 req1 =
841962 if req > largeObjectThreshold
842963 then roundUpToPower2 req
843964 else req
844− len = arrEnd - arrStart
845− if req1 < len
965+ cap = arrBound - arrStart
966+ if req1 < cap
846967 then return arr
847968 else realloc req1 arr
848969
970+{-# DEPRECATED resizeExp "Please use growExp instead." #-}
971+{-# INLINE resizeExp #-}
972+resizeExp :: forall m a. (MonadIO m, Unbox a) =>
973+ Int -> MutArray a -> m (MutArray a)
974+resizeExp = growExp
975+
849976-- | Resize the allocated memory to drop any reserved free space at the end of
850977-- the array and reallocate it to reduce wastage.
851978--
@@ -890,7 +1017,7 @@ rightSize arr@MutArray{..} = do
8901017snocNewEnd :: (MonadIO m, Unbox a) => Int -> MutArray a -> a -> m (MutArray a)
8911018snocNewEnd newEnd arr@MutArray{..} x = liftIO $ do
8921019 assert (newEnd <= arrBound) (return ())
893− pokeByteIndex arrEnd arrContents x
1020+ pokeAt arrEnd arrContents x
8941021 return $ arr {arrEnd = newEnd}
8951022
8961023-- | Really really unsafe, appends the element into the first array, may
@@ -910,7 +1037,7 @@ snocUnsafe arr@MutArray{..} = snocNewEnd (INDEX_NEXT(arrEnd,a)) arr
9101037{-# INLINE snocMay #-}
9111038snocMay :: forall m a. (MonadIO m, Unbox a) =>
9121039 MutArray a -> a -> m (Maybe (MutArray a))
913−snocMay arr@MutArray{..} x = liftIO $ do
1040+snocMay arr@MutArray{..} x = do
9141041 let newEnd = INDEX_NEXT(arrEnd,a)
9151042 if newEnd <= arrBound
9161043 then Just <$> snocNewEnd newEnd arr x
@@ -924,7 +1051,7 @@ snocWithRealloc :: forall m a. (MonadIO m, Unbox a) =>
9241051 -> a
9251052 -> m (MutArray a)
9261053snocWithRealloc sizer arr x = do
927− arr1 <- liftIO $ reallocWith "snocWith" sizer (SIZE_OF(a)) arr
1054+ arr1 <- reallocWith "snocWith" sizer (SIZE_OF(a)) arr
9281055 snocUnsafe arr1 x
9291056
9301057-- | @snocWith sizer arr elem@ mutates @arr@ to append @elem@. The length of
@@ -946,7 +1073,7 @@ snocWith :: forall m a. (MonadIO m, Unbox a) =>
9461073 -> MutArray a
9471074 -> a
9481075 -> m (MutArray a)
949−snocWith allocSize arr x = liftIO $ do
1076+snocWith allocSize arr x = do
9501077 let newEnd = INDEX_NEXT(arrEnd arr,a)
9511078 if newEnd <= arrBound arr
9521079 then snocNewEnd newEnd arr x
@@ -990,6 +1117,126 @@ snoc = snocWith f
9901117 then oldSize * 2
9911118 else roundUpToPower2 oldSize * 2
9921119
1120+-------------------------------------------------------------------------------
1121+-- Serialization/Deserialization
1122+-------------------------------------------------------------------------------
1123+
1124+{-# INLINE pokeNewEnd #-}
1125+pokeNewEnd :: (MonadIO m, Unbox a) =>
1126+ Int -> MutArray Word8 -> a -> m (MutArray Word8)
1127+pokeNewEnd newEnd arr@MutArray{..} x = liftIO $ do
1128+ assert (newEnd <= arrBound) (return ())
1129+ liftIO $ pokeAt arrEnd arrContents x
1130+ return $ arr {arrEnd = newEnd}
1131+
1132+-- | Really really unsafe, unboxes a Haskell type and appends the resulting
1133+-- bytes to the byte array, may cause silent data corruption or if you are
1134+-- lucky a segfault if the array does not have enough space to append the
1135+-- element.
1136+--
1137+-- /Internal/
1138+{-# INLINE pokeAppendUnsafe #-}
1139+pokeAppendUnsafe :: forall m a. (MonadIO m, Unbox a) =>
1140+ MutArray Word8 -> a -> m (MutArray Word8)
1141+pokeAppendUnsafe arr@MutArray{..} = pokeNewEnd (arrEnd + SIZE_OF(a)) arr
1142+
1143+-- | Skip the specified number of bytes in the array. The data in the skipped
1144+-- region remains uninitialzed.
1145+{-# INLINE pokeSkipUnsafe #-}
1146+pokeSkipUnsafe :: Int -> MutArray Word8 -> MutArray Word8
1147+pokeSkipUnsafe n arr@MutArray{..} = do
1148+ let newEnd = arrEnd + n
1149+ in assert (newEnd <= arrBound) (arr {arrEnd = newEnd})
1150+
1151+-- | Like 'pokeAppend' but does not grow the array when pre-allocated array
1152+-- capacity becomes full.
1153+--
1154+-- /Internal/
1155+{-# INLINE pokeAppendMay #-}
1156+pokeAppendMay :: forall m a. (MonadIO m, Unbox a) =>
1157+ MutArray Word8 -> a -> m (Maybe (MutArray Word8))
1158+pokeAppendMay arr@MutArray{..} x = liftIO $ do
1159+ let newEnd = arrEnd + SIZE_OF(a)
1160+ if newEnd <= arrBound
1161+ then Just <$> pokeNewEnd newEnd arr x
1162+ else return Nothing
1163+
1164+{-# NOINLINE pokeWithRealloc #-}
1165+pokeWithRealloc :: forall m a. (MonadIO m, Unbox a) =>
1166+ (Int -> Int)
1167+ -> MutArray Word8
1168+ -> a
1169+ -> m (MutArray Word8)
1170+pokeWithRealloc sizer arr x = do
1171+ arr1 <- liftIO $ reallocWith "pokeWithRealloc" sizer (SIZE_OF(a)) arr
1172+ pokeAppendUnsafe arr1 x
1173+
1174+{-# INLINE pokeAppendWith #-}
1175+pokeAppendWith :: forall m a. (MonadIO m, Unbox a) =>
1176+ (Int -> Int)
1177+ -> MutArray Word8
1178+ -> a
1179+ -> m (MutArray Word8)
1180+pokeAppendWith allocSize arr x = liftIO $ do
1181+ let newEnd = arrEnd arr + SIZE_OF(a)
1182+ if newEnd <= arrBound arr
1183+ then pokeNewEnd newEnd arr x
1184+ else pokeWithRealloc allocSize arr x
1185+
1186+-- | Unbox a Haskell type and append the resulting bytes to a mutable byte
1187+-- array. The array is grown exponentially when more space is needed.
1188+--
1189+-- Definition:
1190+--
1191+-- >>> pokeAppend arr x = MutArray.castUnsafe <$> MutArray.snoc (MutArray.castUnsafe arr) x
1192+--
1193+{-# INLINE pokeAppend #-}
1194+pokeAppend :: forall m a. (MonadIO m, Unbox a) =>
1195+ MutArray Word8 -> a -> m (MutArray Word8)
1196+pokeAppend = pokeAppendWith f
1197+
1198+ where
1199+
1200+ f oldSize =
1201+ if isPower2 oldSize
1202+ then oldSize * 2
1203+ else roundUpToPower2 oldSize * 2
1204+
1205+-- | Really really unsafe, create a Haskell value from an unboxed byte array,
1206+-- does not check if the array is big enough, may return garbage or if you are
1207+-- lucky may cause a segfault.
1208+--
1209+-- /Internal/
1210+{-# INLINE peekUnconsUnsafe #-}
1211+peekUnconsUnsafe :: forall m a. (MonadIO m, Unbox a) =>
1212+ MutArray Word8 -> m (a, MutArray Word8)
1213+peekUnconsUnsafe MutArray{..} = do
1214+ let start1 = arrStart + SIZE_OF(a)
1215+ assert (start1 <= arrEnd) (return ())
1216+ liftIO $ do
1217+ r <- peekAt arrStart arrContents
1218+ return (r, MutArray arrContents start1 arrEnd arrBound)
1219+
1220+-- | Discard the specified number of bytes in the array.
1221+{-# INLINE peekSkipUnsafe #-}
1222+peekSkipUnsafe :: Int -> MutArray Word8 -> MutArray Word8
1223+peekSkipUnsafe n MutArray{..} =
1224+ let start1 = arrStart + n
1225+ in assert (start1 <= arrEnd) (MutArray arrContents start1 arrEnd arrBound)
1226+
1227+-- | Create a Haskell value from its unboxed representation from the head of a
1228+-- byte array, return the value and the remaining array.
1229+{-# INLINE peekUncons #-}
1230+peekUncons :: forall m a. (MonadIO m, Unbox a) =>
1231+ MutArray Word8 -> m (Maybe a, MutArray Word8)
1232+peekUncons arr@MutArray{..} = do
1233+ let start1 = arrStart + SIZE_OF(a)
1234+ if start1 > arrEnd
1235+ then return (Nothing, arr)
1236+ else liftIO $ do
1237+ r <- peekAt arrStart arrContents
1238+ return (Just r, MutArray arrContents start1 arrEnd arrBound)
1239+
9931240-------------------------------------------------------------------------------
9941241-- Random reads
9951242-------------------------------------------------------------------------------
@@ -1004,7 +1251,7 @@ getIndexUnsafe :: forall m a. (MonadIO m, Unbox a) => Int -> MutArray a -> m a
10041251getIndexUnsafe i MutArray{..} = do
10051252 let index = INDEX_OF(arrStart,i,a)
10061253 assert (i >= 0 && INDEX_VALID(index,arrEnd,a)) (return ())
1007− liftIO $ peekByteIndex index arrContents
1254+ liftIO $ peekAt index arrContents
10081255
10091256-- | /O(1)/ Lookup the element at the given index. Index starts from 0.
10101257--
@@ -1013,7 +1260,7 @@ getIndex :: forall m a. (MonadIO m, Unbox a) => Int -> MutArray a -> m (Maybe a)
10131260getIndex i MutArray{..} = do
10141261 let index = INDEX_OF(arrStart,i,a)
10151262 if i >= 0 && INDEX_VALID(index,arrEnd,a)
1016− then liftIO $ Just <$> peekByteIndex index arrContents
1263+ then liftIO $ Just <$> peekAt index arrContents
10171264 else return Nothing
10181265
10191266-- | /O(1)/ Lookup the element at the given index from the end of the array.
@@ -1026,21 +1273,16 @@ getIndexRev :: forall m a. (MonadIO m, Unbox a) => Int -> MutArray a -> m a
10261273getIndexRev i MutArray{..} = do
10271274 let index = RINDEX_OF(arrEnd,i,a)
10281275 if i >= 0 && index >= arrStart
1029− then liftIO $ peekByteIndex index arrContents
1276+ then liftIO $ peekAt index arrContents
10301277 else invalidIndex "getIndexRev" i
10311278
10321279data GetIndicesState contents start end st =
10331280 GetIndicesState contents start end st
10341281
1035−-- | Given an unfold that generates array indices, read the elements on those
1036−-- indices from the supplied MutArray. An error is thrown if an index is out of
1037−-- bounds.
1038−--
1039−-- /Pre-release/
1040−{-# INLINE getIndicesD #-}
1041−getIndicesD :: (Monad m, Unbox a) =>
1282+{-# INLINE indexReaderWith #-}
1283+indexReaderWith :: (Monad m, Unbox a) =>
10421284 (forall b. IO b -> m b) -> D.Stream m Int -> Unfold m (MutArray a) a
1043−getIndicesD liftio (D.Stream stepi sti) = Unfold step inject
1285+indexReaderWith liftio (D.Stream stepi sti) = Unfold step inject
10441286
10451287 where
10461288
@@ -1059,9 +1301,26 @@ getIndicesD liftio (D.Stream stepi sti) = Unfold step inject
10591301 D.Skip s -> return $ D.Skip (GetIndicesState contents start end s)
10601302 D.Stop -> return D.Stop
10611303
1304+{-# DEPRECATED getIndicesWith "Please use indexReaderWith instead." #-}
1305+{-# INLINE getIndicesWith #-}
1306+getIndicesWith :: (Monad m, Unbox a) =>
1307+ (forall b. IO b -> m b) -> D.Stream m Int -> Unfold m (MutArray a) a
1308+getIndicesWith = indexReaderWith
1309+
1310+-- | Given an unfold that generates array indices, read the elements on those
1311+-- indices from the supplied MutArray. An error is thrown if an index is out of
1312+-- bounds.
1313+--
1314+-- /Pre-release/
1315+{-# INLINE indexReader #-}
1316+indexReader :: (MonadIO m, Unbox a) => Stream m Int -> Unfold m (MutArray a) a
1317+indexReader = indexReaderWith liftIO
1318+
1319+-- XXX DO NOT REMOVE, change the signature to use Stream instead of unfold
1320+{-# DEPRECATED getIndices "Please use indexReader instead." #-}
10621321{-# INLINE getIndices #-}
10631322getIndices :: (MonadIO m, Unbox a) => Stream m Int -> Unfold m (MutArray a) a
1064−getIndices = getIndicesD liftIO
1323+getIndices = indexReader
10651324
10661325-------------------------------------------------------------------------------
10671326-- Subarrays
@@ -1166,7 +1425,7 @@ partitionBy f arr@MutArray{..} = liftIO $ do
11661425
11671426 -- Invariant low < high on entry, and on return as well
11681427 moveHigh low high = do
1169− h <- peekByteIndex high arrContents
1428+ h <- peekAt high arrContents
11701429 if f h
11711430 then
11721431 -- Correctly classified, continue the loop
@@ -1186,7 +1445,7 @@ partitionBy f arr@MutArray{..} = liftIO $ do
11861445 -- low <= high
11871446 -- Both low and high are valid locations within the array
11881447 go low high = do
1189− l <- peekByteIndex low arrContents
1448+ l <- peekAt low arrContents
11901449 if f l
11911450 then
11921451 -- low is wrongly classified
@@ -1197,8 +1456,8 @@ partitionBy f arr@MutArray{..} = liftIO $ do
11971456 case r of
11981457 Nothing -> return low
11991458 Just (high1, h) -> do -- low < high1
1200− pokeByteIndex low arrContents h
1201− pokeByteIndex high1 arrContents l
1459+ pokeAt low arrContents h
1460+ pokeAt high1 arrContents l
12021461 let low1 = INDEX_NEXT(low,a)
12031462 high2 = INDEX_PREV(high1,a)
12041463 if low1 <= high2
@@ -1332,7 +1591,7 @@ chunksOfAs ps n (D.Stream step state) =
13321591 r <- step (adaptState gst) st
13331592 case r of
13341593 D.Yield x s -> do
1335− liftIO $ pokeByteIndex end contents x
1594+ liftIO $ pokeAt end contents x
13361595 let end1 = INDEX_NEXT(end,a)
13371596 return $
13381597 if end1 >= bound
@@ -1351,10 +1610,12 @@ chunksOfAs ps n (D.Stream step state) =
13511610
13521611 step' _ GroupFinish = return D.Stop
13531612
1354−-- | @chunksOf n stream@ groups the input stream into a stream of
1355−-- arrays of size n.
1613+-- | @chunksOf n stream@ groups the elements in the input stream into arrays of
1614+-- @n@ elements each.
1615+--
1616+-- Same as the following but may be more efficient:
13561617--
1357−-- @chunksOf n = StreamD.foldMany (MutArray.writeN n)@
1618+-- >>> chunksOf n = Stream.foldMany (MutArray.createOf n)
13581619--
13591620-- /Pre-release/
13601621{-# INLINE_NORMAL chunksOf #-}
@@ -1363,17 +1624,29 @@ chunksOf :: forall m a. (MonadIO m, Unbox a)
13631624-- XXX the idiomatic implementation leads to large regression in the D.reverse'
13641625-- benchmark. It seems it has difficulty producing optimized code when
13651626-- converting to StreamK. Investigate GHC optimizations.
1366−-- chunksOf n = D.foldMany (writeN n)
1627+-- chunksOf n = D.foldMany (createOf n)
13671628chunksOf = chunksOfAs Unpinned
13681629
13691630-- | Like 'chunksOf' but creates pinned arrays.
13701631{-# INLINE_NORMAL pinnedChunksOf #-}
13711632pinnedChunksOf :: forall m a. (MonadIO m, Unbox a)
13721633 => Int -> D.Stream m a -> D.Stream m (MutArray a)
1373−-- pinnedChunksOf n = D.foldMany (pinnedWriteN n)
1634+-- pinnedChunksOf n = D.foldMany (pinnedCreateOf n)
13741635pinnedChunksOf = chunksOfAs Pinned
13751636
1376−-- XXX This should take a PinnedState
1637+-- | When we are buffering a stream of unknown size into an array we do not
1638+-- know how much space to pre-allocate. So we start with the min size and emit
1639+-- the array then keep on doubling the size every time. Thus we do not need to
1640+-- guess the optimum chunk size.
1641+--
1642+-- We can incorporate this in chunksOfAs if the additional size parameter does
1643+-- not impact perf.
1644+--
1645+{-# INLINE _chunksOfRange #-}
1646+_chunksOfRange :: -- (MonadIO m, Unbox a) =>
1647+ PinnedState -> Int -> Int -> D.Stream m a -> D.Stream m (MutArray a)
1648+_chunksOfRange _ps _low _hi = undefined
1649+
13771650-- XXX buffer to a list instead?
13781651-- | Buffer the stream into arrays in memory.
13791652{-# INLINE arrayStreamKFromStreamDAs #-}
@@ -1391,16 +1664,10 @@ data FlattenState s contents a =
13911664 OuterLoop s
13921665 | InnerLoop s contents !Int !Int
13931666
1394−-- | Use the "reader" unfold instead.
1395−--
1396−-- @flattenArrays = unfoldMany reader@
1397−--
1398−-- We can try this if there are any fusion issues in the unfold.
1399−--
1400−{-# INLINE_NORMAL flattenArrays #-}
1401−flattenArrays :: forall m a. (MonadIO m, Unbox a)
1402− => D.Stream m (MutArray a) -> D.Stream m a
1403−flattenArrays (D.Stream step state) = D.Stream step' (OuterLoop state)
1667+{-# INLINE_NORMAL concatWith #-}
1668+concatWith :: forall m a. (Monad m, Unbox a)
1669+ => (forall b. IO b -> m b) -> D.Stream m (MutArray a) -> D.Stream m a
1670+concatWith liftio (D.Stream step state) = D.Stream step' (OuterLoop state)
14041671
14051672 where
14061673
@@ -1417,19 +1684,30 @@ flattenArrays (D.Stream step state) = D.Stream step' (OuterLoop state)
14171684 return $ D.Skip $ OuterLoop st
14181685
14191686 step' _ (InnerLoop st contents p end) = do
1420− x <- liftIO $ peekByteIndex p contents
1687+ x <- liftio $ peekAt p contents
14211688 return $ D.Yield x (InnerLoop st contents (INDEX_NEXT(p,a)) end)
14221689
1423−-- | Use the "readerRev" unfold instead.
1690+-- | Use the "reader" unfold instead.
14241691--
1425−-- @flattenArrays = unfoldMany readerRev@
1692+-- @concat = unfoldMany reader@
14261693--
14271694-- We can try this if there are any fusion issues in the unfold.
14281695--
1429−{-# INLINE_NORMAL flattenArraysRev #-}
1430−flattenArraysRev :: forall m a. (MonadIO m, Unbox a)
1696+{-# INLINE_NORMAL concat #-}
1697+concat :: forall m a. (MonadIO m, Unbox a)
14311698 => D.Stream m (MutArray a) -> D.Stream m a
1432−flattenArraysRev (D.Stream step state) = D.Stream step' (OuterLoop state)
1699+concat = concatWith liftIO
1700+
1701+{-# DEPRECATED flattenArrays "Please use \"unfoldMany reader\" instead." #-}
1702+{-# INLINE flattenArrays #-}
1703+flattenArrays :: forall m a. (MonadIO m, Unbox a)
1704+ => D.Stream m (MutArray a) -> D.Stream m a
1705+flattenArrays = concat
1706+
1707+{-# INLINE_NORMAL concatRevWith #-}
1708+concatRevWith :: forall m a. (Monad m, Unbox a)
1709+ => (forall b. IO b -> m b) -> D.Stream m (MutArray a) -> D.Stream m a
1710+concatRevWith liftio (D.Stream step state) = D.Stream step' (OuterLoop state)
14331711
14341712 where
14351713
@@ -1447,16 +1725,33 @@ flattenArraysRev (D.Stream step state) = D.Stream step' (OuterLoop state)
14471725 return $ D.Skip $ OuterLoop st
14481726
14491727 step' _ (InnerLoop st contents p start) = do
1450− x <- liftIO $ peekByteIndex p contents
1728+ x <- liftio $ peekAt p contents
14511729 let cur = INDEX_PREV(p,a)
14521730 return $ D.Yield x (InnerLoop st contents cur start)
14531731
1732+-- | Use the "readerRev" unfold instead.
1733+--
1734+-- @concat = unfoldMany readerRev@
1735+--
1736+-- We can try this if there are any fusion issues in the unfold.
1737+--
1738+{-# INLINE_NORMAL concatRev #-}
1739+concatRev :: forall m a. (MonadIO m, Unbox a)
1740+ => D.Stream m (MutArray a) -> D.Stream m a
1741+concatRev = concatRevWith liftIO
1742+
1743+{-# DEPRECATED flattenArraysRev "Please use \"unfoldMany readerRev\" instead." #-}
1744+{-# INLINE flattenArraysRev #-}
1745+flattenArraysRev :: forall m a. (MonadIO m, Unbox a)
1746+ => D.Stream m (MutArray a) -> D.Stream m a
1747+flattenArraysRev = concatRev
1748+
14541749-------------------------------------------------------------------------------
14551750-- Unfolds
14561751-------------------------------------------------------------------------------
14571752
14581753data ArrayUnsafe a = ArrayUnsafe
1459− {-# UNPACK #-} !MutableByteArray -- contents
1754+ {-# UNPACK #-} !MutByteArray -- contents
14601755 {-# UNPACK #-} !Int -- index 1
14611756 {-# UNPACK #-} !Int -- index 2
14621757
@@ -1486,7 +1781,7 @@ producerWith liftio = Producer step (return . toArrayUnsafe) extract
14861781 -- few actions for correctness and execution order sanity. We want
14871782 -- the peek to occur right here and not lazily at some later point
14881783 -- because we want the peek to be ordered with respect to the touch.
1489− !x <- liftio $ peekByteIndex cur contents
1784+ !x <- liftio $ peekAt cur contents
14901785 return $ D.Yield x (ArrayUnsafe contents (INDEX_NEXT(cur,a)) end)
14911786
14921787 extract = return . fromArrayUnsafe
@@ -1517,7 +1812,7 @@ readerRevWith liftio = Unfold step inject
15171812 {-# INLINE_LATE step #-}
15181813 step (ArrayUnsafe _ start p) | p < start = return D.Stop
15191814 step (ArrayUnsafe contents start p) = do
1520− !x <- liftio $ peekByteIndex p contents
1815+ !x <- liftio $ peekAt p contents
15211816 return $ D.Yield x (ArrayUnsafe contents start (INDEX_PREV(p,a)))
15221817
15231818-- | Unfold an array into a stream in reverse order.
@@ -1548,7 +1843,7 @@ toListFB c n MutArray{..} = go arrStart
15481843 -- evaluated/written to before we peek at them.
15491844 -- XXX
15501845 let !x = unsafeInlineIO $ do
1551− r <- peekByteIndex arrContents p
1846+ r <- peekAt arrContents p
15521847 return r
15531848 in c x (go (PTR_NEXT(p,a)))
15541849-}
@@ -1565,21 +1860,21 @@ toList MutArray{..} = liftIO $ go arrStart
15651860
15661861 go p | assert (p <= arrEnd) (p == arrEnd) = return []
15671862 go p = do
1568− x <- peekByteIndex p arrContents
1863+ x <- peekAt p arrContents
15691864 (:) x <$> go (INDEX_NEXT(p,a))
15701865
1571−{-# INLINE_NORMAL toStreamDWith #-}
1572−toStreamDWith ::
1866+{-# INLINE_NORMAL toStreamWith #-}
1867+toStreamWith ::
15731868 forall m a. (Monad m, Unbox a)
15741869 => (forall b. IO b -> m b) -> MutArray a -> D.Stream m a
1575−toStreamDWith liftio MutArray{..} = D.Stream step arrStart
1870+toStreamWith liftio MutArray{..} = D.Stream step arrStart
15761871
15771872 where
15781873
15791874 {-# INLINE_LATE step #-}
15801875 step _ p | assert (p <= arrEnd) (p == arrEnd) = return D.Stop
15811876 step _ p = liftio $ do
1582− r <- peekByteIndex p arrContents
1877+ r <- peekAt p arrContents
15831878 return $ D.Yield r (INDEX_NEXT(p,a))
15841879
15851880-- | Convert a 'MutArray' into a stream.
@@ -1588,7 +1883,7 @@ toStreamDWith liftio MutArray{..} = D.Stream step arrStart
15881883--
15891884{-# INLINE_NORMAL read #-}
15901885read :: forall m a. (MonadIO m, Unbox a) => MutArray a -> D.Stream m a
1591−read = toStreamDWith liftIO
1886+read = toStreamWith liftIO
15921887
15931888{-# INLINE toStreamKWith #-}
15941889toStreamKWith ::
@@ -1600,18 +1895,18 @@ toStreamKWith liftio MutArray{..} = go arrStart
16001895
16011896 go p | assert (p <= arrEnd) (p == arrEnd) = K.nil
16021897 | otherwise =
1603− let elemM = peekByteIndex p arrContents
1898+ let elemM = peekAt p arrContents
16041899 in liftio elemM `K.consM` go (INDEX_NEXT(p,a))
16051900
16061901{-# INLINE toStreamK #-}
16071902toStreamK :: forall m a. (MonadIO m, Unbox a) => MutArray a -> StreamK m a
16081903toStreamK = toStreamKWith liftIO
16091904
1610−{-# INLINE_NORMAL toStreamDRevWith #-}
1611−toStreamDRevWith ::
1905+{-# INLINE_NORMAL toStreamRevWith #-}
1906+toStreamRevWith ::
16121907 forall m a. (Monad m, Unbox a)
16131908 => (forall b. IO b -> m b) -> MutArray a -> D.Stream m a
1614−toStreamDRevWith liftio MutArray{..} =
1909+toStreamRevWith liftio MutArray{..} =
16151910 let p = INDEX_PREV(arrEnd,a)
16161911 in D.Stream step p
16171912
@@ -1620,7 +1915,7 @@ toStreamDRevWith liftio MutArray{..} =
16201915 {-# INLINE_LATE step #-}
16211916 step _ p | p < arrStart = return D.Stop
16221917 step _ p = liftio $ do
1623− r <- peekByteIndex p arrContents
1918+ r <- peekAt p arrContents
16241919 return $ D.Yield r (INDEX_PREV(p,a))
16251920
16261921-- | Convert a 'MutArray' into a stream in reverse order.
@@ -1629,7 +1924,7 @@ toStreamDRevWith liftio MutArray{..} =
16291924--
16301925{-# INLINE_NORMAL readRev #-}
16311926readRev :: forall m a. (MonadIO m, Unbox a) => MutArray a -> D.Stream m a
1632−readRev = toStreamDRevWith liftIO
1927+readRev = toStreamRevWith liftIO
16331928
16341929{-# INLINE toStreamKRevWith #-}
16351930toStreamKRevWith ::
@@ -1643,7 +1938,7 @@ toStreamKRevWith liftio MutArray {..} =
16431938
16441939 go p | p < arrStart = K.nil
16451940 | otherwise =
1646− let elemM = peekByteIndex p arrContents
1941+ let elemM = peekAt p arrContents
16471942 in liftio elemM `K.consM` go (INDEX_PREV(p,a))
16481943
16491944{-# INLINE toStreamKRev #-}
@@ -1678,7 +1973,7 @@ foldr f z arr = D.foldr f z $ read arr
16781973
16791974-- XXX Keep the bound intact to not lose any free space? Perf impact?
16801975
1681−-- | @writeAppendNUnsafe n arr@ appends up to @n@ input items to the supplied
1976+-- | @unsafeAppendN n arr@ appends up to @n@ input items to the supplied
16821977-- array.
16831978--
16841979-- Unsafe: Do not drive the fold beyond @n@ elements, it will lead to memory
@@ -1687,13 +1982,12 @@ foldr f z arr = D.foldr f z $ read arr
16871982-- Any free space left in the array after appending @n@ elements is lost.
16881983--
16891984-- /Internal/
1690−{-# INLINE_NORMAL writeAppendNUnsafe #-}
1691−writeAppendNUnsafe :: forall m a. (MonadIO m, Unbox a) =>
1985+{-# INLINE_NORMAL unsafeAppendN #-}
1986+unsafeAppendN :: forall m a. (MonadIO m, Unbox a) =>
16921987 Int
16931988 -> m (MutArray a)
16941989 -> Fold m a (MutArray a)
1695−writeAppendNUnsafe n action =
1696− fmap fromArrayUnsafe $ FL.foldlM' step initial
1990+unsafeAppendN n action = fmap fromArrayUnsafe $ FL.foldlM' step initial
16971991
16981992 where
16991993
@@ -1706,38 +2000,59 @@ writeAppendNUnsafe n action =
17062000 -- otherwise we lose that space.
17072001 arr1 <-
17082002 if free < needed
1709− then noinline reallocWith "writeAppendNUnsafeWith" (+ needed) needed arr
2003+ then noinline reallocWith "unsafeAppendN" (+ needed) needed arr
17102004 else return arr
17112005 return $ toArrayUnsafe arr1
17122006
17132007 step (ArrayUnsafe contents start end) x = do
1714− liftIO $ pokeByteIndex end contents x
2008+ liftIO $ pokeAt end contents x
17152009 return $ ArrayUnsafe contents start (INDEX_NEXT(end,a))
17162010
2011+{-# DEPRECATED writeAppendNUnsafe "Please use unsafeAppendN instead." #-}
2012+{-# INLINE writeAppendNUnsafe #-}
2013+writeAppendNUnsafe :: forall m a. (MonadIO m, Unbox a) =>
2014+ Int
2015+ -> m (MutArray a)
2016+ -> Fold m a (MutArray a)
2017+writeAppendNUnsafe = unsafeAppendN
2018+
17172019-- | Append @n@ elements to an existing array. Any free space left in the array
17182020-- after appending @n@ elements is lost.
17192021--
1720−-- >>> writeAppendN n initial = Fold.take n (MutArray.writeAppendNUnsafe n initial)
2022+-- >>> appendN n initial = Fold.take n (MutArray.unsafeAppendN n initial)
17212023--
1722−{-# INLINE_NORMAL writeAppendN #-}
2024+{-# INLINE_NORMAL appendN #-}
2025+appendN :: forall m a. (MonadIO m, Unbox a) =>
2026+ Int -> m (MutArray a) -> Fold m a (MutArray a)
2027+appendN n initial = FL.take n (unsafeAppendN n initial)
2028+
2029+-- XXX Deprecate in major
2030+-- {-# DEPRECATED writeAppendN "Please use appendN instead." #-}
2031+{-# INLINE writeAppendN #-}
17232032writeAppendN :: forall m a. (MonadIO m, Unbox a) =>
17242033 Int -> m (MutArray a) -> Fold m a (MutArray a)
1725−writeAppendN n initial = FL.take n (writeAppendNUnsafe n initial)
2034+writeAppendN = appendN
17262035
1727−-- | @writeAppendWith realloc action@ mutates the array generated by @action@ to
2036+-- | @appendWith realloc action@ mutates the array generated by @action@ to
17282037-- append the input stream. If there is no reserved space available in the
17292038-- array it is reallocated to a size in bytes determined by @realloc oldSize@,
17302039-- where @oldSize@ is the current size of the array in bytes.
17312040--
17322041-- Note that the returned array may be a mutated version of original array.
17332042--
1734−-- >>> writeAppendWith sizer = Fold.foldlM' (MutArray.snocWith sizer)
2043+-- >>> appendWith sizer = Fold.foldlM' (MutArray.snocWith sizer)
17352044--
17362045-- /Pre-release/
2046+{-# INLINE appendWith #-}
2047+appendWith :: forall m a. (MonadIO m, Unbox a) =>
2048+ (Int -> Int) -> m (MutArray a) -> Fold m a (MutArray a)
2049+appendWith sizer = FL.foldlM' (snocWith sizer)
2050+
2051+{-# DEPRECATED writeAppendWith "Please use appendWith instead." #-}
17372052{-# INLINE writeAppendWith #-}
17382053writeAppendWith :: forall m a. (MonadIO m, Unbox a) =>
17392054 (Int -> Int) -> m (MutArray a) -> Fold m a (MutArray a)
1740−writeAppendWith sizer = FL.foldlM' (snocWith sizer)
2055+writeAppendWith = appendWith
17412056
17422057-- | @append action@ mutates the array generated by @action@ to append the
17432058-- input stream. If there is no reserved space available in the array it is
@@ -1745,70 +2060,101 @@ writeAppendWith sizer = FL.foldlM' (snocWith sizer)
17452060--
17462061-- Note that the returned array may be a mutated version of original array.
17472062--
1748−-- >>> writeAppend = MutArray.writeAppendWith (* 2)
2063+-- >>> append = MutArray.appendWith (* 2)
17492064--
2065+{-# INLINE append #-}
2066+append :: forall m a. (MonadIO m, Unbox a) =>
2067+ m (MutArray a) -> Fold m a (MutArray a)
2068+append = appendWith (* 2)
2069+
2070+-- XXX Deprecate in major
2071+-- {-# DEPRECATED writeAppend "Please use append instead." #-}
17502072{-# INLINE writeAppend #-}
17512073writeAppend :: forall m a. (MonadIO m, Unbox a) =>
17522074 m (MutArray a) -> Fold m a (MutArray a)
1753−writeAppend = writeAppendWith (* 2)
2075+writeAppend = append
17542076
17552077-- XXX We can carry bound as well in the state to make sure we do not lose the
17562078-- remaining capacity. Need to check perf impact.
17572079--
1758−-- | Like 'writeNUnsafe' but takes a new array allocator @alloc size@ function
1759−-- as argument.
2080+-- | Like 'unsafeCreateOf' but takes a new array allocator @alloc size@
2081+-- function as argument.
17602082--
1761−-- >>> writeNWithUnsafe alloc n = MutArray.writeAppendNUnsafe (alloc n) n
2083+-- >>> unsafeCreateOfWith alloc n = MutArray.unsafeAppendN (alloc n) n
17622084--
17632085-- /Pre-release/
1764−{-# INLINE_NORMAL writeNWithUnsafe #-}
1765−writeNWithUnsafe :: forall m a. (MonadIO m, Unbox a)
2086+{-# INLINE_NORMAL unsafeCreateOfWith #-}
2087+unsafeCreateOfWith :: forall m a. (MonadIO m, Unbox a)
17662088 => (Int -> m (MutArray a)) -> Int -> Fold m a (MutArray a)
1767−writeNWithUnsafe alloc n = fromArrayUnsafe <$> FL.foldlM' step initial
2089+unsafeCreateOfWith alloc n = fromArrayUnsafe <$> FL.foldlM' step initial
17682090
17692091 where
17702092
17712093 initial = toArrayUnsafe <$> alloc (max n 0)
17722094
17732095 step (ArrayUnsafe contents start end) x = do
1774− liftIO $ pokeByteIndex end contents x
2096+ liftIO $ pokeAt end contents x
17752097 return
17762098 $ ArrayUnsafe contents start (INDEX_NEXT(end,a))
17772099
2100+{-# DEPRECATED writeNWithUnsafe "Please use unsafeCreateOfWith instead." #-}
2101+{-# INLINE writeNWithUnsafe #-}
2102+writeNWithUnsafe :: forall m a. (MonadIO m, Unbox a)
2103+ => (Int -> m (MutArray a)) -> Int -> Fold m a (MutArray a)
2104+writeNWithUnsafe = unsafeCreateOfWith
2105+
17782106{-# INLINE_NORMAL writeNUnsafeAs #-}
17792107writeNUnsafeAs :: forall m a. (MonadIO m, Unbox a)
17802108 => PinnedState -> Int -> Fold m a (MutArray a)
1781−writeNUnsafeAs ps = writeNWithUnsafe (newAs ps)
2109+writeNUnsafeAs ps = unsafeCreateOfWith (newAs ps)
17822110
1783−-- | Like 'writeN' but does not check the array bounds when writing. The fold
2111+-- | Like 'createOf' but does not check the array bounds when writing. The fold
17842112-- driver must not call the step function more than 'n' times otherwise it will
17852113-- corrupt the memory and crash. This function exists mainly because any
17862114-- conditional in the step function blocks fusion causing 10x performance
17872115-- slowdown.
17882116--
1789−-- >>> writeNUnsafe = MutArray.writeNWithUnsafe MutArray.new
2117+-- >>> unsafeCreateOf = MutArray.unsafeCreateOfWith MutArray.emptyOf
17902118--
1791−{-# INLINE_NORMAL writeNUnsafe #-}
2119+{-# INLINE_NORMAL unsafeCreateOf #-}
2120+unsafeCreateOf :: forall m a. (MonadIO m, Unbox a)
2121+ => Int -> Fold m a (MutArray a)
2122+unsafeCreateOf = writeNUnsafeAs Unpinned
2123+
2124+{-# DEPRECATED writeNUnsafe "Please use unsafeCreateOf instead." #-}
2125+{-# INLINE writeNUnsafe #-}
17922126writeNUnsafe :: forall m a. (MonadIO m, Unbox a)
17932127 => Int -> Fold m a (MutArray a)
1794−writeNUnsafe = writeNUnsafeAs Unpinned
2128+writeNUnsafe = unsafeCreateOf
17952129
1796−-- | Like 'writeNUnsafe' but creates a pinned array.
1797−{-# INLINE_NORMAL pinnedWriteNUnsafe #-}
2130+-- | Like 'unsafeCreateOf' but creates a pinned array.
2131+{-# INLINE_NORMAL unsafePinnedCreateOf #-}
2132+unsafePinnedCreateOf :: forall m a. (MonadIO m, Unbox a)
2133+ => Int -> Fold m a (MutArray a)
2134+unsafePinnedCreateOf = writeNUnsafeAs Pinned
2135+
2136+{-# DEPRECATED pinnedWriteNUnsafe "Please use unsafePinnedCreateOf instead." #-}
2137+{-# INLINE pinnedWriteNUnsafe #-}
17982138pinnedWriteNUnsafe :: forall m a. (MonadIO m, Unbox a)
17992139 => Int -> Fold m a (MutArray a)
1800−pinnedWriteNUnsafe = writeNUnsafeAs Pinned
2140+pinnedWriteNUnsafe = unsafePinnedCreateOf
18012141
1802−-- | @writeNWith alloc n@ folds a maximum of @n@ elements into an array
2142+-- | @createOfWith alloc n@ folds a maximum of @n@ elements into an array
18032143-- allocated using the @alloc@ function.
18042144--
1805−-- >>> writeNWith alloc n = Fold.take n (MutArray.writeNWithUnsafe alloc n)
1806−-- >>> writeNWith alloc n = MutArray.writeAppendN (alloc n) n
2145+-- >>> createOfWith alloc n = Fold.take n (MutArray.unsafeCreateOfWith alloc n)
2146+-- >>> createOfWith alloc n = MutArray.appendN (alloc n) n
18072147--
1808−{-# INLINE_NORMAL writeNWith #-}
2148+{-# INLINE_NORMAL createOfWith #-}
2149+createOfWith :: forall m a. (MonadIO m, Unbox a)
2150+ => (Int -> m (MutArray a)) -> Int -> Fold m a (MutArray a)
2151+createOfWith alloc n = FL.take n (unsafeCreateOfWith alloc n)
2152+
2153+-- {-# DEPRECATED writeNWith "Please use createOfWith instead." #-}
2154+{-# INLINE writeNWith #-}
18092155writeNWith :: forall m a. (MonadIO m, Unbox a)
18102156 => (Int -> m (MutArray a)) -> Int -> Fold m a (MutArray a)
1811−writeNWith alloc n = FL.take n (writeNWithUnsafe alloc n)
2157+writeNWith = createOfWith
18122158
18132159{-# INLINE_NORMAL writeNAs #-}
18142160writeNAs ::
@@ -1816,28 +2162,42 @@ writeNAs ::
18162162 => PinnedState
18172163 -> Int
18182164 -> Fold m a (MutArray a)
1819−writeNAs ps = writeNWith (newAs ps)
2165+writeNAs ps = createOfWith (newAs ps)
18202166
1821−-- | @writeN n@ folds a maximum of @n@ elements from the input stream to an
2167+-- | @createOf n@ folds a maximum of @n@ elements from the input stream to an
18222168-- 'MutArray'.
18232169--
1824−-- >>> writeN = MutArray.writeNWith MutArray.new
1825−-- >>> writeN n = Fold.take n (MutArray.writeNUnsafe n)
1826−-- >>> writeN n = MutArray.writeAppendN n (MutArray.new n)
2170+-- >>> createOf = MutArray.createOfWith MutArray.new
2171+-- >>> createOf n = Fold.take n (MutArray.unsafeCreateOf n)
2172+-- >>> createOf n = MutArray.appendN n (MutArray.emptyOf n)
18272173--
1828−{-# INLINE_NORMAL writeN #-}
2174+{-# INLINE_NORMAL createOf #-}
2175+createOf :: forall m a. (MonadIO m, Unbox a) => Int -> Fold m a (MutArray a)
2176+createOf = writeNAs Unpinned
2177+
2178+-- XXX Deprecate in major
2179+-- {-# DEPRECATED writeN "Please use createOf instead." #-}
2180+{-# INLINE writeN #-}
18292181writeN :: forall m a. (MonadIO m, Unbox a) => Int -> Fold m a (MutArray a)
1830−writeN = writeNAs Unpinned
2182+writeN = createOf
2183+
2184+-- | Like 'createOf' but creates a pinned array.
2185+{-# INLINE_NORMAL pinnedCreateOf #-}
2186+pinnedCreateOf ::
2187+ forall m a. (MonadIO m, Unbox a)
2188+ => Int
2189+ -> Fold m a (MutArray a)
2190+pinnedCreateOf = writeNAs Pinned
18312191
1832−-- | Like 'writeN' but creates a pinned array.
1833−{-# INLINE_NORMAL pinnedWriteN #-}
2192+{-# DEPRECATED pinnedWriteN "Please use pinnedCreateOf instead." #-}
2193+{-# INLINE pinnedWriteN #-}
18342194pinnedWriteN ::
18352195 forall m a. (MonadIO m, Unbox a)
18362196 => Int
18372197 -> Fold m a (MutArray a)
1838−pinnedWriteN = writeNAs Pinned
2198+pinnedWriteN = pinnedCreateOf
18392199
1840−-- | Like writeNWithUnsafe but writes the array in reverse order.
2200+-- | Like unsafeCreateOfWith but writes the array in reverse order.
18412201--
18422202-- /Internal/
18432203{-# INLINE_NORMAL writeRevNWithUnsafe #-}
@@ -1854,11 +2214,11 @@ writeRevNWithUnsafe alloc n = fromArrayUnsafe <$> FL.foldlM' step initial
18542214
18552215 step (ArrayUnsafe contents start end) x = do
18562216 let ptr = INDEX_PREV(start,a)
1857− liftIO $ pokeByteIndex ptr contents x
2217+ liftIO $ pokeAt ptr contents x
18582218 return
18592219 $ ArrayUnsafe contents ptr end
18602220
1861−-- | Like writeNWith but writes the array in reverse order.
2221+-- | Like createOfWith but writes the array in reverse order.
18622222--
18632223-- /Internal/
18642224{-# INLINE_NORMAL writeRevNWith #-}
@@ -1866,31 +2226,36 @@ writeRevNWith :: forall m a. (MonadIO m, Unbox a)
18662226 => (Int -> m (MutArray a)) -> Int -> Fold m a (MutArray a)
18672227writeRevNWith alloc n = FL.take n (writeRevNWithUnsafe alloc n)
18682228
1869−-- | Like writeN but writes the array in reverse order.
2229+-- | Like 'createOf' but writes the array in reverse order.
18702230--
18712231-- /Pre-release/
1872−{-# INLINE_NORMAL writeRevN #-}
2232+{-# INLINE_NORMAL revCreateOf #-}
2233+revCreateOf :: forall m a. (MonadIO m, Unbox a) => Int -> Fold m a (MutArray a)
2234+revCreateOf = writeRevNWith new
2235+
2236+{-# DEPRECATED writeRevN "Please use revCreateOf instead." #-}
2237+{-# INLINE writeRevN #-}
18732238writeRevN :: forall m a. (MonadIO m, Unbox a) => Int -> Fold m a (MutArray a)
1874−writeRevN = writeRevNWith new
2239+writeRevN = revCreateOf
18752240
18762241-- | @pinnedWriteNAligned align n@ folds a maximum of @n@ elements from the
18772242-- input stream to a 'MutArray' aligned to the given size.
18782243--
1879−-- >>> pinnedWriteNAligned align = MutArray.writeNWith (MutArray.pinnedNewAligned align)
1880−-- >>> pinnedWriteNAligned align n = MutArray.writeAppendN n (MutArray.pinnedNewAligned align n)
2244+-- >>> pinnedWriteNAligned align = MutArray.createOfWith (MutArray.pinnedNewAligned align)
2245+-- >>> pinnedWriteNAligned align n = MutArray.appendN n (MutArray.pinnedNewAligned align n)
18812246--
18822247-- /Pre-release/
18832248--
18842249{-# INLINE_NORMAL pinnedWriteNAligned #-}
18852250pinnedWriteNAligned :: forall m a. (MonadIO m, Unbox a)
18862251 => Int -> Int -> Fold m a (MutArray a)
1887−pinnedWriteNAligned align = writeNWith (pinnedNewAligned align)
2252+pinnedWriteNAligned align = createOfWith (pinnedNewAligned align)
18882253
18892254-- XXX Buffer to a list instead?
1890−--
2255+
18912256-- | Buffer a stream into a stream of arrays.
18922257--
1893−-- >>> writeChunks n = Fold.many (MutArray.writeN n) Fold.toStreamK
2258+-- >>> buildChunks n = Fold.many (MutArray.createOf n) Fold.toStreamK
18942259--
18952260-- Breaking an array into an array stream can be useful to consume a large
18962261-- array sequentially such that memory of the array is released incrementatlly.
@@ -1899,37 +2264,43 @@ pinnedWriteNAligned align = writeNWith (pinnedNewAligned align)
18992264--
19002265-- /Unimplemented/
19012266--
1902−{-# INLINE_NORMAL writeChunks #-}
2267+{-# INLINE_NORMAL buildChunks #-}
2268+buildChunks :: (MonadIO m, Unbox a) =>
2269+ Int -> Fold m a (StreamK n (MutArray a))
2270+buildChunks n = FL.many (createOf n) FL.toStreamK
2271+
2272+{-# DEPRECATED writeChunks "Please use buildChunks instead." #-}
2273+{-# INLINE writeChunks #-}
19032274writeChunks :: (MonadIO m, Unbox a) =>
19042275 Int -> Fold m a (StreamK n (MutArray a))
1905−writeChunks n = FL.many (writeN n) FL.toStreamK
2276+writeChunks = buildChunks
19062277
19072278{-# INLINE_NORMAL writeWithAs #-}
19082279writeWithAs :: forall m a. (MonadIO m, Unbox a)
19092280 => PinnedState -> Int -> Fold m a (MutArray a)
1910−-- writeWithAs ps n = FL.rmapM rightSize $ writeAppendWith (* 2) (newAs ps n)
2281+-- writeWithAs ps n = FL.rmapM rightSize $ appendWith (* 2) (newAs ps n)
19112282writeWithAs ps elemCount =
19122283 FL.rmapM extract $ FL.foldlM' step initial
19132284
19142285 where
19152286
19162287 initial = do
1917− when (elemCount < 0) $ error "writeWith: elemCount is negative"
1918− liftIO $ newAs ps elemCount
2288+ when (elemCount < 0) $ error "createWith: elemCount is negative"
2289+ newAs ps elemCount
19192290
19202291 step arr@(MutArray _ start end bound) x
19212292 | INDEX_NEXT(end,a) > bound = do
19222293 let oldSize = end - start
19232294 newSize = max (oldSize * 2) 1
1924− arr1 <- liftIO $ reallocExplicit (SIZE_OF(a)) newSize arr
2295+ arr1 <- liftIO $ reallocExplicitAs ps (SIZE_OF(a)) newSize arr
19252296 snocUnsafe arr1 x
19262297 step arr x = snocUnsafe arr x
19272298
19282299 extract = liftIO . rightSize
19292300
1930−-- XXX Compare writeWith with fromStreamD which uses an array of streams
1931−-- implementation. We can write this using writeChunks above if that is faster.
1932−-- If writeWith is faster then we should use that to implement
2301+-- XXX Compare createWith with fromStreamD which uses an array of streams
2302+-- implementation. We can write this using buildChunks above if that is faster.
2303+-- If createWith is faster then we should use that to implement
19332304-- fromStreamD.
19342305--
19352306-- XXX The realloc based implementation needs to make one extra copy if we use
@@ -1944,39 +2315,56 @@ writeWithAs ps elemCount =
19442315-- XXX check if GHC's memory allocator is efficient enough. We can try the C
19452316-- malloc to compare against.
19462317
1947−-- | @writeWith minCount@ folds the whole input to a single array. The array
2318+-- | @createWith minCount@ folds the whole input to a single array. The array
19482319-- starts at a size big enough to hold minCount elements, the size is doubled
19492320-- every time the array needs to be grown.
19502321--
19512322-- /Caution! Do not use this on infinite streams./
19522323--
1953−-- >>> f n = MutArray.writeAppendWith (* 2) (MutArray.new n)
1954−-- >>> writeWith n = Fold.rmapM MutArray.rightSize (f n)
1955−-- >>> writeWith n = Fold.rmapM MutArray.fromArrayStreamK (MutArray.writeChunks n)
2324+-- >>> f n = MutArray.appendWith (* 2) (MutArray.emptyOf n)
2325+-- >>> createWith n = Fold.rmapM MutArray.rightSize (f n)
2326+-- >>> createWith n = Fold.rmapM MutArray.fromChunksK (MutArray.buildChunks n)
19562327--
19572328-- /Pre-release/
1958−{-# INLINE_NORMAL writeWith #-}
2329+{-# INLINE_NORMAL createWith #-}
2330+createWith :: forall m a. (MonadIO m, Unbox a)
2331+ => Int -> Fold m a (MutArray a)
2332+-- createWith n = FL.rmapM rightSize $ appendWith (* 2) (emptyOf n)
2333+createWith = writeWithAs Unpinned
2334+
2335+{-# DEPRECATED writeWith "Please use createWith instead." #-}
2336+{-# INLINE writeWith #-}
19592337writeWith :: forall m a. (MonadIO m, Unbox a)
19602338 => Int -> Fold m a (MutArray a)
1961−-- writeWith n = FL.rmapM rightSize $ writeAppendWith (* 2) (new n)
1962−writeWith = writeWithAs Unpinned
2339+writeWith = createWith
19632340
19642341-- | Fold the whole input to a single array.
19652342--
1966−-- Same as 'writeWith' using an initial array size of 'arrayChunkBytes' bytes
2343+-- Same as 'createWith' using an initial array size of 'arrayChunkBytes' bytes
19672344-- rounded up to the element size.
19682345--
19692346-- /Caution! Do not use this on infinite streams./
19702347--
2348+{-# INLINE create #-}
2349+create :: forall m a. (MonadIO m, Unbox a) => Fold m a (MutArray a)
2350+create = createWith (allocBytesToElemCount (undefined :: a) arrayChunkBytes)
2351+
2352+-- XXX Deprecate in major
2353+-- {-# DEPRECATED write "Please use create instead." #-}
19712354{-# INLINE write #-}
19722355write :: forall m a. (MonadIO m, Unbox a) => Fold m a (MutArray a)
1973−write = writeWith (allocBytesToElemCount (undefined :: a) arrayChunkBytes)
2356+write = create
2357+
2358+-- | Like 'create' but creates a pinned array.
2359+{-# INLINE pinnedCreate #-}
2360+pinnedCreate :: forall m a. (MonadIO m, Unbox a) => Fold m a (MutArray a)
2361+pinnedCreate =
2362+ writeWithAs Pinned (allocBytesToElemCount (undefined :: a) arrayChunkBytes)
19742363
1975−-- | Like 'write' but creates a pinned array.
2364+{-# DEPRECATED pinnedWrite "Please use pinnedCreate instead." #-}
19762365{-# INLINE pinnedWrite #-}
19772366pinnedWrite :: forall m a. (MonadIO m, Unbox a) => Fold m a (MutArray a)
1978−pinnedWrite =
1979− writeWithAs Pinned (allocBytesToElemCount (undefined :: a) arrayChunkBytes)
2367+pinnedWrite = pinnedCreate
19802368
19812369-------------------------------------------------------------------------------
19822370-- construct from streams, known size
@@ -1986,25 +2374,31 @@ pinnedWrite =
19862374fromStreamDNAs :: forall m a. (MonadIO m, Unbox a)
19872375 => PinnedState -> Int -> D.Stream m a -> m (MutArray a)
19882376fromStreamDNAs ps limit str = do
1989− (arr :: MutArray a) <- liftIO $ newAs ps limit
2377+ (arr :: MutArray a) <- newAs ps limit
19902378 end <- D.foldlM' (fwrite (arrContents arr)) (return $ arrEnd arr) $ D.take limit str
19912379 return $ arr {arrEnd = end}
19922380
19932381 where
19942382
19952383 fwrite arrContents ptr x = do
1996− liftIO $ pokeByteIndex ptr arrContents x
2384+ liftIO $ pokeAt ptr arrContents x
19972385 return $ INDEX_NEXT(ptr,a)
19982386
1999−-- | Use the 'writeN' fold instead.
2387+-- | Use the 'createOf' fold instead.
20002388--
2001−-- >>> fromStreamDN n = Stream.fold (MutArray.writeN n)
2389+-- >>> fromStreamN n = Stream.fold (MutArray.createOf n)
20022390--
2003−{-# INLINE_NORMAL fromStreamDN #-}
2391+{-# INLINE_NORMAL fromStreamN #-}
2392+fromStreamN :: forall m a. (MonadIO m, Unbox a)
2393+ => Int -> D.Stream m a -> m (MutArray a)
2394+-- fromStreamDN n = D.fold (createOf n)
2395+fromStreamN = fromStreamDNAs Unpinned
2396+
2397+{-# DEPRECATED fromStreamDN "Please use fromStreamN instead." #-}
2398+{-# INLINE fromStreamDN #-}
20042399fromStreamDN :: forall m a. (MonadIO m, Unbox a)
20052400 => Int -> D.Stream m a -> m (MutArray a)
2006−-- fromStreamDN n = D.fold (writeN n)
2007−fromStreamDN = fromStreamDNAs Unpinned
2401+fromStreamDN = fromStreamN
20082402
20092403-- | Create a 'MutArray' from the first N elements of a list. The array is
20102404-- allocated to size N, if the list terminates before N elements then the
@@ -2024,16 +2418,70 @@ pinnedFromListN n xs = fromStreamDNAs Pinned n $ D.fromList xs
20242418-- /Pre-release/
20252419{-# INLINE fromListRevN #-}
20262420fromListRevN :: (MonadIO m, Unbox a) => Int -> [a] -> m (MutArray a)
2027−fromListRevN n xs = D.fold (writeRevN n) $ D.fromList xs
2421+fromListRevN n xs = D.fold (revCreateOf n) $ D.fromList xs
2422+
2423+-- | Convert a pure stream in Identity monad to a mutable array.
2424+{-# INLINABLE fromPureStreamN #-}
2425+fromPureStreamN :: (MonadIO m, Unbox a) =>
2426+ Int -> Stream Identity a -> m (MutArray a)
2427+fromPureStreamN n xs =
2428+ D.fold (createOf n) $ D.morphInner (return . runIdentity) xs
20282429
20292430-- | Convert a pure stream in Identity monad to a mutable array.
20302431{-# INLINABLE fromPureStream #-}
20312432fromPureStream :: (MonadIO m, Unbox a) => Stream Identity a -> m (MutArray a)
20322433fromPureStream xs =
2033− liftIO $ D.fold write $ D.morphInner (return . runIdentity) xs
2434+ D.fold create $ D.morphInner (return . runIdentity) xs
2435+
2436+{-# INLINABLE fromPtrN #-}
2437+fromPtrN :: MonadIO m => Int -> Ptr Word8 -> m (MutArray Word8)
2438+fromPtrN len addr = do
2439+ -- memcpy is better than stream copy when the size is known.
2440+ -- XXX We can implement a stream copy in a similar way by streaming Word64
2441+ -- first and then remaining Word8.
2442+ arr <- new len
2443+ _ <- unsafeAsPtr arr
2444+ (\ptr -> liftIO $ c_memcpy ptr addr (fromIntegral len))
2445+ return (arr {arrEnd = len})
2446+
2447+{-# INLINABLE fromByteStr# #-}
2448+fromByteStr# :: MonadIO m => Addr# -> m (MutArray Word8)
2449+fromByteStr# addr = do
2450+ -- It is better to count the size first and allocate exact space.
2451+ -- Also, memcpy is better than stream copy when the size is known.
2452+ -- C strlen compares 4 bytes at a time, so is better than the stream
2453+ -- version. https://github.com/bminor/glibc/blob/master/string/strlen.c
2454+ -- XXX We can possibly use a stream of Word64 to do the same.
2455+ -- fromByteStr# addr = fromPureStream (D.fromByteStr# addr)
2456+ len <- liftIO $ c_strlen (Ptr addr)
2457+ let lenInt = fromIntegral len
2458+ arr <- new lenInt
2459+ _ <- unsafeAsPtr arr (\ptr -> liftIO $ c_memcpy ptr (Ptr addr) len)
2460+ return (arr {arrEnd = lenInt})
20342461
20352462-------------------------------------------------------------------------------
2036−-- convert stream to a single array
2463+-- convert a stream of arrays to a single array by reallocating and copying
2464+-------------------------------------------------------------------------------
2465+
2466+-- XXX Both of these implementations of splicing seem to perform equally well.
2467+-- We need to perform benchmarks over a range of sizes though.
2468+
2469+-- | Also see 'fromChunksK'.
2470+{-# INLINE fromChunksRealloced #-}
2471+fromChunksRealloced :: forall m a. (MonadIO m, Unbox a)
2472+ => Stream m (MutArray a) -> m (MutArray a)
2473+fromChunksRealloced s = do
2474+ res <- D.uncons s
2475+ case res of
2476+ Just (a, strm) -> do
2477+ arr <- D.foldlM' spliceExp (pure a) strm
2478+ -- Reallocation is exponential so there may be 50% empty space in
2479+ -- worst case. One more reallocation to reclaim the space.
2480+ rightSize arr
2481+ Nothing -> pure nil
2482+
2483+-------------------------------------------------------------------------------
2484+-- convert a stream of arrays to a single array by buffering arrays first
20372485-------------------------------------------------------------------------------
20382486
20392487{-# INLINE arrayStreamKLength #-}
@@ -2043,34 +2491,72 @@ arrayStreamKLength as = K.foldl' (+) 0 (K.map length as)
20432491-- | Convert an array stream to an array. Note that this requires peak memory
20442492-- that is double the size of the array stream.
20452493--
2494+{-# INLINE fromChunkskAs #-}
2495+fromChunkskAs :: (Unbox a, MonadIO m) =>
2496+ PinnedState -> StreamK m (MutArray a) -> m (MutArray a)
2497+fromChunkskAs ps as = do
2498+ len <- arrayStreamKLength as
2499+ arr <- newAs ps len
2500+ -- XXX is StreamK fold faster or StreamD fold?
2501+ K.foldlM' spliceUnsafe (pure arr) as
2502+ -- fromStreamDN len $ D.unfoldMany reader $ D.fromStreamK as
2503+
2504+-- XXX Need to compare this with fromChunks and fromChunkList and keep the
2505+-- fastest or simplest one if all are equally fast.
2506+
2507+-- | Convert an array stream to an array. Note that this requires peak memory
2508+-- that is double the size of the array stream.
2509+--
2510+-- Also see 'fromChunksRealloced'.
2511+--
2512+{-# INLINE fromChunksK #-}
2513+fromChunksK :: (Unbox a, MonadIO m) =>
2514+ StreamK m (MutArray a) -> m (MutArray a)
2515+fromChunksK = fromChunkskAs Unpinned
2516+
2517+{-# DEPRECATED fromArrayStreamK "Please use fromChunksK instead." #-}
20462518{-# INLINE fromArrayStreamK #-}
20472519fromArrayStreamK :: (Unbox a, MonadIO m) =>
20482520 StreamK m (MutArray a) -> m (MutArray a)
2049−fromArrayStreamK as = do
2050− len <- arrayStreamKLength as
2051− fromStreamDN len $ D.unfoldMany reader $ D.fromStreamK as
2521+fromArrayStreamK = fromChunksK
20522522
20532523{-# INLINE fromStreamDAs #-}
20542524fromStreamDAs ::
20552525 (MonadIO m, Unbox a) => PinnedState -> D.Stream m a -> m (MutArray a)
2056−fromStreamDAs ps m = arrayStreamKFromStreamDAs ps m >>= fromArrayStreamK
2526+fromStreamDAs ps m =
2527+ arrayStreamKFromStreamDAs Unpinned m >>= fromChunkskAs ps
20572528
2529+-- | Create an 'Array' from a stream. This is useful when we want to create a
2530+-- single array from a stream of unknown size. 'createOf' is at least twice
2531+-- as efficient when the size is already known.
2532+--
2533+-- Note that if the input stream is too large memory allocation for the array
2534+-- may fail. When the stream size is not known, `chunksOf` followed by
2535+-- processing of indvidual arrays in the resulting stream should be preferred.
2536+--
2537+-- /Pre-release/
2538+{-# INLINE fromStream #-}
2539+fromStream :: (MonadIO m, Unbox a) => Stream m a -> m (MutArray a)
2540+fromStream = fromStreamDAs Unpinned
2541+
2542+-- fromStream (Stream m) = P.fold create m
20582543-- CAUTION: a very large number (millions) of arrays can degrade performance
20592544-- due to GC overhead because we need to buffer the arrays before we flatten
20602545-- all the arrays.
20612546--
2062−-- XXX Compare if this is faster or "fold write".
2547+-- XXX Compare if this is faster or "fold create".
20632548--
20642549-- | We could take the approach of doubling the memory allocation on each
20652550-- overflow. This would result in more or less the same amount of copying as in
20662551-- the chunking approach. However, if we have to shrink in the end then it may
20672552-- result in an extra copy of the entire data.
20682553--
2069−-- >>> fromStreamD = StreamD.fold MutArray.write
2554+-- >>> fromStreamD = StreamD.fold MutArray.create
20702555--
20712556{-# INLINE fromStreamD #-}
2557+{-# DEPRECATED fromStreamD "Please use fromStream instead." #-}
20722558fromStreamD :: (MonadIO m, Unbox a) => D.Stream m a -> m (MutArray a)
2073−fromStreamD = fromStreamDAs Unpinned
2559+fromStreamD = fromStream
20742560
20752561-- | Create a 'MutArray' from a list. The list must be of finite size.
20762562--
@@ -2095,22 +2581,44 @@ fromListRev xs = fromListRevN (Prelude.length xs) xs
20952581-- Cloning
20962582-------------------------------------------------------------------------------
20972583
2584+-- Arrays are aligned on 64-bit boundaries. The fastest way to copy an array is
2585+-- to unsafeCast it to Word64, read it, write it to Word64 array and unsafeCast
2586+-- it again. We can use SIMD read/write as well.
2587+
20982588{-# INLINE cloneAs #-}
2099−cloneAs :: MonadIO m => PinnedState -> MutArray a -> m (MutArray a)
2589+cloneAs ::
2590+ ( MonadIO m
2591+#ifdef DEVBUILD
2592+ , Unbox a
2593+#endif
2594+ )
2595+ => PinnedState -> MutArray a -> m (MutArray a)
21002596cloneAs ps src =
2101− liftIO $ do
2597+ do
21022598 let startSrc = arrStart src
21032599 srcLen = arrEnd src - startSrc
2104− newArrContents <- Unboxed.newBytesAs ps srcLen
2105− putSliceUnsafe (arrContents src) startSrc newArrContents 0 srcLen
2600+ newArrContents <-
2601+ Unboxed.cloneSliceUnsafeAs ps startSrc srcLen (arrContents src)
21062602 return $ MutArray newArrContents 0 srcLen srcLen
21072603
21082604{-# INLINE clone #-}
2109−clone :: MonadIO m => MutArray a -> m (MutArray a)
2605+clone ::
2606+ ( MonadIO m
2607+#ifdef DEVBUILD
2608+ , Unbox a
2609+#endif
2610+ )
2611+ => MutArray a -> m (MutArray a)
21102612clone = cloneAs Unpinned
21112613
21122614{-# INLINE pinnedClone #-}
2113−pinnedClone :: MonadIO m => MutArray a -> m (MutArray a)
2615+pinnedClone ::
2616+ ( MonadIO m
2617+#ifdef DEVBUILD
2618+ , Unbox a
2619+#endif
2620+ )
2621+ => MutArray a -> m (MutArray a)
21142622pinnedClone = cloneAs Pinned
21152623
21162624-------------------------------------------------------------------------------
@@ -2125,17 +2633,16 @@ spliceCopy :: forall m a. MonadIO m =>
21252633 Unbox a =>
21262634#endif
21272635 MutArray a -> MutArray a -> m (MutArray a)
2128−spliceCopy arr1 arr2 = liftIO $ do
2636+spliceCopy arr1 arr2 = do
21292637 let start1 = arrStart arr1
21302638 start2 = arrStart arr2
21312639 len1 = arrEnd arr1 - start1
21322640 len2 = arrEnd arr2 - start2
2133− let newLen = len1 + len2
2641+ let len = len1 + len2
21342642 newArrContents <-
21352643 if Unboxed.isPinned (arrContents arr1)
2136− then Unboxed.pinnedNewBytes newLen
2137− else Unboxed.newBytes newLen
2138− let len = len1 + len2
2644+ then liftIO $ Unboxed.pinnedNew len
2645+ else liftIO $ Unboxed.new len
21392646 putSliceUnsafe (arrContents arr1) start1 newArrContents 0 len1
21402647 putSliceUnsafe (arrContents arr2) start2 newArrContents len1 len2
21412648 return $ MutArray newArrContents 0 len len
@@ -2147,7 +2654,7 @@ spliceCopy arr1 arr2 = liftIO $ do
21472654spliceUnsafe :: MonadIO m =>
21482655 MutArray a -> MutArray a -> m (MutArray a)
21492656spliceUnsafe dst src =
2150− liftIO $ do
2657+ do
21512658 let startSrc = arrStart src
21522659 srcLen = arrEnd src - startSrc
21532660 endDst = arrEnd dst
@@ -2169,7 +2676,7 @@ spliceWith :: forall m a. (MonadIO m, Unbox a) =>
21692676 (Int -> Int -> Int) -> MutArray a -> MutArray a -> m (MutArray a)
21702677spliceWith sizer dst@(MutArray _ start end bound) src = do
21712678{-
2172− let f = writeAppendWith (`sizer` byteLength src) (return dst)
2679+ let f = appendWith (`sizer` byteLength src) (return dst)
21732680 in D.fold f (toStreamD src)
21742681-}
21752682 assert (end <= bound) (return ())
@@ -2185,7 +2692,7 @@ spliceWith sizer dst@(MutArray _ start end bound) src = do
21852692 $ "splice: newSize is less than the total size "
21862693 ++ "of arrays being appended. Please check the "
21872694 ++ "sizer function passed."
2188− liftIO $ realloc newSizeInBytes dst
2695+ realloc newSizeInBytes dst
21892696 else return dst
21902697 spliceUnsafe dst1 src
21912698
@@ -2222,14 +2729,24 @@ spliceExp = spliceWith (\l1 l2 -> max (l1 * 2) (l1 + l2))
22222729-- Splitting
22232730-------------------------------------------------------------------------------
22242731
2732+-- | Generate a stream of array slices using a predicate. The array element
2733+-- matching the predicate is dropped.
2734+--
2735+-- /Pre-release/
2736+{-# INLINE splitOn #-}
2737+splitOn :: (MonadIO m, Unbox a) =>
2738+ (a -> Bool) -> MutArray a -> Stream m (MutArray a)
2739+splitOn predicate arr =
2740+ fmap (\(i, len) -> getSliceUnsafe i len arr)
2741+ $ D.indexOnSuffix predicate (read arr)
2742+
22252743-- | Drops the separator byte
22262744{-# INLINE breakOn #-}
22272745breakOn :: MonadIO m
22282746 => Word8 -> MutArray Word8 -> m (MutArray Word8, Maybe (MutArray Word8))
2229−breakOn sep arr@MutArray{..} = asPtrUnsafe arr $ \p -> liftIO $ do
2230− -- XXX Instead of using asPtrUnsafe (pinning memory) we can pass unlifted
2231− -- Addr# to memchr and it should be safe (from ghc 8.4).
2747+breakOn sep arr@MutArray{..} = unsafeAsPtr arr $ \p -> liftIO $ do
22322748 -- XXX We do not need memchr here, we can use a Haskell equivalent.
2749+ -- Need efficient stream based primitives that work on Word64.
22332750 loc <- c_memchr p sep (fromIntegral $ byteLength arr)
22342751 let sepIndex = loc `minusPtr` p
22352752 return $
@@ -2250,32 +2767,41 @@ breakOn sep arr@MutArray{..} = asPtrUnsafe arr $ \p -> liftIO $ do
22502767 }
22512768 )
22522769
2770+-- | Like 'splitAt' but does not check whether the index is valid.
2771+--
2772+{-# INLINE unsafeSplitAt #-}
2773+unsafeSplitAt :: forall a. Unbox a =>
2774+ Int -> MutArray a -> (MutArray a, MutArray a)
2775+unsafeSplitAt i MutArray{..} =
2776+ let off = i * SIZE_OF(a)
2777+ p = arrStart + off
2778+ in ( MutArray
2779+ { arrContents = arrContents
2780+ , arrStart = arrStart
2781+ , arrEnd = p
2782+ , arrBound = p
2783+ }
2784+ , MutArray
2785+ { arrContents = arrContents
2786+ , arrStart = p
2787+ , arrEnd = arrEnd
2788+ , arrBound = arrBound
2789+ }
2790+ )
2791+
22532792-- | Create two slices of an array without copying the original array. The
22542793-- specified index @i@ is the first index of the second slice.
22552794--
2795+{-# INLINE splitAt #-}
22562796splitAt :: forall a. Unbox a => Int -> MutArray a -> (MutArray a, MutArray a)
2257−splitAt i arr@MutArray{..} =
2797+splitAt i arr =
22582798 let maxIndex = length arr - 1
22592799 in if i < 0
22602800 then error "sliceAt: negative array index"
22612801 else if i > maxIndex
22622802 then error $ "sliceAt: specified array index " ++ show i
22632803 ++ " is beyond the maximum index " ++ show maxIndex
2264− else let off = i * SIZE_OF(a)
2265− p = arrStart + off
2266− in ( MutArray
2267− { arrContents = arrContents
2268− , arrStart = arrStart
2269− , arrEnd = p
2270− , arrBound = p
2271− }
2272− , MutArray
2273− { arrContents = arrContents
2274− , arrStart = p
2275− , arrEnd = arrEnd
2276− , arrBound = arrBound
2277− }
2278− )
2804+ else unsafeSplitAt i arr
22792805
22802806-------------------------------------------------------------------------------
22812807-- Casting
@@ -2325,13 +2851,16 @@ cast arr =
23252851-- guarantees that unsafe calls will be performed in the calling thread. Making
23262852-- it safe to pass heap-allocated objects to unsafe functions.
23272853
2328−-- Unsafe because of direct pointer operations. The user must ensure that they
2329−-- are writing within the legal bounds of the array. Should we just name it
2330−-- asPtr, the unsafety is implicit for any pointer operations. And we are safe
2331−-- from Haskell perspective because we will be pinning the memory.
2854+-- XXX Should we just name it asPtr, the unsafety is implicit for any pointer
2855+-- operations. And we are safe from Haskell perspective because we will be
2856+-- pinning the memory.
2857+--
2858+-- XXX we cannot pass the length of the ptr here as in some cases it may not be
2859+-- available e.g. a null terminated C string. However, we can create another
2860+-- flavor of the API e.g. asPtrN.
23322861
2333−-- | Use an @MutArray a@ as @Ptr a@. This is useful when we want to pass an array
2334−-- as a pointer to some operating system call or to a "safe" FFI call.
2862+-- | Use a @MutArray a@ as @Ptr a@. This is useful when we want to pass an
2863+-- array as a pointer to some operating system call or to a "safe" FFI call.
23352864--
23362865-- If the array is not pinned it is copied to pinned memory before passing it
23372866-- to the monadic action.
@@ -2341,57 +2870,382 @@ cast arr =
23412870-- opportunistically before this operation occurs, to avoid the cost of a copy
23422871-- if possible.
23432872--
2344−-- /Unsafe/
2873+-- /Unsafe/ because of direct pointer operations. The user must ensure that
2874+-- they are writing within the legal bounds of the array.
23452875--
23462876-- /Pre-release/
23472877--
2878+{-# INLINE unsafePinnedAsPtr #-}
2879+unsafePinnedAsPtr :: MonadIO m => MutArray a -> (Ptr a -> m b) -> m b
2880+unsafePinnedAsPtr arr f =
2881+ Unboxed.unsafePinnedAsPtr
2882+ (arrContents arr) (\ptr -> f (ptr `plusPtr` arrStart arr))
2883+
2884+{-# DEPRECATED asPtrUnsafe "Please use unsafePinnedAsPtr instead." #-}
23482885{-# INLINE asPtrUnsafe #-}
23492886asPtrUnsafe :: MonadIO m => MutArray a -> (Ptr a -> m b) -> m b
2350−asPtrUnsafe arr f = do
2351− contents <- liftIO $ Unboxed.pin $ arrContents arr
2352− let !ptr = Ptr (byteArrayContents#
2353− (unsafeCoerce# (getMutableByteArray# contents)))
2887+asPtrUnsafe = unsafePinnedAsPtr
23542888
2355− r <- f (ptr `plusPtr` arrStart arr)
2356− liftIO $ touch contents
2357− return r
2889+{-# INLINE unsafeAsPtr #-}
2890+unsafeAsPtr :: MonadIO m => MutArray a -> (Ptr a -> m b) -> m b
2891+unsafeAsPtr arr f =
2892+ Unboxed.unsafeAsPtr
2893+ (arrContents arr) (\ptr -> f (ptr `plusPtr` arrStart arr))
23582894
23592895-------------------------------------------------------------------------------
23602896-- Equality
23612897-------------------------------------------------------------------------------
23622898
2363−-- | Compare the length of the arrays. If the length is equal, compare the
2364−-- lexicographical ordering of two underlying byte arrays otherwise return the
2365−-- result of length comparison.
2899+-- | Byte compare two arrays. Compare the length of the arrays. If the length
2900+-- is equal, compare the lexicographical ordering of two underlying byte arrays
2901+-- otherwise return the result of length comparison.
2902+--
2903+-- /Unsafe/: Note that the 'Unbox' instance of sum types with constructors of
2904+-- different sizes may leave some memory uninitialized which can make byte
2905+-- comparison unreliable.
23662906--
23672907-- /Pre-release/
2908+{-# INLINE byteCmp #-}
2909+byteCmp :: MonadIO m => MutArray a -> MutArray a -> m Ordering
2910+byteCmp arr1 arr2 = do
2911+ let marr1 = getMutableByteArray# (arrContents arr1)
2912+ marr2 = getMutableByteArray# (arrContents arr2)
2913+ !(I# st1#) = arrStart arr1
2914+ !(I# st2#) = arrStart arr2
2915+ !(I# len#) = byteLength arr1
2916+ case compare (byteLength arr1) (byteLength arr2) of
2917+ EQ -> do
2918+ r <- liftIO $ IO $ \s# ->
2919+ let res =
2920+ I#
2921+ (compareByteArrays#
2922+ (unsafeCoerce# marr1)
2923+ st1#
2924+ (unsafeCoerce# marr2)
2925+ st2#
2926+ len#)
2927+ in (# s#, res #)
2928+ return $ compare r 0
2929+ x -> return x
2930+
23682931{-# INLINE cmp #-}
2932+{-# DEPRECATED cmp "Please use byteCmp instead." #-}
23692933cmp :: MonadIO m => MutArray a -> MutArray a -> m Ordering
2370−cmp arr1 arr2 =
2371− liftIO
2372− $ do
2373− let marr1 = getMutableByteArray# (arrContents arr1)
2374− marr2 = getMutableByteArray# (arrContents arr2)
2375− !(I# st1#) = arrStart arr1
2376− !(I# st2#) = arrStart arr2
2377− !(I# len#) = byteLength arr1
2378− case compare (byteLength arr1) (byteLength arr2) of
2379− EQ -> do
2380− r <- liftIO $ IO $ \s# ->
2381− let res =
2382− I#
2383− (compareByteArrays#
2384− (unsafeCoerce# marr1)
2385− st1#
2386− (unsafeCoerce# marr2)
2387− st2#
2388− len#)
2389− in (# s#, res #)
2390− return $ compare r 0
2391− x -> return x
2934+cmp = byteCmp
2935+
2936+-- | Byte equality of two arrays.
2937+--
2938+-- >>> byteEq arr1 arr2 = (==) EQ $ MArray.byteCmp arr1 arr2
2939+--
2940+-- /Unsafe/: See 'byteCmp'.
2941+{-# INLINE byteEq #-}
2942+byteEq :: MonadIO m => MutArray a -> MutArray a -> m Bool
2943+byteEq arr1 arr2 = fmap (EQ ==) $ byteCmp arr1 arr2
2944+
2945+-------------------------------------------------------------------------------
2946+-- Compact
2947+-------------------------------------------------------------------------------
2948+
2949+-- Note: LE versions avoid an extra copy compared to GE. LE parser trades
2950+-- backtracking one array in lieu of avoiding a copy. However, LE and GE both
2951+-- can leave some memory unused. They can split the last array to fit it
2952+-- exactly in the space.
2953+
2954+{-# INLINE_NORMAL pCompactLeAs #-}
2955+pCompactLeAs ::
2956+ forall m a. (MonadIO m, Unbox a)
2957+ => PinnedState -> Int -> Parser (MutArray a) m (MutArray a)
2958+pCompactLeAs ps maxElems = Parser step initial extract
2959+
2960+ where
2961+
2962+ maxBytes = maxElems * SIZE_OF(a)
2963+
2964+ functionName = "Streamly.Internal.Data.MutArray.pCompactLE"
2965+
2966+ initial =
2967+ return
2968+ $ if maxElems <= 0
2969+ then error
2970+ $ functionName
2971+ ++ ": the size of arrays ["
2972+ ++ show maxElems ++ "] must be a natural number"
2973+ else Parser.IPartial Nothing
2974+
2975+ step Nothing arr =
2976+ return
2977+ $ let len = byteLength arr
2978+ in if len >= maxBytes
2979+ then Parser.Done 0 arr
2980+ else Parser.Partial 0 (Just arr)
2981+ -- XXX Split the last array to use the space more compactly.
2982+ step (Just buf) arr =
2983+ let len = byteLength buf + byteLength arr
2984+ in if len > maxBytes
2985+ then return $ Parser.Done 1 buf
2986+ else do
2987+ buf1 <-
2988+ if byteCapacity buf < maxBytes
2989+ then liftIO $ reallocExplicitAs
2990+ ps (SIZE_OF(a)) maxBytes buf
2991+ else return buf
2992+ buf2 <- spliceUnsafe buf1 arr
2993+ return $ Parser.Partial 0 (Just buf2)
2994+
2995+ extract Nothing = return $ Parser.Done 0 nil
2996+ extract (Just buf) = return $ Parser.Done 0 buf
2997+
2998+-- | Parser @pCompactLE maxElems@ coalesces adjacent arrays in the input stream
2999+-- only if the combined size would be less than or equal to @maxElems@
3000+-- elements. Note that it won't split an array if the original array is already
3001+-- larger than maxElems.
3002+--
3003+-- @maxElems@ must be greater than 0.
3004+--
3005+-- Generates unpinned arrays irrespective of the pinning status of input
3006+-- arrays.
3007+--
3008+-- /Internal/
3009+{-# INLINE pCompactLE #-}
3010+pCompactLE ::
3011+ forall m a. (MonadIO m, Unbox a)
3012+ => Int -> Parser (MutArray a) m (MutArray a)
3013+pCompactLE = pCompactLeAs Unpinned
3014+
3015+-- | Pinned version of 'pCompactLE'.
3016+{-# INLINE pPinnedCompactLE #-}
3017+pPinnedCompactLE ::
3018+ forall m a. (MonadIO m, Unbox a)
3019+ => Int -> Parser (MutArray a) m (MutArray a)
3020+pPinnedCompactLE = pCompactLeAs Pinned
3021+
3022+data SpliceState s arr
3023+ = SpliceInitial s
3024+ | SpliceBuffering s arr
3025+ | SpliceYielding arr (SpliceState s arr)
3026+ | SpliceFinish
3027+
3028+-- This mutates the first array (if it has space) to append values from the
3029+-- second one. This would work for immutable arrays as well because an
3030+-- immutable array never has additional space so a new array is allocated
3031+-- instead of mutating it.
3032+
3033+{-# INLINE_NORMAL compactLeAs #-}
3034+compactLeAs :: forall m a. (MonadIO m, Unbox a)
3035+ => PinnedState -> Int -> D.Stream m (MutArray a) -> D.Stream m (MutArray a)
3036+compactLeAs ps maxElems (D.Stream step state) =
3037+ D.Stream step' (SpliceInitial state)
3038+
3039+ where
3040+
3041+ maxBytes = maxElems * SIZE_OF(a)
3042+
3043+ functionName = "Streamly.Internal.Data.MutArray.rCompactLE"
3044+
3045+ {-# INLINE_LATE step' #-}
3046+ step' gst (SpliceInitial st) = do
3047+ when (maxElems <= 0) $
3048+ -- XXX we can pass the module string from the higher level API
3049+ error $ functionName ++ ": the size of arrays [" ++ show maxElems
3050+ ++ "] must be a natural number"
3051+ r <- step gst st
3052+ case r of
3053+ D.Yield arr s -> return $
3054+ let len = byteLength arr
3055+ in if len >= maxBytes
3056+ then D.Skip (SpliceYielding arr (SpliceInitial s))
3057+ else D.Skip (SpliceBuffering s arr)
3058+ D.Skip s -> return $ D.Skip (SpliceInitial s)
3059+ D.Stop -> return D.Stop
3060+
3061+ -- XXX Split the last array to use the space more compactly.
3062+ step' gst (SpliceBuffering st buf) = do
3063+ r <- step gst st
3064+ case r of
3065+ D.Yield arr s -> do
3066+ let len = byteLength buf + byteLength arr
3067+ if len > maxBytes
3068+ then return $
3069+ D.Skip (SpliceYielding buf (SpliceBuffering s arr))
3070+ else do
3071+ buf1 <- if byteCapacity buf < maxBytes
3072+ then liftIO $ reallocExplicitAs
3073+ ps (SIZE_OF(a)) maxBytes buf
3074+ else return buf
3075+ buf2 <- spliceUnsafe buf1 arr
3076+ return $ D.Skip (SpliceBuffering s buf2)
3077+ D.Skip s -> return $ D.Skip (SpliceBuffering s buf)
3078+ D.Stop -> return $ D.Skip (SpliceYielding buf SpliceFinish)
3079+
3080+ step' _ SpliceFinish = return D.Stop
3081+
3082+ step' _ (SpliceYielding arr next) = return $ D.Yield arr next
3083+
3084+
3085+{-# INLINE_NORMAL fCompactGeAs #-}
3086+fCompactGeAs ::
3087+ forall m a. (MonadIO m, Unbox a)
3088+ => PinnedState -> Int -> FL.Fold m (MutArray a) (MutArray a)
3089+fCompactGeAs ps minElems = Fold step initial extract extract
3090+
3091+ where
3092+
3093+ minBytes = minElems * SIZE_OF(a)
3094+
3095+ functionName = "Streamly.Internal.Data.MutArray.fCompactGE"
3096+
3097+ initial =
3098+ return
3099+ $ if minElems < 0
3100+ then error
3101+ $ functionName
3102+ ++ ": the size of arrays ["
3103+ ++ show minElems ++ "] must be a natural number"
3104+ else FL.Partial Nothing
3105+
3106+ step Nothing arr =
3107+ return
3108+ $ let len = byteLength arr
3109+ in if len >= minBytes
3110+ then FL.Done arr
3111+ else FL.Partial (Just arr)
3112+ -- XXX Buffer arrays as a list to avoid copy and reallocations
3113+ step (Just buf) arr = do
3114+ let len = byteLength buf + byteLength arr
3115+ buf1 <-
3116+ if byteCapacity buf < len
3117+ then liftIO $ reallocExplicitAs
3118+ ps (SIZE_OF(a)) (max minBytes len) buf
3119+ else return buf
3120+ buf2 <- spliceUnsafe buf1 arr
3121+ if len >= minBytes
3122+ then return $ FL.Done buf2
3123+ else return $ FL.Partial (Just buf2)
3124+
3125+ extract Nothing = return nil
3126+ extract (Just buf) = return buf
3127+
3128+-- | Fold @fCompactGE minElems@ coalesces adjacent arrays in the input stream
3129+-- until the size becomes greater than or equal to @minElems@.
3130+--
3131+-- Generates unpinned arrays irrespective of the pinning status of input
3132+-- arrays.
3133+{-# INLINE fCompactGE #-}
3134+fCompactGE ::
3135+ forall m a. (MonadIO m, Unbox a)
3136+ => Int -> FL.Fold m (MutArray a) (MutArray a)
3137+fCompactGE = fCompactGeAs Unpinned
3138+
3139+-- | Pinned version of 'fCompactGE'.
3140+{-# INLINE fPinnedCompactGE #-}
3141+fPinnedCompactGE ::
3142+ forall m a. (MonadIO m, Unbox a)
3143+ => Int -> FL.Fold m (MutArray a) (MutArray a)
3144+fPinnedCompactGE = fCompactGeAs Pinned
3145+
3146+{-# INLINE_NORMAL lCompactGeAs #-}
3147+lCompactGeAs :: forall m a. (MonadIO m, Unbox a)
3148+ => PinnedState -> Int -> Fold m (MutArray a) () -> Fold m (MutArray a) ()
3149+-- The fold version turns out to be a little bit slower.
3150+-- lCompactGeAs ps n = FL.many (fCompactGeAs ps n)
3151+lCompactGeAs ps minElems (Fold step1 initial1 _ final1) =
3152+ Fold step initial extract final
3153+
3154+ where
3155+
3156+ minBytes = minElems * SIZE_OF(a)
3157+
3158+ functionName = "Streamly.Internal.Data.MutArray.lCompactGE"
3159+
3160+ initial = do
3161+ when (minElems <= 0) $
3162+ -- XXX we can pass the module string from the higher level API
3163+ error $ functionName ++ ": the size of arrays ["
3164+ ++ show minElems ++ "] must be a natural number"
3165+
3166+ r <- initial1
3167+ return $ first (Tuple' Nothing) r
3168+
3169+ {-# INLINE runInner #-}
3170+ runInner len acc buf =
3171+ if len >= minBytes
3172+ then do
3173+ r <- step1 acc buf
3174+ case r of
3175+ FL.Done _ -> return $ FL.Done ()
3176+ FL.Partial s -> do
3177+ _ <- final1 s
3178+ res <- initial1
3179+ return $ first (Tuple' Nothing) res
3180+ else return $ FL.Partial $ Tuple' (Just buf) acc
3181+
3182+ step (Tuple' Nothing r1) arr =
3183+ runInner (byteLength arr) r1 arr
3184+
3185+ -- XXX Buffer arrays as a list to avoid copy and reallocations
3186+ step (Tuple' (Just buf) r1) arr = do
3187+ let len = byteLength buf + byteLength arr
3188+ buf1 <- if byteCapacity buf < len
3189+ then liftIO $ reallocExplicitAs
3190+ ps (SIZE_OF(a)) (max minBytes len) buf
3191+ else return buf
3192+ buf2 <- spliceUnsafe buf1 arr
3193+ runInner len r1 buf2
3194+
3195+ -- XXX Several folds do extract >=> final, therefore, we need to make final
3196+ -- return "m b" rather than using extract post it if we want extract to be
3197+ -- partial.
3198+ --
3199+ -- extract forces the pending buffer to be sent to the fold which is not
3200+ -- what we want.
3201+ extract _ = error "lCompactGE: not designed for scanning"
3202+
3203+ final (Tuple' Nothing r1) = final1 r1
3204+ final (Tuple' (Just buf) r1) = do
3205+ r <- step1 r1 buf
3206+ case r of
3207+ FL.Partial rr -> final1 rr
3208+ FL.Done _ -> return ()
3209+
3210+-- | Like 'compactGE' but for transforming folds instead of stream.
3211+--
3212+-- >>> lCompactGE n = Fold.many (MutArray.fCompactGE n)
3213+--
3214+-- Generates unpinned arrays irrespective of the pinning status of input
3215+-- arrays.
3216+{-# INLINE lCompactGE #-}
3217+lCompactGE :: forall m a. (MonadIO m, Unbox a)
3218+ => Int -> Fold m (MutArray a) () -> Fold m (MutArray a) ()
3219+lCompactGE = lCompactGeAs Unpinned
3220+
3221+-- | Pinned version of 'lCompactGE'.
3222+{-# INLINE lPinnedCompactGE #-}
3223+lPinnedCompactGE :: forall m a. (MonadIO m, Unbox a)
3224+ => Int -> Fold m (MutArray a) () -> Fold m (MutArray a) ()
3225+lPinnedCompactGE = lCompactGeAs Pinned
3226+
3227+-- | @compactGE n stream@ coalesces adjacent arrays in the @stream@ until
3228+-- the size becomes greater than or equal to @n@.
3229+--
3230+-- >>> compactGE n = Stream.foldMany (MutArray.fCompactGE n)
3231+--
3232+{-# INLINE compactGE #-}
3233+compactGE ::
3234+ (MonadIO m, Unbox a)
3235+ => Int -> Stream m (MutArray a) -> Stream m (MutArray a)
3236+compactGE n = D.foldMany (fCompactGE n)
3237+
3238+-- | 'compactEQ n' coalesces adajacent arrays in the input stream to
3239+-- arrays of exact size @n@.
3240+--
3241+-- /Unimplemented/
3242+{-# INLINE compactEQ #-}
3243+compactEQ :: -- (MonadIO m, Unbox a) =>
3244+ Int -> Stream m (MutArray a) -> Stream m (MutArray a)
3245+compactEQ _n = undefined -- D.parseManyD (pCompactEQ n)
23923246
23933247-------------------------------------------------------------------------------
2394−-- NFData
3248+-- In-place mutation algorithms
23953249-------------------------------------------------------------------------------
23963250
23973251-- | Strip elements which match with predicate from both ends.
@@ -2420,7 +3274,7 @@ strip eq arr@MutArray{..} = liftIO $ do
24203274 getStart cur = do
24213275 if cur < arrEnd
24223276 then do
2423− r <- peekByteIndex cur arrContents
3277+ r <- peekAt cur arrContents
24243278 if eq r
24253279 then getStart (INDEX_NEXT(cur,a))
24263280 else return cur
@@ -2430,7 +3284,7 @@ strip eq arr@MutArray{..} = liftIO $ do
24303284 if cur > low
24313285 then do
24323286 let prev = INDEX_PREV(cur,a)
2433− r <- peekByteIndex prev arrContents
3287+ r <- peekAt prev arrContents
24343288 if eq r
24353289 then getLast prev low
24363290 else return cur