Overview
| Comment: | Fixed a rare case of losing data from the decompressor's internal result buffer. |
|---|---|
| Downloads: | Tarball | ZIP archive | SQL archive |
| Timelines: | family | ancestors | descendants | both | trunk |
| Files: | files | file ages | folders |
| SHA1: | 7b74fd57f8c51aeb7459a1a27cc1d0a023273c3b |
| User & Date: | spaskalev on 2014-12-21 17:23:23 |
| Other Links: | manifest | tags |
Context
|
2014-12-21
| ||
| 19:38 | Added debug/pprof to ease basic cpu profiling check-in: 1a4bdf36e2 user: spaskalev tags: trunk | |
| 17:23 | Fixed a rare case of losing data from the decompressor's internal result buffer. check-in: 7b74fd57f8 user: spaskalev tags: trunk | |
| 01:59 | Added a function that reverses the bits in a byte. Coverage: 100.0% of statements. check-in: 2be2ff6bf7 user: spaskalev tags: trunk | |
Changes
Modified src/0dev.org/predictor/predictor.go from [90c8b12e57] to [d2a3bd9d21].
137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 |
// Check whether we have leftover data in the buffer
if len(ctx.input) > 0 {
readCount = copy(output, ctx.input)
// Check whether we still have leftover data in the buffer :)
if readCount < len(ctx.input) {
copy(ctx.input[:readCount], ctx.input[readCount:])
ctx.input = ctx.input[:readCount]
}
return readCount, nil
}
// This is single-iteration only but it is fine according to io.Reader's contract ?!
// TODO - read all bytes from a block based on the hamming weight of the flag
// and just shuffle them for predictions instead of bite-sized reads ;)
|
< | |
137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 |
// Check whether we have leftover data in the buffer
if len(ctx.input) > 0 {
readCount = copy(output, ctx.input)
// Check whether we still have leftover data in the buffer :)
if readCount < len(ctx.input) {
ctx.input = ctx.input[:copy(ctx.input, ctx.input[readCount:])]
}
return readCount, nil
}
// This is single-iteration only but it is fine according to io.Reader's contract ?!
// TODO - read all bytes from a block based on the hamming weight of the flag
// and just shuffle them for predictions instead of bite-sized reads ;)
|