[Webkit-unassigned] [Bug 56198] New: WebCore/loader/appcache/ApplicationCacheGroup fails to compile on S10/SS12

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=56198

           Summary: WebCore/loader/appcache/ApplicationCacheGroup fails to
                    compile on S10/SS12
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Other
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Page Loading
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: bentaylor.solx86 at gmail.com


qt-everywhere-opensource-src-4.7.1/src/3rdparty/webkit/WebCore/loader/appcache/ApplicationCacheGroup.cpp

Solaris 10, Sun Studio 12 (SS12) C++ compiler


   767          ASSERT(m_cacheBeingUpdated);
   768          if (m_manifestResource)
   769              m_cacheBeingUpdated->setManifestResource(m_manifestResource.release());
   770          else {
   771              // We can get here as a result of retrying the Complete step, following
   772              // a failure of the cache storage to save the newest cache due to hitting
   773              // the maximum size. In such a case, m_manifestResource may be 0, as
   774              // the manifest was already set on the newest cache object.
   775              ASSERT(cacheStorage().isMaximumSizeReached() && m_calledReachedMaxAppCacheSize);
   776          }
   777
   778          RefPtr<ApplicationCache> oldNewestCache = (m_newestCache == m_cacheBeingUpdated) ? 0 : m_newestCache;
   779

"loader/appcache/ApplicationCache.cpp", line 778: Error: Ambiguous "?:" expression, second operand of type "int" and third operand of type "WTF::RefPtr<WebCore::ApplicationCache>" can be converted to one another.

The Sun Studio 12 C++ compiler does not like having to accept 0 untyped when the other conditional is typed.

The following patch makes the compilation error go away.

--- qt-everywhere-opensource-src-4.7.1/src/3rdparty/webkit/WebCore/loader/appcache/ApplicationCacheGroup.cpp.ORIG       2011-03-09 15:16:41.124515331
+0000
+++ qt-everywhere-opensource-src-4.7.1/src/3rdparty/webkit/WebCore/loader/appcache/ApplicationCacheGroup.cpp    2011-03-09 14:50:54.718120159 +0000
@@ -775,7 +775,7 @@
             ASSERT(cacheStorage().isMaximumSizeReached() && m_calledReachedMaxAppCacheSize);
         }

-        RefPtr<ApplicationCache> oldNewestCache = (m_newestCache == m_cacheBeingUpdated) ? 0 : m_newestCache;
+        RefPtr<ApplicationCache> oldNewestCache = (m_newestCache == m_cacheBeingUpdated) ? RefPtr<ApplicationCache>() : m_newestCache;

         setNewestCache(m_cacheBeingUpdated.release());
         if (cacheStorage().storeNewestCache(this)) {

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.