> On 2010-11-22 21:10:52, Christoph Feck wrote: > > Wouldn't it be much simpler to use a scaled QPainter on the QImage to > > render the filmstrip holes at different sizes? > > Thomas Lübking wrote: > ... besides that the (hard)code(d strips - at least form a glimpse on the > diff) does not look endian safe - iff we still care about IBM/Motorola > architectures (no idea what's required for bi-enders like ARM in this regard)
Sure. The QPainter would be a better solution. But that would be the second step. This are the changes from the original project. >From now on we could fork the code and implement our own filmstrip painting >routines ... - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5937/#review8904 ----------------------------------------------------------- On 2010-11-22 20:16:46, Andreas Scherf wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/5937/ > ----------------------------------------------------------- > > (Updated 2010-11-22 20:16:46) > > > Review request for kdelibs. > > > Summary > ------- > > + Some type changes (uinXX to quintXX), some code cleanup and some compile > fixes. > > > Diffs > ----- > > trunk/KDE/kdemultimedia/CMakeLists.txt 1199813 > trunk/KDE/kdemultimedia/cmake/modules/FindFFMPEG.cmake PRE-CREATION > trunk/KDE/kdemultimedia/ffmpegthumbs/CMakeLists.txt 1199813 > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/ChangeLog 1199813 > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/filmstrip.h > PRE-CREATION > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/filmstripfilter.cpp > 1199813 > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/moviedecoder.h > 1199813 > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/moviedecoder.cpp > 1199813 > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/videoframe.h 1199813 > trunk/KDE/kdemultimedia/ffmpegthumbs/ffmpegthumbnailer/videothumbnailer.h > 1199813 > > Diff: http://svn.reviewboard.kde.org/r/5937/diff > > > Testing > ------- > > > Thanks, > > Andreas > >
