muhammara 2.0.0 → 2.3.0

package/src/deps/PDFWriter/PDFObject.cpp

@@ -19,6 +19,7 @@
    
 */
 #include "PDFObject.h"
+#include "IDeletable.h"
 
 const char* PDFObject::scPDFObjectTypeLabel(int index) 
 {
@@ -53,9 +54,9 @@ PDFObject::PDFObject(int inType)
 
 PDFObject::~PDFObject(void)
 {
-	StringToVoidP::iterator it = mMetadata.begin();
+	StringToIDeletable::iterator it = mMetadata.begin();
 	for (; it != mMetadata.end(); ++it) {
-		delete it->second;
+		it->second->DeleteMe();
 	}
 	mMetadata.clear();
 }
@@ -65,15 +66,15 @@ PDFObject::EPDFObjectType PDFObject::GetType()
 	return mType;
 }
 
-void PDFObject::SetMetadata(const std::string& inKey, void* inValue) {
+void PDFObject::SetMetadata(const std::string& inKey, IDeletable* inValue) {
 	// delete old metadata
 	DeleteMetadata(inKey);
 
-	mMetadata.insert(StringToVoidP::value_type(inKey, inValue));
+	mMetadata.insert(StringToIDeletable::value_type(inKey, inValue));
 }
 
-void* PDFObject::GetMetadata(const std::string& inKey) {
-	StringToVoidP::iterator it = mMetadata.find(inKey);
+IDeletable* PDFObject::GetMetadata(const std::string& inKey) {
+	StringToIDeletable::iterator it = mMetadata.find(inKey);
 	
 	if (it == mMetadata.end()) 
 		return NULL;
@@ -81,19 +82,20 @@ void* PDFObject::GetMetadata(const std::string& inKey) {
 		return it->second;
 }
 
-void* PDFObject::DetachMetadata(const std::string& inKey) {
-	StringToVoidP::iterator it = mMetadata.find(inKey);
+IDeletable* PDFObject::DetachMetadata(const std::string& inKey) {
+	StringToIDeletable::iterator it = mMetadata.find(inKey);
 
 	if (it == mMetadata.end())
 		return NULL;
 	else {
-		void* result = it->second;
+		IDeletable* result = it->second;
 		mMetadata.erase(it);
 		return result;
 	}
 }
 
 void PDFObject::DeleteMetadata(const std::string& inKey) {
-	void* result = DetachMetadata(inKey);
-	delete result;
+	IDeletable* result = DetachMetadata(inKey);
+	if(result)
+		result->DeleteMe();
 }

package/src/deps/PDFWriter/PDFObject.h

@@ -25,7 +25,9 @@
 #include <string>
 #include <map>
 
-typedef std::map<std::string, void*> StringToVoidP;
+class IDeletable;
+
+typedef std::map<std::string, IDeletable*> StringToIDeletable;
 
 class PDFObject : public RefCountObject
 {
@@ -58,14 +60,14 @@ public:
 	/*
 		metadata will automatically be deleted when object is released
 	*/
-	void SetMetadata(const std::string& inKey,void* inValue); // will automatically delete old data in the same key
-	void* GetMetadata(const std::string& inKey);
+	void SetMetadata(const std::string& inKey,IDeletable* inValue); // will automatically delete old data in the same key
+	IDeletable* GetMetadata(const std::string& inKey);
 	// Detach will only remove the pointer from metadata map, Delete will also delete the inValue pointer
-	void* DetachMetadata(const std::string& inKey);
+	IDeletable* DetachMetadata(const std::string& inKey);
 	void DeleteMetadata(const std::string& inKey);
 
 
 private:
 	EPDFObjectType mType;
-	StringToVoidP mMetadata;
+	StringToIDeletable mMetadata;
 };

package/src/deps/PDFWriter/PDFObjectParser.cpp

@@ -706,4 +706,12 @@ void PDFObjectParser::SetDecryptionHelper(DecryptionHelper* inDecryptionHelper)
 void PDFObjectParser::SetParserExtender(IPDFParserExtender* inParserExtender)
 {
 	mParserExtender = inParserExtender;
+}
+
+IByteReader* PDFObjectParser::StartExternalRead() {
+	return mStream;
+}
+
+void PDFObjectParser::EndExternalRead() {
+	ResetReadState();
 }

package/src/deps/PDFWriter/PDFObjectParser.h

@@ -49,6 +49,7 @@ public:
 	// Assign the stream to read from (does not take ownership of the stream, unless told so)
 	void SetReadStream(IByteReader* inSourceStream,IReadPositionProvider* inCurrentPositionProvider,bool inOwnsStream=false);
 
+	// the important bit - get next object in content stream
 	PDFObject* ParseNewObject();
 
 	// calls this when changing underlying stream position
@@ -64,6 +65,11 @@ public:
 	// helper method for others who need to parse encoded pdf data
 	std::string DecodeHexString(const std::string inStringToDecode);
 
+	// External reading. use to temporarily get access to the internal stream, instead of reading objects with ParseNewObject.
+	// when done mark with FinishExternalReading to commence reading
+	IByteReader* StartExternalRead();
+	void EndExternalRead();
+
 private:
 	PDFParserTokenizer mTokenizer;
 	StringList mTokenBuffer;

package/src/deps/PDFWriter/PDFParser.cpp

@@ -36,6 +36,7 @@
 #include "PDFStreamInput.h"
 #include "InputLimitedStream.h"
 #include "InputFlateDecodeStream.h"
+#include "InputLZWDecodeStream.h"
 #include "InputStreamSkipperStream.h"
 #include "InputPredictorPNGOptimumStream.h"
 #include "InputPredictorTIFFSubStream.h"
@@ -1925,27 +1926,39 @@ EStatusCodeAndIByteReader PDFParser::CreateFilterForStream(IByteReader* inStream
 	do
 	{
 
-		if(inFilterName->GetValue() == "FlateDecode")
+		if(inFilterName->GetValue() == "FlateDecode" || inFilterName->GetValue() == "LZWDecode")
 		{
-			InputFlateDecodeStream* flateStream;
-			flateStream = new InputFlateDecodeStream(NULL); // assigning null, so later delete, if failure occurs won't delete the input stream
-			result = flateStream;
-
-			// check for predictor n' such
-			if(!inDecodeParams)
+			if (inFilterName->GetValue() == "FlateDecode")
 			{
-				// no predictor, stop here
+				InputFlateDecodeStream* flateStream;
+				flateStream = new InputFlateDecodeStream(NULL); // assigning null, so later delete, if failure occurs won't delete the input stream
 				flateStream->Assign(inStream);
-				break;
+				result = flateStream;
+			}
+			else if (inFilterName->GetValue() == "LZWDecode")
+			{
+				InputLZWDecodeStream* lzwStream;
+				int early = 1;
+				if (inDecodeParams)
+				{
+					PDFObjectCastPtr<PDFInteger> earlyObj(QueryDictionaryObject(inDecodeParams, "EarlyChange"));
+					early = earlyObj->GetValue();
+				}
+				lzwStream = new InputLZWDecodeStream(early);
+				lzwStream->Assign(inStream);
+				result = lzwStream;
 			}
 
+			// check for predictor n' such
+			if (!inDecodeParams)
+				// no predictor, stop here
+				break;			
+
 			// read predictor, and apply the relevant predictor function
 			PDFObjectCastPtr<PDFInteger> predictor(QueryDictionaryObject(inDecodeParams,"Predictor"));
 
 			if(!predictor || predictor->GetValue() == 1)
 			{
-				// no predictor or default, stop here
-				flateStream->Assign(inStream);
 				break;
 			}
 
@@ -1994,7 +2007,6 @@ EStatusCodeAndIByteReader PDFParser::CreateFilterForStream(IByteReader* inStream
 					break;
 				}
 			}
-			flateStream->Assign(inStream);
 		}
 		else if (inFilterName->GetValue() == "ASCIIHexDecode")
 		{
@@ -2028,7 +2040,7 @@ EStatusCodeAndIByteReader PDFParser::CreateFilterForStream(IByteReader* inStream
 		}
 		else
 		{
-			TRACE_LOG("PDFParser::CreateFilterForStream, supporting only flate decode and ascii 85 decode, failing");
+			TRACE_LOG("PDFParser::CreateFilterForStream, supporting only flate decode, lzw, dct, crypt and ascii 85+hex decode, failing");
 			status = PDFHummus::eFailure;
 			break;
 		}

package/src/deps/PDFWriter/PDFParserTokenizer.cpp

@@ -67,7 +67,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 	Byte buffer;
 	OutputStringBufferStream tokenBuffer;
 	
-	if(!mStream || (!mStream->NotEnded() && !mHasTokenBuffer))
+	if(!CanGetNextByte())
 	{
 		result.first = false;
 		return result;
@@ -76,7 +76,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 	do
 	{
 		SkipTillToken();
-		if(!mStream->NotEnded())
+		if(!CanGetNextByte())
 		{
 			result.first = false;
 			break;
@@ -103,11 +103,11 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 			case '%':
 			{
 				// for a comment, the token goes on till the end of line marker [not including]
-				while(mStream->NotEnded())
+				while(CanGetNextByte())
 				{
 					if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 					{	
-						result.first = !mStream->NotEnded();
+						result.first = !CanGetNextByte();
 						break;
 					}
 					if(0xD == buffer|| 0xA == buffer)
@@ -123,11 +123,11 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 				// for a literal string, the token goes on until the balanced-closing right paranthesis
 				int balanceLevel = 1;
 				bool backSlashEncountered = false;
-				while(balanceLevel > 0 && mStream->NotEnded())
+				while(balanceLevel > 0 && CanGetNextByte())
 				{
 					if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 					{	
-						result.first = !mStream->NotEnded();
+						result.first = !CanGetNextByte();
 						break;
 					}
 			
@@ -138,11 +138,11 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 						{
 							// ignore backslash and newline. might also need to read extra
 							// for cr-ln
-							if(0xD == buffer && mStream->NotEnded())
+							if(0xD == buffer && CanGetNextByte())
 							{
 								if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 								{
-									result.first = !mStream->NotEnded();
+									result.first = !CanGetNextByte();
 									break;
 								}
 								if(buffer != 0xA)
@@ -179,7 +179,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 				// k. this might be a dictionary start marker or a hax string start. depending on whether it has a < following it or not
 
 				// Hex string, read till end of hex string marker
-				if(!mStream->NotEnded())
+				if(!CanGetNextByte())
 				{
 					result.second = tokenBuffer.ToString();
 					break;
@@ -187,7 +187,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 
 				if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 				{	
-						result.first = !mStream->NotEnded();
+						result.first = !CanGetNextByte();
 						break;
 				}
 
@@ -204,11 +204,11 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 
 					tokenBuffer.Write(&buffer,1);
 
-					while(mStream->NotEnded() && buffer != '>')
+					while(CanGetNextByte() && buffer != '>')
 					{
 						if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 						{	
-							result.first = !mStream->NotEnded();
+							result.first = !CanGetNextByte();
 							break;
 						}
 
@@ -227,7 +227,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 				break;
 			case '>': // parse end dictionary marker as a single entity or a hex string end marker
 			{
-				if(!mStream->NotEnded()) // this means a loose end string marker...wierd
+				if(!CanGetNextByte()) // this means a loose end string marker...wierd
 				{
 					result.second = tokenBuffer.ToString();
 					break;
@@ -235,7 +235,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 
 				if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 				{	
-					result.first = !mStream->NotEnded();
+					result.first = !CanGetNextByte();
 					break;
 				}
 
@@ -258,11 +258,11 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 
 			default: // regular token. read till next breaker or whitespace
 			{
-				while(mStream->NotEnded())
+				while(CanGetNextByte())
 				{
 					if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 					{	
-						result.first = !mStream->NotEnded();
+						result.first = !CanGetNextByte();
 						break;
 					}
 					if(IsPDFWhiteSpace(buffer))
@@ -279,7 +279,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 				}
 				result.second = tokenBuffer.ToString();
 				
-				if(result.first && mStream->NotEnded() && scStream == result.second)
+				if(result.first && CanGetNextByte() && scStream == result.second)
 				{
 					// k. a bit of a special case here for streams. the reading changes after the keyword "stream", 
 					// essentially forcing the next content to start after either CR, CR-LF or LF. so there might be a little
@@ -288,9 +288,9 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 					// that we should either skip one more "LF" or do nothing (based on what was parsed)
 					
 					// verify that when whitespaces are finished buffer is either CR or LF, and behave accordingly
-					while(mStream->NotEnded()) {
+					while(CanGetNextByte()) {
 						if (!IsPDFWhiteSpace(buffer)) {
-							result.first = !mStream->NotEnded(); // something wrong! not whitespace
+							result.first = !CanGetNextByte(); // something wrong! not whitespace
 							break;
 						}
 
@@ -311,7 +311,7 @@ BoolAndString PDFParserTokenizer::GetNextToken()
 						} // else - some other white space
 
 						if (GetNextByteForToken(buffer) != PDFHummus::eSuccess) {
-							result.first = !mStream->NotEnded(); //can't read but not eof. fail
+							result.first = !CanGetNextByte(); //can't read but not eof. fail
 							break;
 						}
 					}
@@ -332,11 +332,8 @@ void PDFParserTokenizer::SkipTillToken()
 {
 	Byte buffer = 0;
 
-	if(!mStream)
-		return;
-
 	// skip till hitting first non space, or segment end
-	while(mStream->NotEnded())
+	while(CanGetNextByte())
 	{
 		if(GetNextByteForToken(buffer) != PDFHummus::eSuccess)
 			break;
@@ -349,6 +346,10 @@ void PDFParserTokenizer::SkipTillToken()
 	}
 }
 
+bool PDFParserTokenizer::CanGetNextByte() {
+	return !!mStream && (mHasTokenBuffer || mStream->NotEnded());
+}
+
 EStatusCode PDFParserTokenizer::GetNextByteForToken(Byte& outByte)
 {
 	++mStreamPositionTracker; // advance position tracker, because we are reading the next byte.

package/src/deps/PDFWriter/PDFParserTokenizer.h

@@ -83,6 +83,7 @@ private:
 
 	void SkipTillToken();
 
+	bool CanGetNextByte();
 	// failure in GetNextByteForToken actually marks a true read failure, if you checked end of file before calling it...
 	PDFHummus::EStatusCode GetNextByteForToken(IOBasicTypes::Byte& outByte);
 

package/src/deps/PDFWriter/PDFWriter.h

@@ -263,8 +263,8 @@ public:
 
 	// some public image info services, for users of hummus
 	DoubleAndDoublePair GetImageDimensions(const std::string& inImageFile,unsigned long inImageIndex = 0, const PDFParsingOptions& inParsingOptions = PDFParsingOptions::DefaultPDFParsingOptions());
-  DoubleAndDoublePair GetImageDimensions(IByteReaderWithPosition* inImageStream,unsigned long inImageIndex = 0, const PDFParsingOptions& inParsingOptions = PDFParsingOptions::DefaultPDFParsingOptions());
-  EHummusImageType GetImageType(const std::string& inImageFile,unsigned long inImageIndex);
+	DoubleAndDoublePair GetImageDimensions(IByteReaderWithPosition* inImageStream,unsigned long inImageIndex = 0, const PDFParsingOptions& inParsingOptions = PDFParsingOptions::DefaultPDFParsingOptions());
+	EHummusImageType GetImageType(const std::string& inImageFile,unsigned long inImageIndex);
 	unsigned long GetImagePagesCount(const std::string& inImageFile, const PDFParsingOptions& inOptions = PDFParsingOptions::DefaultPDFParsingOptions());
 
 

package/src/deps/PDFWriter/binding.gyp

@@ -73,6 +73,7 @@
                'InputFile.cpp',
                'InputFileStream.cpp',
                'InputFlateDecodeStream.cpp',
+               'InputLZWDecodeStream.cpp',
                'InputLimitedStream.cpp',
                'InputPFBDecodeStream.cpp',
                'InputPredictorPNGOptimumStream.cpp',
@@ -239,6 +240,7 @@
                'InputDCTDecodeStream.h',
                'InputFile.h',
                'InputFileStream.h',
+               'InputLZWDecodeStream.h',
                'InputFlateDecodeStream.h',
                'InputLimitedStream.h',
                'InputPFBDecodeStream.h',

package/node_modules/detect-libc/.npmignore

@@ -1,7 +0,0 @@
-.nyc_output
-.travis.yml
-coverage
-test.js
-node_modules
-/.circleci
-/tests/integration

package/node_modules/detect-libc/bin/detect-libc.js

@@ -1,18 +0,0 @@
-#!/usr/bin/env node
-
-'use strict';
-
-var spawnSync = require('child_process').spawnSync;
-var libc = require('../');
-
-var spawnOptions = {
-  env: process.env,
-  shell: true,
-  stdio: 'inherit'
-};
-
-if (libc.isNonGlibcLinux) {
-  spawnOptions.env.LIBC = process.env.LIBC || libc.family;
-}
-
-process.exit(spawnSync(process.argv[2], process.argv.slice(3), spawnOptions).status);

package/node_modules/semver/CHANGELOG.md

@@ -1,111 +0,0 @@
-# changes log
-
-## 7.3.0
-
-* Add `subset(r1, r2)` method to determine if `r1` range is entirely
-  contained by `r2` range.
-
-## 7.2.3
-
-* Fix handling of `includePrelease` mode where version ranges like `1.0.0 -
-  2.0.0` would include `3.0.0-pre` and not `1.0.0-pre`.
-
-## 7.2.2
-
-* Fix bug where `2.0.0-pre` would be included in `^1.0.0` if
-  `includePrerelease` was set to true.
-
-## 7.2.0
-
-* Add `simplifyRange` method to attempt to generate a more human-readable
-  range expression that is equivalent to a supplied range, for a given set
-  of versions.
-
-## 7.1.2
-
-* Remove fancy lazy-loading logic, as it was causing problems for webpack
-  users.
-
-## 7.1.0
-
-* Add `require('semver/preload')` to load the entire module without using
-  lazy getter methods.
-
-## 7.0.0
-
-* Refactor module into separate files for better tree-shaking
-* Drop support for very old node versions, use const/let, `=>` functions,
-  and classes.
-
-## 6.3.0
-
-* Expose the token enum on the exports
-
-## 6.2.0
-
-* Coerce numbers to strings when passed to semver.coerce()
-* Add `rtl` option to coerce from right to left
-
-## 6.1.3
-
-* Handle X-ranges properly in includePrerelease mode
-
-## 6.1.2
-
-* Do not throw when testing invalid version strings
-
-## 6.1.1
-
-* Add options support for semver.coerce()
-* Handle undefined version passed to Range.test
-
-## 6.1.0
-
-* Add semver.compareBuild function
-* Support `*` in semver.intersects
-
-## 6.0
-
-* Fix `intersects` logic.
-
-    This is technically a bug fix, but since it is also a change to behavior
-    that may require users updating their code, it is marked as a major
-    version increment.
-
-## 5.7
-
-* Add `minVersion` method
-
-## 5.6
-
-* Move boolean `loose` param to an options object, with
-  backwards-compatibility protection.
-* Add ability to opt out of special prerelease version handling with
-  the `includePrerelease` option flag.
-
-## 5.5
-
-* Add version coercion capabilities
-
-## 5.4
-
-* Add intersection checking
-
-## 5.3
-
-* Add `minSatisfying` method
-
-## 5.2
-
-* Add `prerelease(v)` that returns prerelease components
-
-## 5.1
-
-* Add Backus-Naur for ranges
-* Remove excessively cute inspection methods
-
-## 5.0
-
-* Remove AMD/Browserified build artifacts
-* Fix ltr and gtr when using the `*` range
-* Fix for range `*` with a prerelease identifier

package/src/deps/PDFWriter/InputPredictorPNGAverageStream.cpp

@@ -1,120 +0,0 @@
-/*
-   Source File : InputPredictorPNGAverageStream.cpp
-
-
-   Copyright 2011 Gal Kahana PDFWriter
-
-   Licensed under the Apache License, Version 2.0 (the "License");
-   you may not use this file except in compliance with the License.
-   You may obtain a copy of the License at
-
-       http://www.apache.org/licenses/LICENSE-2.0
-
-   Unless required by applicable law or agreed to in writing, software
-   distributed under the License is distributed on an "AS IS" BASIS,
-   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-   See the License for the specific language governing permissions and
-   limitations under the License.
-
-   
-*/
-#include "InputPredictorPNGAverageStream.h"
-
-#include "Trace.h"
-
-
-using namespace IOBasicTypes;
-
-InputPredictorPNGAverageStream::InputPredictorPNGAverageStream(void)
-{
-	mSourceStream = NULL;
-	mBuffer = NULL;
-	mIndex = NULL;
-	mBufferSize = 0;
-	mUpValues = NULL;
-
-}
-
-InputPredictorPNGAverageStream::~InputPredictorPNGAverageStream(void)
-{
-	delete[] mBuffer;
-	delete[] mUpValues;
-	delete mSourceStream;
-}
-
-InputPredictorPNGAverageStream::InputPredictorPNGAverageStream(IByteReader* inSourceStream,IOBasicTypes::LongBufferSizeType inColumns)
-{	
-	mSourceStream = NULL;
-	mBuffer = NULL;
-	mIndex = NULL;
-	mBufferSize = 0;
-	mUpValues = NULL;
-
-	Assign(inSourceStream,inColumns);
-}
-
-
-LongBufferSizeType InputPredictorPNGAverageStream::Read(Byte* inBuffer,LongBufferSizeType inBufferSize)
-{
-	LongBufferSizeType readBytes = 0;
-	
-
-	// exhaust what's in the buffer currently
-	while(mBufferSize > (LongBufferSizeType)(mIndex - mBuffer) && readBytes < inBufferSize)
-	{
-		DecodeNextByte(inBuffer[readBytes]);
-		++readBytes;
-	}
-
-	// now repeatedly read bytes from the input stream, and decode
-	while(readBytes < inBufferSize && mSourceStream->NotEnded())
-	{
-		memcpy(mUpValues,mBuffer,mBufferSize);
-		LongBufferSizeType readFromSource = mSourceStream->Read(mBuffer, mBufferSize);
-		if (readFromSource == 0) {
-			break; // a belated end. must be flate
-		}
-		if (readFromSource != mBufferSize)
-		{
-			TRACE_LOG("InputPredictorPNGOptimumStream::Read, problem, expected columns number read. didn't make it");
-			break;
-		}
-		*mBuffer = 0; // so i can use this as "left" value...we don't care about this one...it's just a tag
-		mIndex = mBuffer+1; // skip the first tag
-
-		while(mBufferSize > (LongBufferSizeType)(mIndex - mBuffer) && readBytes < inBufferSize)
-		{
-			DecodeNextByte(inBuffer[readBytes]);
-			++readBytes;
-		}
-	}
-	return readBytes;
-}
-
-bool InputPredictorPNGAverageStream::NotEnded()
-{
-	return mSourceStream->NotEnded() || (LongBufferSizeType)(mIndex - mBuffer) < mBufferSize;
-}
-
-void InputPredictorPNGAverageStream::DecodeNextByte(Byte& outDecodedByte)
-{
-	outDecodedByte = (Byte)((char)mBuffer[mIndex-mBuffer - 1]/2 + (char)mUpValues[mIndex-mBuffer]/2 + (char)*mIndex);
-
-	*mIndex = outDecodedByte; // saving the encoded value back to the buffer, for later copying as "Up value", and current using as "Left" value
-	++mIndex;
-}
-
-void InputPredictorPNGAverageStream::Assign(IByteReader* inSourceStream,IOBasicTypes::LongBufferSizeType inColumns)
-{
-	mSourceStream = inSourceStream;
-
-	delete[] mBuffer;
-	delete[] mUpValues;
-	mBufferSize = inColumns + 1;
-	mBuffer = new Byte[mBufferSize];
-	memset(mBuffer,0,mBufferSize);
-	mUpValues = new Byte[mBufferSize];
-	memset(mUpValues,0,mBufferSize); // that's less important
-	mIndex = mBuffer + mBufferSize;
-
-}