Bug 1142544 - delete now-unnecessary |extern "C++"| block from nsISupportsUtils.h; r=mccr8
authorNathan Froyd <froydnj@mozilla.com>
Thu, 12 Mar 2015 11:24:53 -0400
changeset 233560 af8b6c7b05f8a3ce7898a83d788a080b0c0466de
parent 233559 27533222749d6f1787dcf4b385285287338a1b6f
child 233561 f0f3cba361d1cc2ae315b440ba9c7fe05275eff4
push id28417
push userryanvm@gmail.com
push dateFri, 13 Mar 2015 19:52:44 +0000
treeherdermozilla-central@977add19414a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmccr8
bugs1142544
milestone39.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1142544 - delete now-unnecessary |extern "C++"| block from nsISupportsUtils.h; r=mccr8 We have blatant C++ code later on in the file; whatever C file(s) were including nsISupportsUtils.h are now a distant memory.
xpcom/glue/nsISupportsUtils.h
--- a/xpcom/glue/nsISupportsUtils.h
+++ b/xpcom/glue/nsISupportsUtils.h
@@ -26,34 +26,27 @@
  * because NS_ADDREF (when tracing) may require an ambiguous cast
  * from the pointers primary type to nsISupports. This macro sidesteps
  * that entire problem.
  */
 #define NS_ADDREF_THIS() \
   AddRef()
 
 
-extern "C++" {
-// ...because some one is accidentally including this file inside
-// an |extern "C"|
-
-
 // Making this a |inline| |template| allows |aExpr| to be evaluated only once,
 // yet still denies you the ability to |AddRef()| an |nsCOMPtr|.
 template<class T>
 inline void
 ns_if_addref(T aExpr)
 {
   if (aExpr) {
     aExpr->AddRef();
   }
 }
 
-} /* extern "C++" */
-
 /**
  * Macro for adding a reference to an interface that checks for nullptr.
  * @param _expr The interface pointer.
  */
 #define NS_IF_ADDREF(_expr) ns_if_addref(_expr)
 
 /*
  * Given these declarations, it explicitly OK and efficient to end a `getter' with: