From 41ca39d1fa680cf6a9b57e23107c0d83bbda2a54 Mon Sep 17 00:00:00 2001
From: Timofey Solomko <tsolomko@gmail.com>
Date: Tue, 10 Apr 2018 23:26:35 +0300
Subject: [PATCH] Remove certain TODOs

---
 Sources/TAR/TarEntryInfoProvider.swift | 5 ++---
 Sources/ZIP/ZipLocalHeader.swift       | 1 -
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/Sources/TAR/TarEntryInfoProvider.swift b/Sources/TAR/TarEntryInfoProvider.swift
index 0c68b832..fee3d5de 100644
--- a/Sources/TAR/TarEntryInfoProvider.swift
+++ b/Sources/TAR/TarEntryInfoProvider.swift
@@ -8,7 +8,6 @@ import BitByteData
 
 // While it is tempting to make Provider conform to `IteratorProtocol` and `Sequence` protocols, it is in fact
 // impossible to do so, since `TarEntryInfo.init(...)` is throwing and `IteratorProtocol.next()` cannot be throwing.
-// TODO: Struct or class?
 struct TarEntryInfoProvider {
 
     private let byteReader: ByteReader
@@ -22,8 +21,8 @@ struct TarEntryInfoProvider {
     }
 
     mutating func next() throws -> TarEntryInfo? {
-        // TODO: Check, if bytes left is >= 1024.
-        guard byteReader.data[byteReader.offset..<byteReader.offset + 1024] != Data(count: 1024)
+        guard byteReader.bytesLeft >= 1024,
+            byteReader.data[byteReader.offset..<byteReader.offset + 1024] != Data(count: 1024)
             else { return nil }
 
         let info = try TarEntryInfo(byteReader, lastGlobalExtendedHeader, lastLocalExtendedHeader,
diff --git a/Sources/ZIP/ZipLocalHeader.swift b/Sources/ZIP/ZipLocalHeader.swift
index e927a6bb..0320401a 100644
--- a/Sources/ZIP/ZipLocalHeader.swift
+++ b/Sources/ZIP/ZipLocalHeader.swift
@@ -130,7 +130,6 @@ struct ZipLocalHeader {
         guard cdEntry.diskNumberStart == currentDiskNumber
             else { throw ZipError.multiVolumesNotSupported }
 
-        // TODO: Maybe local header should "overwrite" Central Directory values?
         // Check if Local Header is consistent with Central Directory record.
         guard self.generalPurposeBitFlags == cdEntry.generalPurposeBitFlags &&
             self.compressionMethod == cdEntry.compressionMethod &&