on master help sign in
13 files changed+454−101
since splitsnapshots
unifiedsplit
.gitignore+1 BitsQC.hs+69−36 Data/Binary/Bits.hs+3−3 Data/Binary/Bits/Alignment.hs+22 Data/Binary/Bits/BitOrder.hs+2−2 Data/Binary/Bits/Get.hs+54−30 Data/Binary/Bits/Internal.hs+27−6 Data/Binary/Bits/Put.hs+111−22 binary-bits.cabal+4−2 flake.lock+59 flake.nix+24 stack.yaml+66 stack.yaml.lock+12
.gitignore+1
@@ -1,3 +1,4 @@
11.cabal-sandbox
22cabal.sandbox.config
33dist/
4+.stack-work
BitsQC.hs+69−36
@@ -9,6 +9,7 @@ import Data.Binary.Put ( runPut )
99import Data.Binary.Bits
1010import Data.Binary.Bits.Get
1111import Data.Binary.Bits.Put
12+import Data.Binary.Bits.BitOrder
1213
1314import qualified Data.ByteString as B
1415import qualified Data.ByteString.Lazy as L
@@ -17,7 +18,8 @@ import Control.Applicative
1718import Data.Bits
1819import Data.Word
1920import Foreign.Storable
20−import System.Random
21+import Data.Traversable (traverse)
22+import Data.Foldable (traverse_)
2123
2224import Test.Framework.Providers.QuickCheck2 ( testProperty )
2325import Test.Framework.Runners.Console ( defaultMain )
@@ -36,7 +38,10 @@ tests =
3638 [ testProperty "prop_composite_case" prop_composite_case ]
3739
3840 , testGroup "getByteString"
39− [ testProperty "prop_getByteString_negative" prop_getByteString_negative ]
41+ [ testProperty "prop_getByteString_negative" prop_getByteString_negative
42+ , testProperty "prop_putByteString_getByteString" (prop_putByteString_getByteString :: BitOrder -> B.ByteString -> Property)
43+ , testProperty "prop_putByteString_getByteString_many" (prop_putByteString_getByteString_many :: BitOrder -> [B.ByteString] -> Property)
44+ ]
4045
4146 , testGroup "getLazyByteString"
4247 [ testProperty "getLazyByteString == getByteString"
@@ -84,10 +89,10 @@ tests =
8489 ]
8590
8691 , testGroup "prop_put_with_bitreq"
87− [ testProperty "Word8" (prop_putget_with_bitreq :: W Word8 -> Property)
88− , testProperty "Word16" (prop_putget_with_bitreq :: W Word16 -> Property)
89− , testProperty "Word32" (prop_putget_with_bitreq :: W Word32 -> Property)
90− , testProperty "Word64" (prop_putget_with_bitreq :: W Word64 -> Property)
92+ [ testProperty "Word8" (prop_putget_with_bitreq :: BitOrder -> W Word8 -> Property)
93+ , testProperty "Word16" (prop_putget_with_bitreq :: BitOrder -> W Word16 -> Property)
94+ , testProperty "Word32" (prop_putget_with_bitreq :: BitOrder -> W Word32 -> Property)
95+ , testProperty "Word64" (prop_putget_with_bitreq :: BitOrder -> W Word64 -> Property)
9196 ]
9297
9398 , testGroup "prop_putget_list_simple"
@@ -112,10 +117,10 @@ tests =
112117 , testProperty "Word64" (prop_putget_list_with_bitreq :: W [Word64] -> Property)
113118 ]
114119 , testGroup "prop_bitget_bytestring_interspersed"
115− [ testProperty "Word8" (prop_bitget_bytestring_interspersed :: W Word8 -> [B.ByteString] -> Property)
116− , testProperty "Word16" (prop_bitget_bytestring_interspersed :: W Word16 -> [B.ByteString] -> Property)
117− , testProperty "Word32" (prop_bitget_bytestring_interspersed :: W Word32 -> [B.ByteString] -> Property)
118− , testProperty "Word64" (prop_bitget_bytestring_interspersed :: W Word64 -> [B.ByteString] -> Property)
120+ [ testProperty "Word8" (prop_bitget_bytestring_interspersed :: BitOrder -> W Word8 -> [B.ByteString] -> Property)
121+ , testProperty "Word16" (prop_bitget_bytestring_interspersed :: BitOrder -> W Word16 -> [B.ByteString] -> Property)
122+ , testProperty "Word32" (prop_bitget_bytestring_interspersed :: BitOrder -> W Word32 -> [B.ByteString] -> Property)
123+ , testProperty "Word64" (prop_bitget_bytestring_interspersed :: BitOrder -> W Word64 -> [B.ByteString] -> Property)
119124 ]
120125 , testGroup "Simulate programs"
121126 [ testProperty "primitive" prop_primitive
@@ -158,13 +163,28 @@ prop_getByteString_negative n =
158163 n < 1 ==>
159164 runGet (runBitGet (getByteString n)) L.empty == B.empty
160165
161−prop_putget_with_bitreq :: (BinaryBit a, Num a, Bits a, Ord a) => W a -> Property
162−prop_putget_with_bitreq (W w) = property $
166+prop_putByteString_getByteString :: BitOrder -> B.ByteString -> Property
167+prop_putByteString_getByteString bo bs = property $ bs' == bs
168+ where
169+ n = B.length bs
170+ w = runPut (runBitPut (withBitOrder bo (putByteString bs)))
171+ bs' = runGet (runBitGet (withBitOrder bo (getByteString n))) w
172+
173+prop_putByteString_getByteString_many :: BitOrder -> [B.ByteString] -> Property
174+prop_putByteString_getByteString_many bo bs = property $ bs' == bs
175+ where
176+ n = fmap B.length bs
177+ w = runPut (runBitPut (withBitOrder bo (traverse_ putByteString bs)))
178+ bs' = runGet (runBitGet (withBitOrder bo (traverse getByteString n))) w
179+
180+
181+prop_putget_with_bitreq :: (BinaryBit a, Num a, Bits a, Ord a) => BitOrder -> W a -> Property
182+prop_putget_with_bitreq bo (W w) = property $
163183 -- write all words with as many bits as it's required
164184 let p = putBits (bitreq w) w
165185 g = getBits (bitreq w)
166− lbs = runPut (runBitPut p)
167− w' = runGet (runBitGet g) lbs
186+ lbs = runPut (runBitPut (withBitOrder bo p))
187+ w' = runGet (runBitGet (withBitOrder bo g)) lbs
168188 in w == w'
169189
170190-- | Write a list of items. Each item is written with the maximum amount of
@@ -226,12 +246,12 @@ prop_bitget_with_put_from_binary (W ws) = property $
226246 in ws == ws'
227247
228248-- | Write each 'ByteString' with a variable sized value as a separator.
229−prop_bitget_bytestring_interspersed :: (BinaryBit a, Binary a, Num a, Ord a, Bits a) => W a -> [B.ByteString] -> Property
230−prop_bitget_bytestring_interspersed (W ws) bss = property $
249+prop_bitget_bytestring_interspersed :: (BinaryBit a, Binary a, Num a, Ord a, Bits a) => BitOrder -> W a -> [B.ByteString] -> Property
250+prop_bitget_bytestring_interspersed bo (W ws) bss = property $
231251 let p = mapM_ (\bs -> putBits (bitreq ws) ws >> putByteString bs) bss
232252 g = mapM (\bs -> (,) <$> getBits (bitreq ws) <*> getByteString (B.length bs)) bss
233− lbs = runPut (runBitPut p)
234− r = runGet (runBitGet g) lbs
253+ lbs = runPut (runBitPut (withBitOrder bo p))
254+ r = runGet (runBitGet (withBitOrder bo g)) lbs
235255 in map (ws,) bss == r
236256
237257-- | Test failing.
@@ -241,8 +261,8 @@ prop_fail lbs errMsg0 = forAll (choose (0, 8 * L.length lbs)) $ \len ->
241261 expectedBytesConsumed
242262 | bits == 0 = bytes
243263 | otherwise = bytes + 1
244− p = do getByteString (fromIntegral bytes)
245− getBits (fromIntegral bits) :: BitGet Word8
264+ p = do _ <- getByteString (fromIntegral bytes)
265+ _ <- getBits (fromIntegral bits) :: BitGet Word8
246266 fail errMsg0
247267 r = runGetIncremental (runBitGet p) `pushChunks` lbs
248268 in case r of
@@ -268,7 +288,7 @@ prop_bitreq (W w) = property $
268288prop_composite_case :: Bool -> W Word16 -> Property
269289prop_composite_case b (W w) = w < 0x8000 ==>
270290 let p = do putBool b
271− putWord16be 15 w
291+ putWord16 15 w
272292 g = do v <- getBool
273293 case v of
274294 True -> getWord16 15
@@ -374,11 +394,6 @@ instance (Arbitrary (W a), Arbitrary (W b), Arbitrary (W c)) => Arbitrary (W (a,
374394 arbitrary = ((W .) .) . (,,) <$> arbitraryW <*> arbitraryW <*> arbitraryW
375395 shrink (W (a,b,c)) = ((W .) .) . (,,) <$> shrinkW a <*> shrinkW b <*> shrinkW c
376396
377−integralRandomR :: (Integral a, RandomGen g) => (a,a) -> g -> (a,g)
378−integralRandomR (a,b) g = case randomR (fromIntegral a :: Integer,
379− fromIntegral b :: Integer) g of
380− (x,g) -> (fromIntegral x, g)
381−
382397data Primitive
383398 = Bool Bool
384399 | W8 Int Word8
@@ -387,6 +402,7 @@ data Primitive
387402 | W64 Int Word64
388403 | BS Int B.ByteString
389404 | LBS Int L.ByteString
405+ | Skip Int
390406 | IsEmpty
391407 deriving (Eq, Show)
392408
@@ -405,6 +421,7 @@ instance Arbitrary Primitive where
405421 , gen W16
406422 , gen W32
407423 , gen W64
424+ , Skip <$> choose (0, 3000)
408425 , do n <- choose (0,10)
409426 cs <- vector n
410427 return (BS n (B.pack cs))
@@ -421,10 +438,19 @@ instance Arbitrary Primitive where
421438 W16 _ x -> snk W16 x
422439 W32 _ x -> snk W32 x
423440 W64 _ x -> snk W64 x
441+ Skip x -> Skip <$> shrink x
424442 BS _ bs -> let ws = B.unpack bs in map (\ws' -> BS (length ws') (B.pack ws')) (shrink ws)
425443 LBS _ lbs -> let ws = L.unpack lbs in map (\ws' -> LBS (length ws') (L.pack ws')) (shrink ws)
426444 IsEmpty -> []
427445
446+instance Arbitrary BitOrder where
447+ arbitrary = elements [BB, LB, LL, BL]
448+ shrink LL = [BB,LB,BL]
449+ shrink BL = [BB,LB]
450+ shrink LB = [BB]
451+ shrink BB = []
452+
453+
428454prop_primitive :: Primitive -> Property
429455prop_primitive prim = property $
430456 let p = putPrimitive prim
@@ -446,9 +472,10 @@ putPrimitive p =
446472 case p of
447473 Bool b -> putBool b
448474 W8 n x -> putWord8 n x
449− W16 n x -> putWord16be n x
450− W32 n x -> putWord32be n x
451− W64 n x -> putWord64be n x
475+ W16 n x -> putWord16 n x
476+ W32 n x -> putWord32 n x
477+ W64 n x -> putWord64 n x
478+ Skip n -> skipBits n
452479 BS _ bs -> putByteString bs
453480 LBS _ lbs -> mapM_ putByteString (L.toChunks lbs)
454481 IsEmpty -> return ()
@@ -461,10 +488,22 @@ getPrimitive p =
461488 W16 n _ -> W16 n <$> getWord16 n
462489 W32 n _ -> W32 n <$> getWord32 n
463490 W64 n _ -> W64 n <$> getWord64 n
491+ Skip n -> skipBits n >> return (Skip n)
464492 BS n _ -> BS n <$> getByteString n
465493 LBS n _ -> LBS n <$> getLazyByteString n
466494 IsEmpty -> isEmpty >> return IsEmpty
467495
496+getPrimitiveSize :: Primitive -> Int
497+getPrimitiveSize p = case p of
498+ Bool _ -> 1
499+ W8 n _ -> n
500+ W16 n _ -> n
501+ W32 n _ -> n
502+ W64 n _ -> n
503+ Skip n -> n
504+ BS n _ -> n*8
505+ LBS n _ -> n*8
506+ IsEmpty -> 0
468507
469508verifyProgram :: Int -> Program -> BitGet Bool
470509verifyProgram totalLength ps0 = go 0 ps0
@@ -472,13 +511,6 @@ verifyProgram totalLength ps0 = go 0 ps0
472511 go _ [] = return True
473512 go pos (p:ps) =
474513 case p of
475− Bool x -> check x getBool >> go (pos+1) ps
476− W8 n x -> check x (getWord8 n) >> go (pos+n) ps
477− W16 n x -> check x (getWord16 n) >> go (pos+n) ps
478− W32 n x -> check x (getWord32 n) >> go (pos+n) ps
479− W64 n x -> check x (getWord64 n) >> go (pos+n) ps
480− BS n x -> check x (getByteString n) >> go (pos+(8*n)) ps
481− LBS n x -> check x (getLazyByteString n) >> go (pos+(8*n)) ps
482514 IsEmpty -> do
483515 let expected = pos == totalLength
484516 actual <- isEmpty
@@ -486,6 +518,7 @@ verifyProgram totalLength ps0 = go 0 ps0
486518 then go pos ps
487519 else error $ "isEmpty returned wrong value, expected "
488520 ++ show expected ++ " but got " ++ show actual
521+ _ -> check p (getPrimitive p) >> go (pos + getPrimitiveSize p) ps
489522 check x g = do
490523 y <- g
491524 if x == y
Data/Binary/Bits.hs+3−3
@@ -33,13 +33,13 @@ instance BinaryBit Word8 where
3333 getBits = getWord8
3434
3535instance BinaryBit Word16 where
36− putBits = putWord16be
36+ putBits = putWord16
3737 getBits = getWord16
3838
3939instance BinaryBit Word32 where
40− putBits = putWord32be
40+ putBits = putWord32
4141 getBits = getWord32
4242
4343instance BinaryBit Word64 where
44− putBits = putWord64be
44+ putBits = putWord64
4545 getBits = getWord64
Data/Binary/Bits/Alignment.hsnew file+22
@@ -0,0 +1,22 @@
1+-----------------------------------------------------------------------------
2+-- |
3+-- Module : Data.Binary.Bits.Get
4+-- Copyright : (c) Lennart Kolmodin 2010-2011
5+-- (c) Sylvain Henry 2015
6+-- License : BSD3-style (see LICENSE)
7+--
8+-- Maintainer : kolmodin@gmail.com
9+-- Stability : experimental
10+-- Portability : portable (should run where the package binary runs)
11+
12+module Data.Binary.Bits.Alignment
13+ ( Alignable(..)
14+ )
15+where
16+
17+class Monad m => Alignable m where
18+ -- | Skip the given number of bits
19+ skipBits :: Int -> m ()
20+
21+ -- | Skip bits if necessary to align to the next byte
22+ alignByte :: m ()
Data/Binary/Bits/BitOrder.hs+2−2
@@ -19,9 +19,9 @@ where
1919--
2020-- E.g. two words of 5 bits: ABCDE, VWXYZ
2121-- - BB: ABCDEVWX YZxxxxxx
22−-- - LB: XYZABCDE xxxxxxVW
22+-- - LL: XYZABCDE xxxxxxVW
2323-- - BL: EDCBAZYX WVxxxxxx
24−-- - LL: XWVEDCBA xxxxxxZY
24+-- - LB: XWVEDCBA xxxxxxZY
2525data BitOrder
2626 = BB -- ^ Big-endian bytes and bits
2727 | LB -- ^ Little-endian bytes, big-endian bits
Data/Binary/Bits/Get.hs+54−30
@@ -60,6 +60,7 @@ module Data.Binary.Bits.Get
6060
6161 -- ** Get bytes
6262 , getBool
63+ , getBool#
6364 , getWord8
6465 , getWord16
6566 , getWord32
@@ -98,6 +99,7 @@ import Data.Binary.Get as B ( Get, getLazyByteString, isEmpty )
9899import Data.Binary.Get.Internal as B ( get, put, ensureN )
99100import Data.Binary.Bits.BitOrder
100101import Data.Binary.Bits.Internal
102+import Data.Binary.Bits.Alignment
101103
102104import Data.ByteString as B
103105import qualified Data.ByteString.Lazy as L
@@ -113,6 +115,8 @@ import Control.Applicative
113115import Control.Monad (when,foldM_)
114116
115117import Prelude as P
118+import GHC.Word
119+import GHC.Base (uncheckedShiftLWord8#, wordToWord8#, Int (..), andWord8#, Word8#, (-#), Int#, xorI#, eqWord8#, TYPE)
116120
117121-- $bitget
118122-- Parse bits using a monad.
@@ -178,8 +182,20 @@ readBool :: S -> Bool
178182readBool (S bs o bo) = case bo of
179183 BB -> testBit (unsafeHead bs) (7-o)
180184 BL -> testBit (unsafeHead bs) (7-o)
181− LB -> testBit (unsafeHead bs) o
182185 LL -> testBit (unsafeHead bs) o
186+ LB -> testBit (unsafeHead bs) o
187+
188+-- | Read a single bit
189+readBool# :: S -> Int
190+readBool# (S bs (I# o) bo) = case bo of
191+ BB -> testBit# (7# -# o)
192+ BL -> testBit# (7# -# o)
193+ LL -> testBit# o
194+ LB -> testBit# o
195+ where
196+ !(W8# w) = unsafeHead bs
197+ testBit# n = I# (((w `andWord8#` (wordToWord8# 1## `uncheckedShiftLWord8#` n)) `eqWord8#` wordToWord8# 0##) `xorI#` 1#)
198+
183199
184200-- | Extract a range of bits from (ws :: ByteString)
185201--
@@ -202,17 +218,17 @@ extract bo bs o n
202218
203219 -- shift offset depending on the byte position (0..B.length-1)
204220 off i = case bo of
205− LB -> 8*i - o
206221 LL -> 8*i - o
222+ LB -> 8*i - o
207223 BB -> (B.length bs -1 - i) * 8 - r
208224 BL -> (B.length bs -1 - i) * 8 - r
209225
210226 -- reverse bits if necessary
211227 rev = case bo of
212− LL -> reverseBits n
228+ LB -> reverseBits n
213229 BL -> reverseBits n
214230 BB -> id
215− LB -> id
231+ LL -> id
216232
217233
218234-- | Generic readWord
@@ -233,22 +249,23 @@ readWordChecked m n s
233249--
234250-- Examples:
235251-- BB: xxxABCDE FGHIJKLM NOPxxxxx -> ABCDEFGH IJKLMNOP
236−-- LB: LMNOPxxx DEFGHIJK xxxxxABC -> ABCDEFGH IJKLMNOP
252+-- LL: LMNOPxxx DEFGHIJK xxxxxABC -> ABCDEFGH IJKLMNOP
237253-- BL: xxxPONML KJIHGFED CBAxxxxx -> ABCDEFGH IJKLMNOP
238−-- LL: EDCBAxxx MLKJIHGF xxxxxPON -> ABCDEFGH IJKLMNOP
254+-- LB: EDCBAxxx MLKJIHGF xxxxxPON -> ABCDEFGH IJKLMNOP
239255readByteString :: Int -> S -> ByteString
240256readByteString n (S bs o bo) =
241257 let
242− bs' = unsafeTake (n+1) bs
243− rev = B.map (reverseBits 8)
258+ bs' = unsafeTake (n+1) bs
259+ bs'' = unsafeTake n bs
260+ rev = B.map (reverseBits 8)
244261 in case (o,bo) of
245− (0,BB) -> unsafeTake n bs
246− (0,LB) -> B.reverse (unsafeTake n bs)
247− (0,BL) -> rev (unsafeTake n bs)
248− (0,LL) -> rev (B.reverse (unsafeTake n bs))
249− (_,BL) -> rev (readByteString n (S (B.reverse bs') o LB))
250− (_,LL) -> rev (readByteString n (S (B.reverse bs') o BB))
251− (_,LB) -> readByteString n (S (B.reverse bs') o BB)
262+ (0,BB) -> bs''
263+ (0,LL) -> B.reverse bs''
264+ (0,LB) -> rev bs''
265+ (0,BL) -> rev . B.reverse $ bs''
266+ (_,LL) -> readByteString n (S (B.reverse bs') (8-o) BB)
267+ (_,BL) -> rev . B.reverse $ readByteString n (S bs' o BB)
268+ (_,LB) -> rev . B.reverse $ readByteString n (S bs' o LL)
252269 (_,BB) -> unsafePerformIO $ do
253270 let len = n+1
254271 ptr <- mallocBytes len
@@ -270,7 +287,6 @@ newtype BitGet a = B { runState :: S -> Get (S,a) }
270287
271288instance Monad BitGet where
272289 return x = B $ \s -> return (s,x)
273− fail str = B $ \s -> putBackState s >> fail str
274290 (B f) >>= g = B $ \s -> do (s',a) <- f s
275291 runState (g a) s'
276292
@@ -290,6 +306,20 @@ instance BitOrderable BitGet where
290306 (S _ _ bo) <- getState
291307 return bo
292308
309+instance Alignable BitGet where
310+ -- | Skip the given number of bits
311+ skipBits n = do
312+ ensureBits n
313+ withState (incS n)
314+
315+ -- | Skip bits if necessary to align to the next byte
316+ alignByte = do
317+ (S _ o _) <- getState
318+ when (o /= 0) $
319+ skipBits (8-o)
320+
321+
322+
293323-- | Run a 'BitGet' within the Binary packages 'Get' monad. If a byte has
294324-- been partially consumed it will be discarded once 'runBitGet' is finished.
295325runBitGet :: BitGet a -> Get a
@@ -337,20 +367,6 @@ ensureBits n = do
337367 put B.empty
338368 return (S (bs`append`bs') o bo, ())
339369
340−-- | Skip the given number of bits
341−skipBits :: Int -> BitGet ()
342−skipBits n = do
343− ensureBits n
344− withState (incS n)
345−
346−-- | Skip bits if necessary to align to the next byte
347−alignByte :: BitGet ()
348−alignByte = do
349− (S _ o _) <- getState
350− when (o /= 0) $
351− skipBits (8-o)
352−
353−
354370-- | Test whether all input has been consumed, i.e. there are no remaining
355371-- undecoded bytes.
356372isEmpty :: BitGet Bool
@@ -362,6 +378,10 @@ isEmpty = B $ \ (S bs o bo) -> if B.null bs
362378getBool :: BitGet Bool
363379getBool = block bool
364380
381+-- | Get 1 bit as a 'Bool'.
382+getBool# :: BitGet Int
383+getBool# = block bool#
384+
365385-- | Get @n@ bits as a 'Word8'. @n@ must be within @[0..8]@.
366386getWord8 :: Int -> BitGet Word8
367387getWord8 n = block (word8 n)
@@ -437,6 +457,10 @@ block (Block i p) = do
437457bool :: Block Bool
438458bool = Block 1 readBool
439459
460+-- | Read a 1 bit 'Bool'.
461+bool# :: Block Int
462+bool# = Block 1 readBool#
463+
440464-- | Read @n@ bits as a 'Word8'. @n@ must be within @[0..8]@.
441465word8 :: Int -> Block Word8
442466word8 n = Block n (readWordChecked 8 n)
Data/Binary/Bits/Internal.hs+27−6
@@ -12,10 +12,13 @@
1212
1313module Data.Binary.Bits.Internal
1414 ( make_mask
15+ , make_mask#
1516 , mask
17+ , mask#
1618 , bit_offset
1719 , byte_offset
1820 , reverseBits
21+ , reverseBits#
1922 , FastBits(..)
2023 )
2124where
@@ -38,11 +41,18 @@ make_mask n = (1 `shiftL` fromIntegral n) - 1
3841{-# SPECIALIZE make_mask :: Int -> Word32 #-}
3942{-# SPECIALIZE make_mask :: Int -> Word64 #-}
4043
44+make_mask# :: Int# -> Word#
45+make_mask# n = (1## `shiftL#` n) `minusWord#` 1##
46+
4147-- | Keep only the n least-significant bits of the given value
4248mask :: (Bits a, Num a) => Int -> a -> a
4349mask n v = v .&. make_mask n
4450{-# INLINE mask #-}
4551
52+-- | Keep only the n least-significant bits of the given value
53+mask# :: Int# -> Word# -> Word#
54+mask# n v = v `and#` make_mask# n
55+
4656-- | Compute bit offset (equivalent to x `mod` 8 but faster)
4757bit_offset :: Int -> Int
4858bit_offset n = make_mask 3 .&. n
@@ -65,6 +75,17 @@ reverseBits n value = rec value n 0
6575 rec 0 i r = r `fastShiftL` i
6676 rec v i r = rec (v `fastShiftR` 1) (i-1) ((r `fastShiftL` 1) .|. (v .&. 0x1))
6777
78+reverseBits# :: Int# -> Word# -> Word#
79+reverseBits# n value = rec value n 0##
80+ where
81+ -- rec v i r, where
82+ -- v is orginal value shifted
83+ -- i is the remaining number of bits
84+ -- r is current value
85+ rec 0## 0# r = r
86+ rec 0## i r = r `shiftL#` i
87+ rec v i r = rec (v `shiftRL#` 1#) (i -# 1#) ((r `shiftL#` 1#) `or#` (v `and#` 0x1##))
88+
6889
6990---------------------------------------------------------------------
7091-- Unchecked shifts, from the "binary" package
@@ -116,13 +137,13 @@ shiftr_w32 :: Word32 -> Int -> Word32
116137shiftr_w64 :: Word64 -> Int -> Word64
117138
118139#if defined(__GLASGOW_HASKELL__) && !defined(__HADDOCK__)
119−shiftl_w8 (W8# w) (I# i) = W8# (w `uncheckedShiftL#` i)
120−shiftl_w16 (W16# w) (I# i) = W16# (w `uncheckedShiftL#` i)
121−shiftl_w32 (W32# w) (I# i) = W32# (w `uncheckedShiftL#` i)
140+shiftl_w8 w i = w `unsafeShiftL` i
141+shiftl_w16 w i = w `unsafeShiftL` i
142+shiftl_w32 w i = w `unsafeShiftL` i
122143
123−shiftr_w8 (W8# w) (I# i) = W8# (w `uncheckedShiftRL#` i)
124−shiftr_w16 (W16# w) (I# i) = W16# (w `uncheckedShiftRL#` i)
125−shiftr_w32 (W32# w) (I# i) = W32# (w `uncheckedShiftRL#` i)
144+shiftr_w8 w i = w `unsafeShiftR` i
145+shiftr_w16 w i = w `unsafeShiftR` i
146+shiftr_w32 w i = w `unsafeShiftR` i
126147
127148
128149#if WORD_SIZE_IN_BITS < 64
Data/Binary/Bits/Put.hs+111−22
@@ -1,3 +1,5 @@
1+{-# LANGUAGE MagicHash #-}
2+
13-----------------------------------------------------------------------------
24-- |
35-- Module : Data.Binary.Bits.Put
@@ -20,9 +22,14 @@ module Data.Binary.Bits.Put
2022 -- * Data types
2123 -- ** Bool
2224 , putBool
25+ , putBool#
2326
2427 -- ** Words
28+ , putWord#
2529 , putWord8
30+ , putWord16
31+ , putWord32
32+ , putWord64
2633 , putWord16be
2734 , putWord32be
2835 , putWord64be
@@ -38,14 +45,16 @@ import qualified Data.Binary.Put as Put
3845import Data.Binary.Put ( Put )
3946import Data.Binary.Bits.Internal
4047import Data.Binary.Bits.BitOrder
48+import Data.Binary.Bits.Alignment
4149
4250import Data.ByteString as BS
4351import Data.ByteString.Unsafe as BS
4452
45−import Control.Applicative
53+import Control.Monad (when)
4654import Data.Bits
47−import Data.Monoid
4855import Data.Word
56+import GHC.Base (Word#, shiftRL#, Int (..), Int#, (-#), (<#), isTrue#, (+#), shiftL#, wordToWord8#, orWord8#)
57+import GHC.Word (Word8(..))
4958
5059data BitPut a = BitPut { run :: (S -> PairS a) }
5160
@@ -57,11 +66,18 @@ data S = S !Builder !Word8 !Int !BitOrder
5766putBool :: Bool -> BitPut ()
5867putBool b = putWord8 1 (if b then 0xff else 0x00)
5968
69+-- | Put a 1 bit 'Bool'.
70+putBool# :: Word# -> BitPut ()
71+putBool# b = putWord# 1# b
72+
6073
6174-- | Generic putWord
6275putWord :: (Num a, FastBits a, Integral a) => Int -> a -> BitPut ()
6376putWord n w = BitPut $ \s -> PairS () (putWordS n w s)
6477
78+putWord# :: Int# -> Word# -> BitPut ()
79+putWord# n w = BitPut $ \s -> PairS () (putWordS# n w s)
80+
6581putWordS :: (Num a, FastBits a, Integral a) => Int -> a -> S -> S
6682putWordS n w s@(S builder b o bo) = s'
6783 where
@@ -79,9 +95,9 @@ putWordS n w s@(S builder b o bo) = s'
7995 -- Word containing the remaining (n-cn) bits to store in its LSB
8096 w' = case bo of
8197 BB -> w
82− BL -> w
83− LB -> w `fastShiftR` cn
98+ BL -> w `fastShiftR` cn
8499 LL -> w `fastShiftR` cn
100+ LB -> w
85101
86102 -- Select bits to store in the current byte.
87103 -- Put them in the correct order and return them in the least-significant
@@ -89,17 +105,64 @@ putWordS n w s@(S builder b o bo) = s'
89105 selectBits :: (Num a, FastBits a, Integral a) => a -> Word8
90106 selectBits x = fromIntegral $ case bo of
91107 BB -> mask cn $ x `fastShiftR` (n-cn)
92− BL -> reverseBits cn $ mask cn $ x `fastShiftR` (n-cn)
93− LB -> mask cn x
94− LL -> reverseBits cn $ mask cn x
108+ LB -> reverseBits cn $ mask cn $ x `fastShiftR` (n-cn)
109+ LL -> mask cn x
110+ BL -> reverseBits cn $ mask cn x
95111
96112 -- shift left at the correct position
97113 shl :: Word8 -> Word8
98114 shl x = case bo of
99115 BB -> x `fastShiftL` (8-o-cn)
100116 BL -> x `fastShiftL` (8-o-cn)
101− LB -> x `fastShiftL` o
102117 LL -> x `fastShiftL` o
118+ LB -> x `fastShiftL` o
119+
120+ flush s2@(S b2 w2 o2 bo2)
121+ | o2 == 8 = S (b2 `mappend` B.singleton w2) 0 0 bo2
122+ | otherwise = s2
123+
124+putWordS# :: Int# -> Word# -> S -> S
125+putWordS# n w s@(S builder (W8# b) (I# o) bo) = s'
126+ where
127+ min# :: Int# -> Int# -> Int#
128+ min# x y | isTrue# (x <# y) = x
129+ min# _ y = y
130+
131+ -- number of bits that will be stored in the current byte
132+ cn = min# (8# -# o) n
133+
134+ -- new state
135+ s' = case n of
136+ 0# -> s
137+ _ -> putWordS# (n -# cn) w' (flush (S builder (W8# b') (I# (o +# cn)) bo))
138+
139+ -- new current byte
140+ b' = wordToWord8# (shl (selectBits w)) `orWord8#` b
141+
142+ -- Word containing the remaining (n-cn) bits to store in its LSB
143+ w' = case bo of
144+ BB -> w
145+ BL -> w `shiftRL#` cn
146+ LL -> w `shiftRL#` cn
147+ LB -> w
148+
149+ -- Select bits to store in the current byte.
150+ -- Put them in the correct order and return them in the least-significant
151+ -- bits of the returned value
152+ selectBits :: Word# -> Word#
153+ selectBits x = case bo of
154+ BB -> mask# cn (x `shiftRL#` (n -# cn))
155+ LB -> reverseBits# cn (mask# cn (x `shiftRL#` (n -# cn)))
156+ LL -> mask# cn x
157+ BL -> reverseBits# cn (mask# cn x)
158+
159+ -- shift left at the correct position
160+ shl :: Word# -> Word#
161+ shl x = case bo of
162+ BB -> x `shiftL#` (8# -# o -# cn)
163+ BL -> x `shiftL#` (8# -# o -# cn)
164+ LL -> x `shiftL#` o
165+ LB -> x `shiftL#` o
103166
104167 flush s2@(S b2 w2 o2 bo2)
105168 | o2 == 8 = S (b2 `mappend` B.singleton w2) 0 0 bo2
@@ -111,42 +174,53 @@ putWord8 :: Int -> Word8 -> BitPut ()
111174putWord8 = putWord
112175
113176-- | Put the @n@ lower bits of a 'Word16'.
177+putWord16 :: Int -> Word16 -> BitPut ()
178+putWord16 = putWord
179+
114180putWord16be :: Int -> Word16 -> BitPut ()
115181putWord16be = putWord
182+{-# DEPRECATED putWord16be "Use 'putWord16' instead" #-}
116183
117184-- | Put the @n@ lower bits of a 'Word32'.
185+putWord32 :: Int -> Word32 -> BitPut ()
186+putWord32 = putWord
187+
118188putWord32be :: Int -> Word32 -> BitPut ()
119189putWord32be = putWord
190+{-# DEPRECATED putWord32be "Use 'putWord32' instead" #-}
120191
121192-- | Put the @n@ lower bits of a 'Word64'.
193+putWord64 :: Int -> Word64 -> BitPut ()
194+putWord64 = putWord
195+
122196putWord64be :: Int -> Word64 -> BitPut ()
123197putWord64be = putWord
198+{-# DEPRECATED putWord64be "Use 'putWord64' instead" #-}
124199
125200-- | Put a 'ByteString'.
126201--
127202-- Examples: 3 bits are already written in the current byte
128203-- BB: ABCDEFGH IJKLMNOP -> xxxABCDE FGHIJKLM NOPxxxxx
129−-- LB: ABCDEFGH IJKLMNOP -> LMNOPxxx DEFGHIJK xxxxxABC
204+-- LL: ABCDEFGH IJKLMNOP -> LMNOPxxx DEFGHIJK xxxxxABC
130205-- BL: ABCDEFGH IJKLMNOP -> xxxPONML KJIHGFED CBAxxxxx
131−-- LL: ABCDEFGH IJKLMNOP -> EDCBAxxx MLKJIHGF xxxxxPON
206+-- LB: ABCDEFGH IJKLMNOP -> EDCBAxxx MLKJIHGF xxxxxPON
132207putByteString :: ByteString -> BitPut ()
133208putByteString bs = BitPut $ \s -> PairS () (putByteStringS bs s)
134209
135210putByteStringS :: ByteString -> S -> S
136−putByteStringS bs (S builder b 0 BB) = S (builder `mappend` B.fromByteString bs) b 0 BB
137−putByteStringS bs s@(S _ _ _ BB)
211+putByteStringS bs s
138212 | BS.null bs = s
139− | otherwise = putByteStringS (BS.unsafeTail bs) (putWordS 8 (BS.unsafeHead bs) s)
140−putByteStringS bs (S builder b o bo) = putByteStringS bs' (S builder b o BB)
213+ | otherwise = case s of
214+ (S builder b 0 BB) -> S (builder `mappend` B.fromByteString bs) b 0 BB
215+ (S builder b 0 LL) -> S (builder `mappend` B.fromByteString (BS.reverse bs)) b 0 LL
216+ (S builder b 0 LB) -> S (builder `mappend` B.fromByteString (rev bs)) b 0 LB
217+ (S builder b 0 BL) -> S (builder `mappend` B.fromByteString (rev (BS.reverse bs))) b 0 BL
218+ (S _ _ _ BB) -> putByteStringS (BS.unsafeTail bs) (putWordS 8 (BS.unsafeHead bs) s)
219+ (S _ _ _ LL) -> putByteStringS (BS.unsafeInit bs) (putWordS 8 (BS.unsafeLast bs) s)
220+ (S _ _ _ BL) -> putByteStringS (BS.unsafeInit bs) (putWordS 8 (BS.unsafeLast bs) s)
221+ (S _ _ _ LB) -> putByteStringS (BS.unsafeTail bs) (putWordS 8 (BS.unsafeHead bs) s)
141222 where
142− rev = BS.map (reverseBits 8)
143−
144− bs' = case bo of
145− BL -> BS.reverse (rev bs)
146− LB -> BS.reverse bs
147− LL -> rev bs
148− BB -> bs -- should not occur, already matched but the compiler doesn't detect it
149−
223+ rev = BS.map (reverseBits 8)
150224
151225
152226-- | Run a 'Put' inside 'BitPut'. Any partially written bytes will be flushed
@@ -164,6 +238,9 @@ flushIncomplete s@(S b w o bo)
164238 | o == 0 = s
165239 | otherwise = (S (b `mappend` B.singleton w) 0 0 bo)
166240
241+getOffset :: BitPut Int
242+getOffset = BitPut $ \s@(S _ _ o _) -> PairS o s
243+
167244-- | Run the 'BitPut' monad inside 'Put'.
168245runBitPut :: BitPut () -> Put.Put
169246runBitPut m = Put.putBuilder b
@@ -194,3 +271,15 @@ instance BitOrderable BitPut where
194271 setBitOrder bo = BitPut $ \(S bu b o _) -> PairS () (S bu b o bo)
195272
196273 getBitOrder = BitPut $ \s@(S _ _ _ bo) -> PairS bo s
274+
275+instance Alignable BitPut where
276+ -- | Skip the given number of bits
277+ skipBits n
278+ | n <= 64 = putWord64 n 0
279+ | otherwise = putWord64 64 0 >> skipBits (n-64)
280+
281+ -- | Skip bits if necessary to align to the next byte
282+ alignByte = do
283+ o <- getOffset
284+ when (o /= 0) $
285+ skipBits (8-o)
binary-bits.cabal+4−2
@@ -1,6 +1,6 @@
11-- http://www.haskell.org/cabal/release/cabal-latest/doc/users-guide/
22name: binary-bits
3−version: 0.5.1
3+version: 0.6
44synopsis: Bit parsing/writing on top of binary.
55description: Bit parsing/writing on top of binary. Provides functions to
66 read and write bits to and from 8\/16\/32\/64 words.
@@ -25,7 +25,8 @@ library
2525 exposed-modules: Data.Binary.Bits ,
2626 Data.Binary.Bits.Put ,
2727 Data.Binary.Bits.Get ,
28− Data.Binary.Bits.BitOrder
28+ Data.Binary.Bits.BitOrder ,
29+ Data.Binary.Bits.Alignment
2930 other-modules: Data.Binary.Bits.Internal
3031
3132 default-language: Haskell98
@@ -36,6 +37,7 @@ test-suite qc
3637 type: exitcode-stdio-1.0
3738 main-is: BitsQC.hs
3839 default-language: Haskell98
40+ --ghc-options: -O2 -Wall
3941
4042 build-depends: base==4.*, binary >= 0.6.0.0, bytestring,
4143 QuickCheck>=2, random,
flake.locknew file+59
@@ -0,0 +1,59 @@
1+{
2+ "nodes": {
3+ "flake-utils": {
4+ "inputs": {
5+ "systems": "systems"
6+ },
7+ "locked": {
8+ "lastModified": 1731533236,
9+ "narHash": "sha256-l0KFg5HjrsfsO/JpG+r7fRrqm12kzFHyUHqHCVpMMbI=",
10+ "owner": "numtide",
11+ "repo": "flake-utils",
12+ "rev": "11707dc2f618dd54ca8739b309ec4fc024de578b",
13+ "type": "github"
14+ },
15+ "original": {
16+ "id": "flake-utils",
17+ "type": "indirect"
18+ }
19+ },
20+ "nixpkgs": {
21+ "locked": {
22+ "lastModified": 1737469691,
23+ "narHash": "sha256-nmKOgAU48S41dTPIXAq0AHZSehWUn6ZPrUKijHAMmIk=",
24+ "owner": "NixOS",
25+ "repo": "nixpkgs",
26+ "rev": "9e4d5190a9482a1fb9d18adf0bdb83c6e506eaab",
27+ "type": "github"
28+ },
29+ "original": {
30+ "id": "nixpkgs",
31+ "ref": "nixos-unstable",
32+ "type": "indirect"
33+ }
34+ },
35+ "root": {
36+ "inputs": {
37+ "flake-utils": "flake-utils",
38+ "nixpkgs": "nixpkgs"
39+ }
40+ },
41+ "systems": {
42+ "locked": {
43+ "lastModified": 1681028828,
44+ "narHash": "sha256-Vy1rq5AaRuLzOxct8nz4T6wlgyUR7zLU309k9mBC768=",
45+ "owner": "nix-systems",
46+ "repo": "default",
47+ "rev": "da67096a3b9bf56a91d16901293e51ba5b49a27e",
48+ "type": "github"
49+ },
50+ "original": {
51+ "owner": "nix-systems",
52+ "repo": "default",
53+ "type": "github"
54+ }
55+ }
56+ },
57+ "root": "root",
58+ "version": 7
59+}
flake.nixnew file+24
@@ -0,0 +1,24 @@
1+{
2+ inputs = {
3+ nixpkgs.url = "nixpkgs/nixos-unstable";
4+ flake-utils.url = "flake-utils";
5+ };
6+
7+ outputs = { self, flake-utils, nixpkgs }:
8+ let
9+ supportedSystems =
10+ [ "x86_64-linux" "x86_64-darwin" "aarch64-darwin" "aarch64-linux" ];
11+ compilerVersion = "966";
12+ in flake-utils.lib.eachSystem supportedSystems (system:
13+ let pkgs = import nixpkgs { inherit system; };
14+ in {
15+ devShells.default = pkgs.mkShell {
16+ buildInputs = with pkgs; [
17+ haskell.compiler."ghc${compilerVersion}"
18+ (pkgs.haskell-language-server.override {
19+ supportedGhcVersions = [ compilerVersion ];
20+ })
21+ ];
22+ };
23+ });
24+}
stack.yamlnew file+66
@@ -0,0 +1,66 @@
1+# This file was automatically generated by 'stack init'
2+#
3+# Some commonly used options have been documented as comments in this file.
4+# For advanced use and comprehensive documentation of the format, please see:
5+# https://docs.haskellstack.org/en/stable/yaml_configuration/
6+
7+# Resolver to choose a 'specific' stackage snapshot or a compiler version.
8+# A snapshot resolver dictates the compiler version and the set of packages
9+# to be used for project dependencies. For example:
10+#
11+# resolver: lts-3.5
12+# resolver: nightly-2015-09-21
13+# resolver: ghc-7.10.2
14+#
15+# The location of a snapshot can be provided as a file or url. Stack assumes
16+# a snapshot provided as a file might change, whereas a url resource does not.
17+#
18+# resolver: ./custom-snapshot.yaml
19+# resolver: https://example.com/snapshots/2018-01-01.yaml
20+resolver: lts-22.43
21+
22+# User packages to be built.
23+# Various formats can be used as shown in the example below.
24+#
25+# packages:
26+# - some-directory
27+# - https://example.com/foo/bar/baz-0.0.2.tar.gz
28+# subdirs:
29+# - auto-update
30+# - wai
31+packages:
32+- .
33+# Dependency packages to be pulled from upstream that are not in the resolver.
34+# These entries can reference officially published versions as well as
35+# forks / in-progress versions pinned to a git hash. For example:
36+#
37+# extra-deps:
38+# - acme-missiles-0.3
39+# - git: https://github.com/commercialhaskell/stack.git
40+# commit: e7b331f14bcffb8367cd58fbfc8b40ec7642100a
41+#
42+# extra-deps: []
43+
44+# Override default flag values for local packages and extra-deps
45+# flags: {}
46+
47+# Extra package databases containing global packages
48+# extra-package-dbs: []
49+
50+# Control whether we use the GHC we find on the path
51+# system-ghc: true
52+#
53+# Require a specific version of stack, using version ranges
54+# require-stack-version: -any # Default
55+# require-stack-version: ">=2.9"
56+#
57+# Override the architecture used by stack, especially useful on Windows
58+# arch: i386
59+# arch: x86_64
60+#
61+# Extra directories used by stack for building
62+# extra-include-dirs: [/path/to/dir]
63+# extra-lib-dirs: [/path/to/dir]
64+#
65+# Allow a newer minor version of GHC than the snapshot specifies
66+# compiler-check: newer-minor
stack.yaml.locknew file+12
@@ -0,0 +1,12 @@
1+# This file was autogenerated by Stack.
2+# You should not edit this file by hand.
3+# For more information, please see the documentation at:
4+# https://docs.haskellstack.org/en/stable/lock_files
5+
6+packages: []
7+snapshots:
8+- completed:
9+ sha256: 08bd13ce621b41a8f5e51456b38d5b46d7783ce114a50ab604d6bbab0d002146
10+ size: 720271
11+ url: https://raw.githubusercontent.com/commercialhaskell/stackage-snapshots/master/lts/22/43.yaml
12+ original: lts-22.43