Merge pull request #40 from dfed/dfed--bugfix

Ensure overwriting cache can always read the full cache
dfed · Apr 24, 2020 · d6cfc12 · d6cfc12
2 parents b0f588d + 2bf6010
commit d6cfc12
Show file tree

Hide file tree

Showing 4 changed files with 553 additions and 255 deletions.
diff --git a/CacheAdvance.podspec b/CacheAdvance.podspec
@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
  s.name = 'CacheAdvance'
- s.version = '1.0.0'
+ s.version = '1.0.1'
  s.license = 'Apache License, Version 2.0'
  s.summary = 'A cache that enables the performant persistence of individual messages to disk'
  s.homepage = 'https://github.com/dfed/CacheAdvance'

diff --git a/Sources/CacheAdvance/CacheAdvance.swift b/Sources/CacheAdvance/CacheAdvance.swift
@@ -189,7 +189,7 @@ public final class CacheAdvance<T: Codable> {
  /// - Parameter messageLength: the length of the next message that will be written.
  private func prepareReaderForWriting(dataOfLength messageLength: Bytes) throws {
  while writer.offsetInFile < reader.offsetInFile
- && reader.offsetInFile < writer.offsetInFile + messageLength
+ && reader.offsetInFile <= writer.offsetInFile + messageLength
  {
  // The current position of the writer is before the oldest message, which means that writing this message would write into the current message.
  // Advance to the next message.