Skip to content

Commit 788a785

Browse files
authored
Merge pull request #1359 from kiwix/1352-fix-libkiwix-14.1-loading-issue
Fix extreme time trying to load favicon data
2 parents 4fc33a3 + f2b52b9 commit 788a785

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

Model/Entities/ZimFileMetaData/ZimFileMetaData.mm

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -157,16 +157,20 @@ - (NSURL * _Nullable)getFaviconURLFromBook:(kiwix::Book *)book {
157157
}
158158

159159
- (NSData * _Nullable)getFaviconDataFromBook:(kiwix::Book *)book {
160-
try {
161-
std::string dataString = book->getIllustrations().at(0)->getData();
162-
if(dataString.length() == 0) {
163-
return nil;
164-
}
165-
NSData *favIconData = [NSData dataWithBytes: dataString.data() length: dataString.length()];
166-
return favIconData;
167-
} catch (std::exception) {
168-
return nil;
169-
}
160+
return nil;
161+
// this was never returning any data (always nil)
162+
// but since libkiwix 14.1 it takes a huge amount of time even trying it
163+
// so better not to even try, until it's not fixed
164+
// try {
165+
// std::string dataString = book->getIllustrations().at(0)->getData();
166+
// if(dataString.length() == 0) {
167+
// return nil;
168+
// }
169+
// NSData *favIconData = [NSData dataWithBytes: dataString.data() length: dataString.length()];
170+
// return favIconData;
171+
// } catch (std::exception) {
172+
// return nil;
173+
// }
170174
}
171175

172176
- (NSString *)getFlavorFromBook:(kiwix::Book *)book {

0 commit comments

Comments
 (0)