GH #1050 Zip: fix gcc -Wshadow warnings

This commit is contained in:
Miklos Vajna
2016-03-25 08:42:58 +01:00
parent 75502b81b1
commit 714dc6ddd2
9 changed files with 20 additions and 20 deletions

View File

@@ -68,17 +68,17 @@ inline CppUnitException::CppUnitException(const CppUnitException& other): except
} }
inline CppUnitException::CppUnitException (const std::string& message, long lineNumber, const std::string& fileName): _message(message), _lineNumber(lineNumber), _data1lineNumber(CPPUNIT_UNKNOWNLINENUMBER), _data2lineNumber(CPPUNIT_UNKNOWNLINENUMBER), _fileName(fileName) inline CppUnitException::CppUnitException (const std::string& message, long line, const std::string& rFileName): _message(message), _lineNumber(line), _data1lineNumber(CPPUNIT_UNKNOWNLINENUMBER), _data2lineNumber(CPPUNIT_UNKNOWNLINENUMBER), _fileName(rFileName)
{ {
} }
inline CppUnitException::CppUnitException (const std::string& message, long lineNumber, long data1lineNumber, const std::string& fileName): _message(message), _lineNumber(lineNumber), _data1lineNumber(data1lineNumber), _data2lineNumber(CPPUNIT_UNKNOWNLINENUMBER), _fileName(fileName) inline CppUnitException::CppUnitException (const std::string& message, long line, long data1lineNumber, const std::string& rFileName): _message(message), _lineNumber(line), _data1lineNumber(data1lineNumber), _data2lineNumber(CPPUNIT_UNKNOWNLINENUMBER), _fileName(rFileName)
{ {
} }
inline CppUnitException::CppUnitException (const std::string& message, long lineNumber, long data1lineNumber, long data2lineNumber, const std::string& fileName): _message(message), _lineNumber(lineNumber), _data1lineNumber(data1lineNumber), _data2lineNumber(data2lineNumber), _fileName(fileName) inline CppUnitException::CppUnitException (const std::string& message, long line, long data1lineNumber, long data2lineNumber, const std::string& rFileName): _message(message), _lineNumber(line), _data1lineNumber(data1lineNumber), _data2lineNumber(data2lineNumber), _fileName(rFileName)
{ {
} }

View File

@@ -56,10 +56,10 @@ class TestCaller: public TestCase
typedef void (Fixture::*TestMethod)(); typedef void (Fixture::*TestMethod)();
public: public:
TestCaller(const std::string& name, TestMethod test): TestCaller(const std::string& rName, TestMethod test):
TestCase(name), TestCase(rName),
_test(test), _test(test),
_fixture(new Fixture(name)) _fixture(new Fixture(rName))
{ {
} }

View File

@@ -173,7 +173,7 @@ protected:
// Constructs a test case // Constructs a test case
inline TestCase::TestCase(const std::string& name): _name (name) inline TestCase::TestCase(const std::string& rName): _name (rName)
{ {
} }

View File

@@ -52,7 +52,7 @@ protected:
// Constructs a TestFailure with the given test and exception. // Constructs a TestFailure with the given test and exception.
inline TestFailure::TestFailure(Test* failedTest, CppUnitException* thrownException): _failedTest(failedTest), _thrownException(thrownException) inline TestFailure::TestFailure(Test* pFailedTest, CppUnitException* pThrownException): _failedTest(pFailedTest), _thrownException(pThrownException)
{ {
} }

View File

@@ -155,8 +155,8 @@ void Compress::addFileRaw(std::istream& in, const ZipLocalFileHeader& h, const P
if(hdr.hasExtraField()) // Update sizes in header extension. if(hdr.hasExtraField()) // Update sizes in header extension.
hdr.setZip64Data(); hdr.setZip64Data();
_out.seekp(hdr.getStartPos(), std::ios_base::beg); _out.seekp(hdr.getStartPos(), std::ios_base::beg);
std::string header = hdr.createHeader(); std::string headerString = hdr.createHeader();
_out.write(header.c_str(), static_cast<std::streamsize>(header.size())); _out.write(headerString.c_str(), static_cast<std::streamsize>(headerString.size()));
_out.seekp(0, std::ios_base::end); _out.seekp(0, std::ios_base::end);
} }
@@ -273,8 +273,8 @@ void Compress::addRecursive(const Poco::Path& entry, ZipCommon::CompressionMetho
{ {
Poco::Path realFile(entry, *it); Poco::Path realFile(entry, *it);
Poco::Path renamedFile(aName, *it); Poco::Path renamedFile(aName, *it);
Poco::File aFile(realFile); Poco::File file(realFile);
if (aFile.isDirectory()) if (file.isDirectory())
{ {
realFile.makeDirectory(); realFile.makeDirectory();
renamedFile.makeDirectory(); renamedFile.makeDirectory();

View File

@@ -31,7 +31,7 @@ const char ZipLocalFileHeader::HEADER[ZipCommon::HEADER_SIZE] = {'\x50', '\x4b',
ZipLocalFileHeader::ZipLocalFileHeader(const Poco::Path& fileName, ZipLocalFileHeader::ZipLocalFileHeader(const Poco::Path& fileName,
const Poco::DateTime& lastModifiedAt, const Poco::DateTime& lastModified,
ZipCommon::CompressionMethod cm, ZipCommon::CompressionMethod cm,
ZipCommon::CompressionLevel cl, ZipCommon::CompressionLevel cl,
bool forceZip64): bool forceZip64):
@@ -62,7 +62,7 @@ ZipLocalFileHeader::ZipLocalFileHeader(const Poco::Path& fileName,
setHostSystem(hs); setHostSystem(hs);
setEncryption(false); setEncryption(false);
setExtraFieldSize(0); setExtraFieldSize(0);
setLastModifiedAt(lastModifiedAt); setLastModifiedAt(lastModified);
init(fileName, cm, cl); init(fileName, cm, cl);
} }
@@ -197,12 +197,12 @@ bool ZipLocalFileHeader::searchCRCAndSizesAfterData() const
} }
void ZipLocalFileHeader::setFileName(const std::string& fileName, bool isDirectory) void ZipLocalFileHeader::setFileName(const std::string& fileName, bool directory)
{ {
poco_assert (!fileName.empty()); poco_assert (!fileName.empty());
Poco::Path aPath(fileName); Poco::Path aPath(fileName);
if (isDirectory) if (directory)
{ {
aPath.makeDirectory(); aPath.makeDirectory();
setCRC(0); setCRC(0);
@@ -218,7 +218,7 @@ void ZipLocalFileHeader::setFileName(const std::string& fileName, bool isDirecto
_fileName = aPath.toString(Poco::Path::PATH_UNIX); _fileName = aPath.toString(Poco::Path::PATH_UNIX);
if (_fileName[0] == '/') if (_fileName[0] == '/')
_fileName = _fileName.substr(1); _fileName = _fileName.substr(1);
if (isDirectory) if (directory)
{ {
poco_assert_dbg (_fileName[_fileName.size()-1] == '/'); poco_assert_dbg (_fileName[_fileName.size()-1] == '/');
} }

View File

@@ -28,7 +28,7 @@
using namespace Poco::Zip; using namespace Poco::Zip;
CompressTest::CompressTest(const std::string& name): CppUnit::TestCase(name) CompressTest::CompressTest(const std::string& rName): CppUnit::TestCase(rName)
{ {
} }

View File

@@ -20,7 +20,7 @@
using namespace Poco::Zip; using namespace Poco::Zip;
PartialStreamTest::PartialStreamTest(const std::string& name): CppUnit::TestCase(name) PartialStreamTest::PartialStreamTest(const std::string& rName): CppUnit::TestCase(rName)
{ {
} }

View File

@@ -35,7 +35,7 @@
using namespace Poco::Zip; using namespace Poco::Zip;
ZipTest::ZipTest(const std::string& name): CppUnit::TestCase(name) ZipTest::ZipTest(const std::string& rName): CppUnit::TestCase(rName)
{ {
} }