Summary: | Loading meta information failed. | ||
---|---|---|---|
Product: | [Applications] gwenview | Reporter: | Zack Rusin <zack> |
Component: | general | Assignee: | Gwenview Bugs <gwenview-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | Mathias.Homann, pzcdyhbb0nzq9layi4mcj1hhp, ru.energy |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Debian testing | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | sample image |
Description
Zack Rusin
2009-03-26 20:38:43 UTC
Created attachment 32418 [details]
sample image
The same problem. It seems the tga reader from KImgIO does not recognize your image as a .tga. Can you have a look at this patch I posted to reviewboard? http://reviewboard.kde.org/r/634/ I think the patch looks good. I wouldn't worry about performance at all. In the worst case working-slow is still a lot better than not-working-fast. I don't remember of the top of my head what IsSupported does with TgaHeader, but QRegExp is slow, and since the image header was just read the caches should still be warm for the qdatastream to read the file so it should be ok (unless IsSupported does some global matching with QRegExp of course =) ). > In the worst case working-slow is still a lot better than not-working-fast.
I was worried about performance when people open other images, since canRead() is called for all image, unless the user specify the format.
SVN commit 960288 by gateau: Added test image from bug report CCBUG:188191 A 188191_does_not_load.tga M +2 -0 documenttest.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=960288 SVN commit 963718 by gateau: Use TgaHeader instead of a regex in canRead(). BUG:188191 M +12 -6 tga.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=963718 *** Bug 194130 has been marked as a duplicate of this bug. *** *** Bug 189338 has been marked as a duplicate of this bug. *** |