Renamed BitBufferRead.seek to seek_to_bit to match BitBufferWrite

Also added an optional argument to BitBufferRead constructor to decide
whether or not a copy of the passed buffer should be created

Author: Ayuto

src/core/modules/bitbuffers/bitbuffers.h

@@ -31,6 +31,7 @@
 // Includes.
 //-----------------------------------------------------------------------------
 #include "tier1/bitbuf.h"
+#include "utilities/sp_util.h"
 
 
 //-----------------------------------------------------------------------------
@@ -57,12 +58,23 @@ class BitBufferWriteExt
 class BitBufferReadExt
 {
 public:
-	static boost::shared_ptr<bf_read> __init__(bf_write& buffer)
+	static boost::shared_ptr<bf_read> __init__(bf_write& buffer, bool create_copy)
 	{
 		int size = buffer.GetNumBytesWritten();
-		void* pData = new unsigned char[size];
-		memcpy(pData, buffer.GetData(), size);
-		return boost::shared_ptr<bf_read>(new bf_read(pData, size), &__del__);
+		if (create_copy)
+		{
+			void* pData = new unsigned char[size];
+			memcpy(pData, buffer.GetData(), size);
+			return boost::shared_ptr<bf_read>(
+				new bf_read(pData, size),
+				&__del__);
+		}
+		else
+		{
+			return boost::shared_ptr<bf_read>(
+				new bf_read(buffer.GetData(), size),
+				&NeverDeleteDeleter<bf_read*>);
+		}
 	}
 
 	static void __del__(bf_read* buffer)

src/core/modules/bitbuffers/bitbuffers_wrap.cpp

@@ -228,7 +228,10 @@ void export_bf_read(scope _bitbuffers)
 {
 	class_<bf_read>("BitBufferRead", init<void*, int, optional<int> >())
 		.def("__init__",
-			make_constructor(&BitBufferReadExt::__init__)
+			make_constructor(
+				&BitBufferReadExt::__init__,
+				default_call_policies(),
+				("buffer_write", arg("create_copy")=true))
 		)
 
 		.def("read_one_bit",
@@ -335,7 +338,7 @@ void export_bf_read(scope _bitbuffers)
 			&bf_read::IsOverflowed
 		)
 
-		.def("seek",
+		.def("seek_to_bit",
 			&bf_read::Seek
 		)