fix(file): fixed reader buffer management

fenying · fenying · commit 70c8bbd0435a · 2022-10-06T23:33:42.000+08:00
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,5 +1,9 @@
 # Changes Logs
 
+## v0.3.1
+
+- fix(file): fixed reader buffer management.
+
 ## v0.3.0
 
 - build(project): upgrade TypeScript to v4.8.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@litert/binary",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "The binary data operate library for LiteRT.",
   "main": "lib/index.js",
   "scripts": {
diff --git a/src/lib/File/AsyncFileBufferReader.ts b/src/lib/File/AsyncFileBufferReader.ts
@@ -151,8 +151,8 @@ export class AsyncFileBufferReader extends AbstractAsyncFileReader implements IF
 
         if (pos >= this._bufferStartAt && pos < this._bufferStartAt + this.bufferSize) {
 
-            this._buffer = this._buffer.subarray(pos - this._bufferStartAt);
-            this._bufferStartAt = this._pos = pos;
+            this._buffer = this._realBuffer.subarray(pos - this._bufferStartAt);
+            this._pos = pos;
         }
         else if (pos >= 0 && pos < this._length) {
 
diff --git a/src/lib/File/SyncFileBufferReader.ts b/src/lib/File/SyncFileBufferReader.ts
@@ -150,8 +150,8 @@ export class SyncFileBufferReader extends AbstractSyncFileReader {
 
         if (pos >= this._bufferStartAt && pos < this._bufferStartAt + this.bufferSize) {
 
-            this._buffer = this._buffer.subarray(pos - this._bufferStartAt);
-            this._bufferStartAt = this._pos = pos;
+            this._buffer = this._realBuffer.subarray(pos - this._bufferStartAt);
+            this._pos = pos;
         }
         else if (pos >= 0 && pos < this._length) {
 

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@litert/binary",`
`3`		`- "version": "0.3.0",`
	`3`	`+ "version": "0.3.1",`
`4`	`4`	`"description": "The binary data operate library for LiteRT.",`
`5`	`5`	`"main": "lib/index.js",`
`6`	`6`	`"scripts": {`
Original file line number	Diff line number	Diff line change
`@@ -151,8 +151,8 @@ export class AsyncFileBufferReader extends AbstractAsyncFileReader implements IF`
`151`	`151`
`152`	`152`	`if (pos >= this._bufferStartAt && pos < this._bufferStartAt + this.bufferSize) {`
`153`	`153`
`154`		`- this._buffer = this._buffer.subarray(pos - this._bufferStartAt);`
`155`		`- this._bufferStartAt = this._pos = pos;`
	`154`	`+ this._buffer = this._realBuffer.subarray(pos - this._bufferStartAt);`
	`155`	`+ this._pos = pos;`
`156`	`156`	`}`
`157`	`157`	`else if (pos >= 0 && pos < this._length) {`
`158`	`158`
Original file line number	Diff line number	Diff line change
`@@ -150,8 +150,8 @@ export class SyncFileBufferReader extends AbstractSyncFileReader {`
`150`	`150`
`151`	`151`	`if (pos >= this._bufferStartAt && pos < this._bufferStartAt + this.bufferSize) {`
`152`	`152`
`153`		`- this._buffer = this._buffer.subarray(pos - this._bufferStartAt);`
`154`		`- this._bufferStartAt = this._pos = pos;`
	`153`	`+ this._buffer = this._realBuffer.subarray(pos - this._bufferStartAt);`
	`154`	`+ this._pos = pos;`
`155`	`155`	`}`
`156`	`156`	`else if (pos >= 0 && pos < this._length) {`
`157`	`157`