jaewonjoo / webp

Automatically exported from code.google.com/p/webp
0 stars 0 forks source link

Now we have WIC codec on Windows, how about other OS? #53

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The attach file is a very simple plugin for QT to show WebP image on any 
QT-based applications. It's also not hard to implement others.

In my personal comparison, x264 intra frame and Microsoft's HD photo are still 
better than WebP, especially on decoder speed, which is important for thumbnail 
view.

Original issue reported on code.google.com by lvs...@gmail.com on 11 Mar 2011 at 7:35

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks! that's interesting. Would you like to include it in the code tree?

Some code remarks:

in: bool webpHandlerPrivate::load(QIODevice *device)

you're not calling "free(cache);" which will lead to memory leak.
Note that 'cache' can be NULL on error, too.

in webp.pro: it seems 'webphandler.cpp' isn't compiled or mentioned in the 
source list.

About speed: code optimization has just started. There's plenty of room for 
optimizing the decoder (in dsp.c).
Another planned feature is support decoding at reduced resolution directly in 
the core decoder, which would help pre-visualization.

Original comment by s...@google.com on 30 Mar 2011 at 9:39

GoogleCodeExporter commented 9 years ago
Also, i forgot:

you could use the WebPDecodeIntoRGB() variant (instead of WebPDecodeRGB()) in 
order to decode the picture directly into a pre-allocated 'buffer'. This would 
save a memcpy() and speed the decoding up.

Original comment by s...@google.com on 30 Mar 2011 at 9:41