Merge pull request #878 from shenlebantongying/staged

guard against null img for ZIM
This commit is contained in:
xiaoyifang 2023-06-19 17:47:47 +08:00 committed by GitHub
commit 1185e8896f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -290,7 +290,14 @@ void ZimDictionary::loadIcon() noexcept
try {
auto illustration = df.getIllustrationItem( 48 ).getData();
QImage img = QImage::fromData( reinterpret_cast< const uchar * >( illustration.data() ), illustration.size() );
dictionaryIcon = QIcon( QPixmap::fromImage( img ) );
if ( img.isNull() ) {
// Fallback to default icon
dictionaryIcon = QIcon( ":/icons/icon32_zim.png" );
}
else {
dictionaryIcon = QIcon( QPixmap::fromImage( img ) );
}
dictionaryIconLoaded = true;
return;
@ -298,11 +305,6 @@ void ZimDictionary::loadIcon() noexcept
catch ( zim::EntryNotFound & e ) {
gdDebug( "ZIM icon not loaded for: %s", dictionaryName.c_str() );
}
// Fallback to default icon
dictionaryIcon = QIcon( ":/icons/icon32_zim.png" );
dictionaryIconLoaded = true;
}
quint32 ZimDictionary::loadArticle( quint32 address, string & articleText, bool rawText )