[Webkit-unassigned] [Bug 278275] New: Build fix: Move Inspector::RemoteInspector destructor to source file for upstream clang

[bugzilla-daemon at webkit.org]

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

            Bug ID: 278275
           Summary: Build fix: Move Inspector::RemoteInspector destructor
                    to source file for upstream clang
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: JavaScriptCore
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ddkilzer at webkit.org

Move Inspector::RemoteInspector destructor to source file to fix upstream clang build.

Example build error:

In file included from WebKitBuild/Release/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource76.cpp:1:
In file included from ./inspector/JSGlobalObjectConsoleClient.cpp:27:
In file included from ./inspector/JSGlobalObjectConsoleClient.h:28:
In file included from Source/JavaScriptCore/runtime/ConsoleClient.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CanMakeWeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CompactRefPtrTuple.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/RefPtr.h:28:
WebKitBuild/Release/usr/local/include/wtf/Ref.h:62:16: error: member access into incomplete type 'Inspector::RemoteConnectionToTarget'
   62 |             ptr->deref();
      |                ^
In file included from WebKitBuild/Release/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource76.cpp:1:
In file included from ./inspector/JSGlobalObjectConsoleClient.cpp:27:
In file included from ./inspector/JSGlobalObjectConsoleClient.h:28:
In file included from Source/JavaScriptCore/runtime/ConsoleClient.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CanMakeWeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CompactRefPtrTuple.h:30:
WebKitBuild/Release/usr/local/include/wtf/RefPtr.h:60:47: note: in instantiation of member function 'WTF::DefaultRefDerefTraits<Inspector::RemoteConnectionToTarget>::derefIfNotNull' requested here
   60 |     ALWAYS_INLINE ~RefPtr() { RefDerefTraits::derefIfNotNull(PtrTraits::exchange(m_ptr, nullptr)); }
      |                                               ^
In file included from WebKitBuild/Release/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource76.cpp:1:
In file included from ./inspector/JSGlobalObjectConsoleClient.cpp:27:
In file included from ./inspector/JSGlobalObjectConsoleClient.h:28:
In file included from Source/JavaScriptCore/runtime/ConsoleClient.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CanMakeWeakPtr.h:31:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtrFactory.h:31:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakRef.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/HashTraits.h:27:
WebKitBuild/Release/usr/local/include/wtf/KeyValuePair.h:33:8: note: in instantiation of member function 'WTF::RefPtr<Inspector::RemoteConnectionToTarget>::~RefPtr' requested here
   33 | struct KeyValuePair {
      |        ^
In file included from WebKitBuild/Release/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource76.cpp:1:
In file included from ./inspector/JSGlobalObjectConsoleClient.cpp:27:
In file included from ./inspector/JSGlobalObjectConsoleClient.h:28:
In file included from Source/JavaScriptCore/runtime/ConsoleClient.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CanMakeWeakPtr.h:31:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtrFactory.h:31:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakRef.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/SingleThreadIntegralWrapper.h:28:
In file included from WebKitBuild/Release/usr/local/include/wtf/Threading.h:39:
In file included from WebKitBuild/Release/usr/local/include/wtf/HashMap.h:25:
WebKitBuild/Release/usr/local/include/wtf/HashTable.h:425:17: note: in instantiation of member function 'WTF::HashTable<unsigned int, WTF::KeyValuePair<unsigned int, WTF::RefPtr<Inspector::RemoteConnectionToTarget>>, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned int, WTF::RefPtr<Inspector::RemoteConnectionToTarget>>>, WTF::DefaultHash<unsigned int>, WTF::HashMap<unsigned int, WTF::RefPtr<Inspector::RemoteConnectionToTarget>>::KeyValuePairTraits, WTF::HashTraits<unsigned int>>::deallocateTable' requested here
  425 |                 deallocateTable(m_table);
      |                 ^
In file included from WebKitBuild/Release/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource76.cpp:1:
In file included from ./inspector/JSGlobalObjectConsoleClient.cpp:27:
In file included from ./inspector/JSGlobalObjectConsoleClient.h:28:
In file included from Source/JavaScriptCore/runtime/ConsoleClient.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtr.h:29:
In file included from WebKitBuild/Release/usr/local/include/wtf/CanMakeWeakPtr.h:31:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakPtrFactory.h:31:
In file included from WebKitBuild/Release/usr/local/include/wtf/WeakRef.h:30:
In file included from WebKitBuild/Release/usr/local/include/wtf/SingleThreadIntegralWrapper.h:28:
In file included from WebKitBuild/Release/usr/local/include/wtf/Threading.h:39:
WebKitBuild/Release/usr/local/include/wtf/HashMap.h:35:7: note: in instantiation of member function 'WTF::HashTable<unsigned int, WTF::KeyValuePair<unsigned int, WTF::RefPtr<Inspector::RemoteConnectionToTarget>>, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned int, WTF::RefPtr<Inspector::RemoteConnectionToTarget>>>, WTF::DefaultHash<unsigned int>, WTF::HashMap<unsigned int, WTF::RefPtr<Inspector::RemoteConnectionToTarget>>::KeyValuePairTraits, WTF::HashTraits<unsigned int>>::~HashTable' requested here
   35 | class HashMap final {
      |       ^
In file included from WebKitBuild/Release/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource76.cpp:3:
In file included from ./inspector/JSGlobalObjectInspectorController.cpp:56:
Source/JavaScriptCore/inspector/remote/RemoteInspector.h:70:7: note: forward declaration of 'Inspector::RemoteConnectionToTarget'
   70 | class RemoteConnectionToTarget;
      |       ^
1 error generated.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20240817/e2c6bccd/attachment.htm>