diff options
author | vng <viktor.govako@gmail.com> | 2013-05-29 15:42:29 +0400 |
---|---|---|
committer | Alex Zolotarev <alex@maps.me> | 2015-09-23 01:55:33 +0300 |
commit | 13f6b4ab21b1cc6e7b976f9997f2cc2687d977b1 (patch) | |
tree | 14f4e0207e70f96288578011165c2236c08d4c6a /coding/zip_reader.cpp | |
parent | 50393615dde78a45d83ce2e6878a96da07c85928 (diff) |
Minor code style fix. Correct close guard place.
Diffstat (limited to 'coding/zip_reader.cpp')
-rw-r--r-- | coding/zip_reader.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/coding/zip_reader.cpp b/coding/zip_reader.cpp index cb123cac2a..843d9ef8b5 100644 --- a/coding/zip_reader.cpp +++ b/coding/zip_reader.cpp @@ -23,10 +23,10 @@ ZipFileReader::ZipFileReader(string const & container, string const & file) MYTHROW(LocateZipException, ("Can't locate file inside zip", file)); if (UNZ_OK != unzOpenCurrentFile(zip)) - MYTHROW(LocateZipException, ("Can't open file inside zip", file)); + MYTHROW(LocateZipException, ("Can't open file inside zip", file)); - uint64_t offset = unzGetCurrentFileZStreamPos64(zip); - unzCloseCurrentFile(zip); + uint64_t const offset = unzGetCurrentFileZStreamPos64(zip); + (void) unzCloseCurrentFile(zip); if (offset > Size()) MYTHROW(LocateZipException, ("Invalid offset inside zip", file)); @@ -82,15 +82,14 @@ void ZipFileReader::UnzipFile(string const & zipContainer, string const & fileIn MYTHROW(LocateZipException, ("Can't locate file inside zip", fileInZip)); if (UNZ_OK != unzOpenCurrentFile(zip)) - MYTHROW(LocateZipException, ("Can't open file inside zip", fileInZip)); + MYTHROW(LocateZipException, ("Can't open file inside zip", fileInZip)); + MY_SCOPE_GUARD(currentFileGuard, bind(&unzCloseCurrentFile, zip)); unz_file_info64 fileInfo; if (UNZ_OK != unzGetCurrentFileInfo64(zip, &fileInfo, NULL, 0, NULL, 0, NULL, 0)) MYTHROW(LocateZipException, ("Can't get uncompressed file size inside zip", fileInZip)); - MY_SCOPE_GUARD(currentFileGuard, bind(&unzCloseCurrentFile, zip)); - - static size_t const BUF_SIZE = 1024 * 50; + size_t const BUF_SIZE = 1024 * 50; vector<char> buf(BUF_SIZE); // First outFile should be closed, then FileWriter::DeleteFileX is called, |