Allocate proper byte count in InputMemoryBitStream copy constructor, thanks to mwistauder!

Joshua Glazer · Joshua Glazer · commit 11ee617698f2 · 2017-05-20T10:00:28.000-07:00
diff --git a/Chapter 4/RoboCat/Inc/MemoryBitStream.h b/Chapter 4/RoboCat/Inc/MemoryBitStream.h
@@ -97,7 +97,7 @@ class InputMemoryBitStream
 	mIsBufferOwner( true )
 	{
 		//allocate buffer of right size
-		int byteCount = mBitCapacity / 8;
+		int byteCount = ( mBitCapacity + 7 ) / 8;
 		mBuffer = static_cast< char* >( malloc( byteCount ) );
 		//copy
 		memcpy( mBuffer, inOther.mBuffer, byteCount );

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ class InputMemoryBitStream`
`97`	`97`	`mIsBufferOwner( true )`
`98`	`98`	`{`
`99`	`99`	`//allocate buffer of right size`
`100`		`- int byteCount = mBitCapacity / 8;`
	`100`	`+ int byteCount = ( mBitCapacity + 7 ) / 8;`
`101`	`101`	`mBuffer = static_cast< char* >( malloc( byteCount ) );`
`102`	`102`	`//copy`
`103`	`103`	`memcpy( mBuffer, inOther.mBuffer, byteCount );`