1035747 - xpcom/base/nsMemoryReporterManager.cpp:540:1: error: static_assert failed "Reference-counted class VsizeMaxContiguousReporter should not have a public destructor...

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Jan Beich]

VsizeMaxContiguousReporter and PrivateReporter are only implemented on Windows and FreeBSD. MOZ_IS_DESTRUCTIBLE is disabled on Windows. I wonder if this'll change with clang-cl.

In file included from xpcom/base/Unified_cpp_xpcom_base0.cpp:184:
xpcom/base/nsMemoryReporterManager.cpp:540:1: error:
      static_assert failed "Reference-counted class VsizeMaxContiguousReporter
      should not have a public destructor. Try to make this class's destructor
      non-public. If that is really not possible, you can whitelist this class by
      providing a HasDangerousPublicDestructor specialization for it."
NS_IMPL_ISUPPORTS(VsizeMaxContiguousReporter, nsIMemoryReporter)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dist/include/nsISupportsImpl.h:1023:3: note: expanded from macro
      'NS_IMPL_ISUPPORTS'
  NS_IMPL_ADDREF(aClass)                                                      \
  ^~~~~~~~~~~~~~~~~~~~~~
dist/include/nsISupportsImpl.h:559:3: note: expanded from macro
      'NS_IMPL_ADDREF'
  MOZ_ASSERT_TYPE_OK_FOR_REFCOUNTING(_class)                                  \
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dist/include/nsISupportsImpl.h:91:3: note: expanded from macro
      'MOZ_ASSERT_TYPE_OK_FOR_REFCOUNTING'
  static_assert(!MOZ_IS_DESTRUCTIBLE(X) || \
  ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from xpcom/base/Unified_cpp_xpcom_base0.cpp:184:
xpcom/base/nsMemoryReporterManager.cpp:562:1: error:
      static_assert failed "Reference-counted class PrivateReporter should not have
      a public destructor. Try to make this class's destructor non-public. If that
      is really not possible, you can whitelist this class by providing a
      HasDangerousPublicDestructor specialization for it."
NS_IMPL_ISUPPORTS(PrivateReporter, nsIMemoryReporter)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dist/include/nsISupportsImpl.h:1023:3: note: expanded from macro
      'NS_IMPL_ISUPPORTS'
  NS_IMPL_ADDREF(aClass)                                                      \
  ^~~~~~~~~~~~~~~~~~~~~~
dist/include/nsISupportsImpl.h:559:3: note: expanded from macro
      'NS_IMPL_ADDREF'
  MOZ_ASSERT_TYPE_OK_FOR_REFCOUNTING(_class)                                  \
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dist/include/nsISupportsImpl.h:91:3: note: expanded from macro
      'MOZ_ASSERT_TYPE_OK_FOR_REFCOUNTING'
  static_assert(!MOZ_IS_DESTRUCTIBLE(X) || \
  ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

Comment 1

[Jan Beich]

Attachment: fix

Continue similar change in the file. It fixes my build and doesn't crash about:memory.

https://hg.mozilla.org/mozilla-central/diff/bb961917a75c/xpcom/base/nsMemoryReporterManager.cpp

Comment 2

[Jan Beich]

https://tbpl.mozilla.org/?tree=Try&rev=f346e4538ab4

Comment 3

[Benoit Jacob [:bjacob] (mostly away)]

Yes, clanng-cl will bring MOZ_IS_DESTRUCTIBLE to Windows.

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4a0eebc295b3

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4a0eebc295b3