xymp
/
exiv2
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

clang-tidy: for loop conversion

Browse Source 
Found with modernize-loop-convert

Signed-off-by: Rosen Penev <rosenp@gmail.com>
main
Rosen Penev 10 months ago
parent 169b6364a8
commit 5605379b3d
9 changed files with 43 additions and 55 deletions
Show Stats Download Patch File Download Diff File

14
samples/exifdata-test.cpp
Unescape Escape View File

@ -98,13 +98,11 @@ void print(const std::string& file) {
auto image = Exiv2::ImageFactory::open(file);
image->readMetadata();
Exiv2::ExifData& ed = image->exifData();
auto end = ed.end();
for (auto i = ed.begin(); i != end; ++i) {
std::cout << std::setw(45) << std::setfill(' ') << std::left << i->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i->tag() << " " << std::setw(12)
<< std::setfill(' ') << std::left << i->ifdName() << " " << std::setw(9) << std::setfill(' ') << std::left
<< i->typeName() << " " << std::dec << std::setw(3) << std::setfill(' ') << std::right << i->count()
<< " " << std::dec << i->toString() << "\n";
for (const auto& i : image->exifData()) {
std::cout << std::setw(45) << std::setfill(' ') << std::left << i.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i.tag() << " " << std::setw(12)
<< std::setfill(' ') << std::left << i.ifdName() << " " << std::setw(9) << std::setfill(' ') << std::left
<< i.typeName() << " " << std::dec << std::setw(3) << std::setfill(' ') << std::right << i.count() << " "
<< std::dec << i.toString() << "\n";
}
}

1
samples/exifdata.cpp
Unescape Escape View File

@ -7,7 +7,6 @@
#include <string>
using format_t = std::map<std::string, int>;
using format_i = format_t::const_iterator;
enum format_e { wolf, csv, json, xml };
void syntax(const char* argv[], format_t& formats) {

19
samples/exifprint.cpp
Unescape Escape View File

@ -77,25 +77,24 @@ int main(int argc, char* const argv[]) {
shortLong.insert("Exif.Photo.StripOffsets");
shortLong.insert("Exif.Photo.StripByteCounts");
auto end = exifData.end();
for (auto i = exifData.begin(); i != end; ++i) {
for (const auto& i : exifData) {
if (!bLint) {
const char* tn = i->typeName();
std::cout << std::setw(44) << std::setfill(' ') << std::left << i->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i->tag() << " "
const char* tn = i.typeName();
std::cout << std::setw(44) << std::setfill(' ') << std::left << i.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i.tag() << " "
<< std::setw(9) << std::setfill(' ') << std::left << (tn ? tn : "Unknown") << " " << std::dec
<< std::setw(3) << std::setfill(' ') << std::right << i->count() << " " << std::dec << i->toString()
<< std::setw(3) << std::setfill(' ') << std::right << i.count() << " " << std::dec << i.toString()
<< "\n";
} else {
const Exiv2::TagInfo* tagInfo = findTag(Exiv2::ExifTags::tagList(i->groupName()), i->tag());
const Exiv2::TagInfo* tagInfo = findTag(Exiv2::ExifTags::tagList(i.groupName()), i.tag());
if (tagInfo) {
Exiv2::TypeId type = i->typeId();
Exiv2::TypeId type = i.typeId();
if (type != tagInfo->typeId_ &&
(tagInfo->typeId_ != Exiv2::comment || type != Exiv2::undefined) // comment is stored as undefined
&& (shortLong.find(i->key()) == shortLong.end() ||
&& (shortLong.find(i.key()) == shortLong.end() ||
(type != Exiv2::unsignedShort && type != Exiv2::unsignedLong)) // can be short or long!
) {
std::cerr << i->key() << " type " << i->typeName() << " (" << type << ")"
std::cerr << i.key() << " type " << i.typeName() << " (" << type << ")"
<< " expected " << Exiv2::TypeInfo::typeName(tagInfo->typeId_) << " (" << tagInfo->typeId_ << ")"
<< '\n';
rc = 2;

11
samples/iptcprint.cpp
Unescape Escape View File

@ -24,12 +24,11 @@ int main(int argc, char* const argv[]) try {
throw Exiv2::Error(Exiv2::ErrorCode::kerErrorMessage, error);
}
auto end = iptcData.end();
for (auto md = iptcData.begin(); md != end; ++md) {
std::cout << std::setw(44) << std::setfill(' ') << std::left << md->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << md->tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << md->typeName() << " " << std::dec << std::setw(3)
<< std::setfill(' ') << std::right << md->count() << " " << std::dec << md->value() << '\n';
for (const auto& md : iptcData) {
std::cout << std::setw(44) << std::setfill(' ') << std::left << md.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << md.tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << md.typeName() << " " << std::dec << std::setw(3) << std::setfill(' ')
<< std::right << md.count() << " " << std::dec << md.value() << '\n';
}
return EXIT_SUCCESS;

3
samples/prevtest.cpp
Unescape Escape View File

@ -19,8 +19,7 @@ int main(int argc, char* const argv[]) try {
image->readMetadata();
Exiv2::PreviewManager loader(*image);
Exiv2::PreviewPropertiesList list = loader.getPreviewProperties();
for (auto&& pos : list) {
for (const auto& pos : loader.getPreviewProperties()) {
std::cout << pos.mimeType_ << " preview, type " << pos.id_ << ", " << pos.size_ << " bytes, " << pos.width_ << 'x'
<< pos.height_ << " pixels"
<< "\n";

14
samples/remotetest.cpp
Unescape Escape View File

@ -56,14 +56,12 @@ int main(int argc, char* const argv[]) {
error += ": No Exif data found in the file";
throw Exiv2::Error(Exiv2::ErrorCode::kerErrorMessage, error);
}
auto end = exifReadData.end();
for (auto i = exifReadData.begin(); i != end; ++i) {
const char* tn = i->typeName();
std::cout << std::setw(44) << std::setfill(' ') << std::left << i->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i->tag() << " "
<< std::setw(9) << std::setfill(' ') << std::left << (tn ? tn : "Unknown") << " " << std::dec
<< std::setw(3) << std::setfill(' ') << std::right << i->count() << " " << std::dec << i->value()
<< "\n";
for (const auto& i : exifReadData) {
const char* tn = i.typeName();
std::cout << std::setw(44) << std::setfill(' ') << std::left << i.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i.tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << (tn ? tn : "Unknown") << " " << std::dec << std::setw(3)
<< std::setfill(' ') << std::right << i.count() << " " << std::dec << i.value() << "\n";
}
// del, reset the metadata

11
samples/tiff-test.cpp
Unescape Escape View File

@ -87,11 +87,10 @@ void print(const ExifData& exifData) {
std::string error("No Exif data found in the file");
throw Exiv2::Error(ErrorCode::kerErrorMessage, error);
}
auto end = exifData.end();
for (auto i = exifData.begin(); i != end; ++i) {
std::cout << std::setw(44) << std::setfill(' ') << std::left << i->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i->tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << i->typeName() << " " << std::dec << std::setw(3) << std::setfill(' ')
<< std::right << i->count() << " " << std::dec << i->value() << "\n";
for (const auto& i : exifData) {
std::cout << std::setw(44) << std::setfill(' ') << std::left << i.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i.tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << i.typeName() << " " << std::dec << std::setw(3) << std::setfill(' ')
<< std::right << i.count() << " " << std::dec << i.value() << "\n";
}
}

11
samples/write-test.cpp
Unescape Escape View File

@ -153,11 +153,10 @@ void testCase(const std::string& file1, const std::string& file2, const std::str
// *****************************************************************************
void exifPrint(const ExifData& exifData) {
auto i = exifData.begin();
for (; i != exifData.end(); ++i) {
std::cout << std::setw(44) << std::setfill(' ') << std::left << i->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i->tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << i->typeName() << " " << std::dec << std::setw(3) << std::setfill(' ')
<< std::right << i->count() << " " << std::dec << i->value() << "\n";
for (const auto& i : exifData) {
std::cout << std::setw(44) << std::setfill(' ') << std::left << i.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i.tag() << " " << std::setw(9)
<< std::setfill(' ') << std::left << i.typeName() << " " << std::dec << std::setw(3) << std::setfill(' ')
<< std::right << i.count() << " " << std::dec << i.value() << "\n";
}
}

14
samples/write2-test.cpp
Unescape Escape View File

@ -201,13 +201,11 @@ void print(const std::string& file) {
auto image = Exiv2::ImageFactory::open(file);
image->readMetadata();
Exiv2::ExifData& ed = image->exifData();
auto end = ed.end();
for (auto i = ed.begin(); i != end; ++i) {
std::cout << std::setw(45) << std::setfill(' ') << std::left << i->key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i->tag() << " " << std::setw(12)
<< std::setfill(' ') << std::left << i->ifdName() << " " << std::setw(9) << std::setfill(' ') << std::left
<< i->typeName() << " " << std::dec << std::setw(3) << std::setfill(' ') << std::right << i->count()
<< " " << std::dec << i->value() << "\n";
for (const auto& i : image->exifData()) {
std::cout << std::setw(45) << std::setfill(' ') << std::left << i.key() << " "
<< "0x" << std::setw(4) << std::setfill('0') << std::right << std::hex << i.tag() << " " << std::setw(12)
<< std::setfill(' ') << std::left << i.ifdName() << " " << std::setw(9) << std::setfill(' ') << std::left
<< i.typeName() << " " << std::dec << std::setw(3) << std::setfill(' ') << std::right << i.count() << " "
<< std::dec << i.value() << "\n";
}
}

Write Preview
Loading…
Cancel
Save