Changeset 769 for trunk/src/gui/image

Timestamp:

Aug 2, 2010, 9:27:30 PM (15 years ago)

Author:

Dmitry A. Kuminov

Message:

trunk: Merged in qt 4.6.3 sources from branches/vendor/nokia/qt.

Location:

trunk

Files:

• . (modified) (1 prop)
• src/gui/image/qicon.cpp (modified) (2 diffs)
• src/gui/image/qiconloader.cpp (modified) (2 diffs)
• src/gui/image/qiconloader_p.h (modified) (2 diffs)
• src/gui/image/qimage.cpp (modified) (2 diffs)
• src/gui/image/qimagepixmapcleanuphooks.cpp (modified) (4 diffs)
• src/gui/image/qimagepixmapcleanuphooks_p.h (modified) (1 diff)
• src/gui/image/qimagereader.cpp (modified) (2 diffs)
• src/gui/image/qimagewriter.cpp (modified) (1 diff)
• src/gui/image/qnativeimage.cpp (modified) (1 diff)
• src/gui/image/qpaintengine_pic.cpp (modified) (1 diff)
• src/gui/image/qpicture.cpp (modified) (1 diff)
• src/gui/image/qpixmap.cpp (modified) (5 diffs)
• src/gui/image/qpixmap_raster.cpp (modified) (1 diff)
• src/gui/image/qpixmap_s60.cpp (modified) (2 diffs)
• src/gui/image/qpixmap_x11.cpp (modified) (1 diff)
• src/gui/image/qpixmapfilter.cpp (modified) (7 diffs)

trunk/src/gui/image/qicon.cpp

@@ -105 +105 @@
 static QBasicAtomicInt serialNumCounter = Q_BASIC_ATOMIC_INITIALIZER(1);
 
+
+
+
+
+
+
+
+
+
 QIconPrivate::QIconPrivate()
     : engine(0), ref(1),
@@ -964 +973 @@
 QIcon QIcon::fromTheme(const QString &name, const QIcon &fallback)
 {
-    static QCache <QString, QIcon> iconCache;
-
     QIcon icon;
 
-    if (iconCache.contains(name)) {
-        icon = *iconCache.object(name);
+    if (contains(name)) {
+        icon = *object(name);
     } else {
         QIcon *cachedIcon  = new QIcon(new QIconLoaderEngine(name));
-        iconCache.insert(name, cachedIcon);
+        insert(name, cachedIcon);
         icon = *cachedIcon;
     }
 
-    if (icon.availableSizes().isEmpty())
+    // Note the qapp check is to allow lazy loading of static icons
+    // Supporting fallbacks will not work for this case.
+    if (qApp && icon.availableSizes().isEmpty())
         return fallback;
 

trunk/src/gui/image/qiconloader.cpp

@@ -86 +86 @@
 
 QIconLoader::QIconLoader() :
-        m_themeKey(1), m_supportsSvg(false)
-{
-    m_systemTheme = qt_guiPlatformPlugin()->systemIconThemeName();
-    if (m_systemTheme.isEmpty())
-        m_systemTheme = fallbackTheme();
-
+        m_themeKey(1), m_supportsSvg(false), m_initialized(false)
+{
+}
+
+// We lazily initialize the loader to make static icons
+// work. Though we do not officially support this.
+void QIconLoader::ensureInitialized()
+{
+    if (!m_initialized) {
+        m_initialized = true;
+
+        Q_ASSERT(qApp);
+
+        m_systemTheme = qt_guiPlatformPlugin()->systemIconThemeName();
+        if (m_systemTheme.isEmpty())
+            m_systemTheme = fallbackTheme();
 #ifndef QT_NO_LIBRARY
-    QFactoryLoader iconFactoryLoader(QIconEngineFactoryInterfaceV2_iid,
-                                     QLatin1String("/iconengines"),
-                                     Qt::CaseInsensitive);
-    if (iconFactoryLoader.keys().contains(QLatin1String("svg")))
-        m_supportsSvg = true;
+    QFactoryLoader iconFactoryLoader(QIconEngineFactoryInterfaceV2_iid,
+                                     QLatin1String("/iconengines"),
+                                     Qt::CaseInsensitive);
+    if (iconFactoryLoader.keys().contains(QLatin1String("svg")))
+        m_supportsSvg = true;
 #endif //QT_NO_LIBRARY
+
 }
 
@@ -340 +351 @@
 void QIconLoaderEngine::ensureLoaded()
 {
+
+
+
     if (!(iconLoaderInstance()->themeKey() == m_key)) {
 

trunk/src/gui/image/qiconloader_p.h

@@ -170 +170 @@
     void updateSystemTheme();
     void invalidateKey() { m_themeKey++; }
+
 
 private:
@@ -177 +178 @@
     uint m_themeKey;
     bool m_supportsSvg;
+
 
     mutable QString m_userTheme;

trunk/src/gui/image/qimage.cpp

@@ -119 +119 @@
 }
 
-extern int qt_defaultDpiX();
-extern int qt_defaultDpiY();
+extern int qt_defaultDpiX();
+extern int qt_defaultDpiY();
 
 QBasicAtomicInt qimage_serial_number = Q_BASIC_ATOMIC_INITIALIZER(1);
@@ -5668 +5668 @@
     detach();
 
-    *this = convertToFormat(QImage::Format_ARGB32_Premultiplied);
+    QImage converted = convertToFormat(QImage::Format_ARGB32_Premultiplied);
+    if (!converted.isNull())
+        *this = converted;
+    else
+        return;
 
     // Slight optimization since alphachannels are returned as 8-bit grays.

trunk/src/gui/image/qimagepixmapcleanuphooks.cpp

@@ -97 +97 @@
 {
     QImagePixmapCleanupHooks *h = qt_image_and_pixmap_cleanup_hooks();
+
+
+
+
+
     for (int i = 0; i < h->pixmapModificationHooks.count(); ++i)
         h->pixmapModificationHooks[i](pmd);
@@ -107 +112 @@
 {
     QImagePixmapCleanupHooks *h = qt_image_and_pixmap_cleanup_hooks();
+
+
+
+
+
     for (int i = 0; i < h->pixmapDestructionHooks.count(); ++i)
         h->pixmapDestructionHooks[i](pmd);
@@ -123 +133 @@
 }
 
-void QImagePixmapCleanupHooks::enableCleanupHooks(const QPixmap &pixmap)
-{
-    enableCleanupHooks(const_cast<QPixmap &>(pixmap).data_ptr().data());
-}
 
 void QImagePixmapCleanupHooks::enableCleanupHooks(QPixmapData *pixmapData)
 {
     pixmapData->is_cached = true;
+
+
+
+
+
 }
 
@@ -138 +149 @@
 }
 
+
+
+
+
+
+
+
+
+
+
+
+
 QT_END_NAMESPACE

trunk/src/gui/image/qimagepixmapcleanuphooks_p.h

@@ -73 +73 @@
     static void enableCleanupHooks(QPixmapData *pixmapData);
 
+
+
+
     // Gets called when a pixmap data is about to be modified:
     void addPixmapDataModificationHook(_qt_pixmap_cleanup_hook_pmd);

trunk/src/gui/image/qimagereader.cpp

@@ -264 +264 @@
     }
 
-    if (!handler && !testFormat.isEmpty() && autoDetectImageFormat && !ignoresFormatAndExtension) {
+    if (!handler && !testFormat.isEmpty() && !ignoresFormatAndExtension) {
         // check if any plugin supports the format (they are not allowed to
         // read from the device yet).
         const qint64 pos = device ? device->pos() : 0;
-        for (int i = 0; i < keys.size(); ++i) {
-            if (i != suffixPluginIndex) {
-                QImageIOPlugin *plugin = qobject_cast<QImageIOPlugin *>(l->instance(keys.at(i)));
-                if (plugin && plugin->capabilities(device, testFormat) & QImageIOPlugin::CanRead) {
+
+        if (autoDetectImageFormat) {
+            for (int i = 0; i < keys.size(); ++i) {
+                if (i != suffixPluginIndex) {
+                    QImageIOPlugin *plugin = qobject_cast<QImageIOPlugin *>(l->instance(keys.at(i)));
+                    if (plugin && plugin->capabilities(device, testFormat) & QImageIOPlugin::CanRead) {
 #ifdef QIMAGEREADER_DEBUG
-                    qDebug() << "QImageReader::createReadHandler: the" << keys.at(i) << "plugin can read this format";
-#endif
-                    handler = plugin->create(device, testFormat);
-                    break;
+                        qDebug() << "QImageReader::createReadHandler: the" << keys.at(i) << "plugin can read this format";
+#endif
+                        handler = plugin->create(device, testFormat);
+                        break;
+                    }
                 }
+
+
+
+
+
+
+
+
             }
         }
@@ -283 +294 @@
             device->seek(pos);
     }
+
 #endif // QT_NO_LIBRARY
 

trunk/src/gui/image/qimagewriter.cpp

@@ -198 +198 @@
             QImageIOPlugin *plugin = qobject_cast<QImageIOPlugin *>(l->instance(keys.at(i)));
             if (plugin && (plugin->capabilities(device, testFormat) & QImageIOPlugin::CanWrite)) {
+
                 handler = plugin->create(device, testFormat);
                 break;

trunk/src/gui/image/qnativeimage.cpp

@@ -202 +202 @@
         ok = (xshminfo.shmaddr != (char*)-1);
         if (ok)
-            image = QImage((uchar *)xshmimg->data, width, height, systemFormat());
+            image = QImage((uchar *)xshmimg->data, width, height, );
     }
     xshminfo.readOnly = false;

trunk/src/gui/image/qpaintengine_pic.cpp

@@ -478 +478 @@
 }
 
-extern int qt_defaultDpi();
+extern int qt_defaultDpi();
 
 void QPicturePaintEngine::drawTextItem(const QPointF &p , const QTextItem &ti)

trunk/src/gui/image/qpicture.cpp

@@ -109 +109 @@
 static const quint16 mfhdr_maj = 11; // major version #
 static const quint16 mfhdr_min = 0; // minor version #
-extern int qt_defaultDpiX();
-extern int qt_defaultDpiY();
+extern int qt_defaultDpiX();
+extern int qt_defaultDpiY();
 
 /*!

trunk/src/gui/image/qpixmap.cpp

@@ -832 +832 @@
         return true;
 
-    bool ok;
-
-    if (data) {
-        ok = data->fromFile(fileName, format, flags);
-    } else {
-        QScopedPointer<QPixmapData> tmp(QPixmapData::create(0, 0, QPixmapData::PixmapType));
-        ok = tmp->fromFile(fileName, format, flags);
-        if (ok)
-            data = tmp.take();
-    }
-
-    if (ok)
+    QScopedPointer<QPixmapData> tmp(QPixmapData::create(0, 0, data ? data->type : QPixmapData::PixmapType));
+    if (tmp->fromFile(fileName, format, flags)) {
+        data = tmp.take();
         QPixmapCache::insert(key, *this);
-
-    return ok;
+        return true;
+    }
+
+    return false;
 }
 
@@ -1671 +1664 @@
 
     The hasAlphaChannel() returns true if the pixmap has a format that
-    respects the alpha channel, otherwise returns false, while the
-    hasAlpha() function returns true if the pixmap has an alpha
-    channel \e or a mask (otherwise false). The mask() function returns
-    the mask as a QBitmap object, which can be set using setMask().
+    respects the alpha channel, otherwise returns false. The hasAlpha(),
+    setMask() and mask() functions are legacy and should not be used.
+    They are potentially very slow.
 
     The createHeuristicMask() function creates and returns a 1-bpp
@@ -1760 +1752 @@
     Returns true if this pixmap has an alpha channel, \e or has a
     mask, otherwise returns false.
+
+
 
     \sa hasAlphaChannel(), mask()
@@ -2028 +2022 @@
     too. The mouse cursor is generally not grabbed.
 
-    Note on X11that if the given \a window doesn't have the same depth
+    Note on X11that if the given \a window doesn't have the same depth
     as the root window, and another window partially or entirely
     obscures the one you grab, you will \e not get pixels from the
@@ -2034 +2028 @@
     pixmap will be undefined and uninitialized.
 
+
+
+
+
     \warning In general, grabbing an area outside the screen is not
     safe. This depends on the underlying window system.

trunk/src/gui/image/qpixmap_raster.cpp

@@ -358 +358 @@
 }
 
-extern int qt_defaultDpiX();
-extern int qt_defaultDpiY();
+extern int qt_defaultDpiX();
+extern int qt_defaultDpiY();
 
 int QRasterPixmapData::metric(QPaintDevice::PaintDeviceMetric metric) const

trunk/src/gui/image/qpixmap_s60.cpp

@@ -338 +338 @@
         return QPixmap();
 
-    QScopedPointer<QS60PixmapData> data(new QS60PixmapData(QPixmapData::PixmapType));
+    QScopedPointer<QQPixmapData::PixmapType));
     data->fromNativeType(reinterpret_cast<void*>(bitmap), QPixmapData::FbsBitmap);
     QPixmap pixmap(data.take());
@@ -736 +736 @@
         return QPixmap();
 
-    QScopedPointer<QS60PixmapData> data(new QS60PixmapData(QPixmapData::PixmapType));
+    QScopedPointer<QQPixmapData::PixmapType));
     data->fromNativeType(reinterpret_cast<void*>(sgImage), QPixmapData::SgImage);
     QPixmap pixmap(data.take());

trunk/src/gui/image/qpixmap_x11.cpp

@@ -384 +384 @@
         // Will implicitly also check format and return quickly for opaque types...
         checked = true;
-        has = const_cast<QImage *>(image)->data_ptr()->checkForAlphaPixels();
+        has = const_cast<QImage *>(image)->data_ptr()->checkForAlphaPixels();
         return has;
     }

trunk/src/gui/image/qpixmapfilter.cpp

@@ -423 +423 @@
         return;
 
+
+
+
     QPixmapFilter *filter = painter->paintEngine() && painter->paintEngine()->isExtended() ?
         static_cast<QPaintEngineEx *>(painter->paintEngine())->pixmapFilter(type(), this) : 0;
@@ -775 +778 @@
 Q_GUI_EXPORT QImage qt_halfScaled(const QImage &source)
 {
+
+
+
     QImage srcImage = source;
 
@@ -867 +873 @@
 
     qreal scale = 1;
-    if (radius >= 4) {
+    if (radius >= 4) {
         blurImage = qt_halfScaled(blurImage);
         scale = 2;
@@ -893 +899 @@
 }
 
-bool qt_scaleForTransform(const QTransform &transform, qreal *scale);
+bool qt_scaleForTransform(const QTransform &transform, qreal *scale);
 
 /*!
@@ -902 +908 @@
     Q_D(const QPixmapBlurFilter);
     if (!painter->isActive())
+
+
+
         return;
 
@@ -1084 +1093 @@
 {
     Q_D(const QPixmapColorizeFilter);
+
+
+
+
     QPixmapFilter *filter = painter->paintEngine() && painter->paintEngine()->isExtended() ?
         static_cast<QPaintEngineEx *>(painter->paintEngine())->pixmapFilter(type(), this) : 0;
@@ -1314 +1327 @@
 {
     Q_D(const QPixmapDropShadowFilter);
+
+
+
+
     QPixmapFilter *filter = p->paintEngine() && p->paintEngine()->isExtended() ?
         static_cast<QPaintEngineEx *>(p->paintEngine())->pixmapFilter(type(), this) : 0;