Skip to content

Commit 4abbf10

Browse files
committed
fix review feedback
1 parent db0544b commit 4abbf10

2 files changed

Lines changed: 10 additions & 10 deletions

File tree

OpenDocumentReader.xcodeproj/project.pbxproj

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -604,7 +604,7 @@
604604
CLANG_CXX_LANGUAGE_STANDARD = "gnu++20";
605605
CLANG_ENABLE_MODULES = YES;
606606
CODE_SIGN_STYLE = Automatic;
607-
CURRENT_PROJECT_VERSION = 40;
607+
CURRENT_PROJECT_VERSION = 42;
608608
DEFINES_MODULE = YES;
609609
DEVELOPMENT_TEAM = 5LS6X97G6J;
610610
ENABLE_BITCODE = NO;
@@ -616,7 +616,7 @@
616616
"$(inherited)",
617617
"@executable_path/Frameworks",
618618
);
619-
MARKETING_VERSION = 1.33;
619+
MARKETING_VERSION = 1.34;
620620
ONLY_ACTIVE_ARCH = YES;
621621
PRODUCT_BUNDLE_IDENTIFIER = at.tomtasche.reader.lite1;
622622
PRODUCT_NAME = "$(BUNDLE_DISPLAY_NAME)";
@@ -695,7 +695,7 @@
695695
CLANG_CXX_LANGUAGE_STANDARD = "gnu++20";
696696
CLANG_ENABLE_MODULES = YES;
697697
CODE_SIGN_STYLE = Automatic;
698-
CURRENT_PROJECT_VERSION = 40;
698+
CURRENT_PROJECT_VERSION = 42;
699699
DEFINES_MODULE = YES;
700700
DEVELOPMENT_TEAM = 5LS6X97G6J;
701701
ENABLE_BITCODE = NO;
@@ -707,7 +707,7 @@
707707
"$(inherited)",
708708
"@executable_path/Frameworks",
709709
);
710-
MARKETING_VERSION = 1.33;
710+
MARKETING_VERSION = 1.34;
711711
PRODUCT_BUNDLE_IDENTIFIER = at.tomtasche.reader.lite1;
712712
PRODUCT_NAME = "$(BUNDLE_DISPLAY_NAME)";
713713
SUPPORTED_PLATFORMS = "iphoneos iphonesimulator";
@@ -930,7 +930,7 @@
930930
CLANG_CXX_LANGUAGE_STANDARD = "gnu++20";
931931
CLANG_ENABLE_MODULES = YES;
932932
CODE_SIGN_STYLE = Automatic;
933-
CURRENT_PROJECT_VERSION = 40;
933+
CURRENT_PROJECT_VERSION = 42;
934934
DEFINES_MODULE = YES;
935935
DEVELOPMENT_TEAM = 5LS6X97G6J;
936936
ENABLE_BITCODE = NO;
@@ -942,7 +942,7 @@
942942
"$(inherited)",
943943
"@executable_path/Frameworks",
944944
);
945-
MARKETING_VERSION = 1.33;
945+
MARKETING_VERSION = 1.34;
946946
ONLY_ACTIVE_ARCH = YES;
947947
PRODUCT_BUNDLE_IDENTIFIER = "at.tomtasche.reader$(BUNDLE_ID_SUFFIX)";
948948
PRODUCT_NAME = "$(BUNDLE_DISPLAY_NAME)";
@@ -965,7 +965,7 @@
965965
CLANG_CXX_LANGUAGE_STANDARD = "gnu++20";
966966
CLANG_ENABLE_MODULES = YES;
967967
CODE_SIGN_STYLE = Automatic;
968-
CURRENT_PROJECT_VERSION = 40;
968+
CURRENT_PROJECT_VERSION = 42;
969969
DEFINES_MODULE = YES;
970970
DEVELOPMENT_TEAM = 5LS6X97G6J;
971971
ENABLE_BITCODE = NO;
@@ -977,7 +977,7 @@
977977
"$(inherited)",
978978
"@executable_path/Frameworks",
979979
);
980-
MARKETING_VERSION = 1.33;
980+
MARKETING_VERSION = 1.34;
981981
PRODUCT_BUNDLE_IDENTIFIER = "at.tomtasche.reader$(BUNDLE_ID_SUFFIX)";
982982
PRODUCT_NAME = "$(BUNDLE_DISPLAY_NAME)";
983983
SUPPORTED_PLATFORMS = "iphoneos iphonesimulator";

OpenDocumentReader/CoreWrapper.mm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ - (bool)translate:(NSString *)inputPath into:(NSString *)outputPath with:(NSStri
5050
std::vector<odr::FileType> fileTypes;
5151
try {
5252
fileTypes = odr::OpenDocumentReader::types(inputPathCpp);
53-
if (types.empty()) {
53+
if (fileTypes.empty()) {
5454
_errorCode = @(-5);
5555
return false;
5656
}
@@ -59,7 +59,7 @@ - (bool)translate:(NSString *)inputPath into:(NSString *)outputPath with:(NSStri
5959
return false;
6060
}
6161

62-
if (fileTypes.find(odr::FileType::portable_document_format) != fileTypes.end()) {
62+
if (std::find(fileTypes.begin(), fileTypes.end(), odr::FileType::portable_document_format) != fileTypes.end()) {
6363
_errorCode = @(-5);
6464
return false;
6565
}

0 commit comments

Comments
 (0)