Bug 1007741 - Fix -Wunused-function warning in ipc/glue/ and mark ipc/ subdirectories as FAIL_ON_WARNINGS. r=bsmedberg
authorChris Peterson <cpeterson@mozilla.com>
Wed, 07 May 2014 23:55:30 -0700
changeset 182297 0759f8afcf557c8f21fcc71e84d6e73a91ec49da
parent 182296 c80cc079cbc874dd1a58ab8f5765e97bce8f3daf
child 182298 10bf5fc4bfc3b839a37925a440f1b67363860025
push id26755
push usercbook@mozilla.com
push dateFri, 09 May 2014 11:54:47 +0000
treeherdermozilla-central@5700a88f895f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbsmedberg
bugs1007741
milestone32.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 1007741 - Fix -Wunused-function warning in ipc/glue/ and mark ipc/ subdirectories as FAIL_ON_WARNINGS. r=bsmedberg
ipc/app/moz.build
ipc/dbus/moz.build
ipc/glue/BackgroundImpl.cpp
ipc/glue/moz.build
ipc/netd/moz.build
ipc/nfc/moz.build
ipc/ril/moz.build
--- a/ipc/app/moz.build
+++ b/ipc/app/moz.build
@@ -42,8 +42,10 @@ LDFLAGS += [CONFIG['MOZ_ALLOW_HEAP_EXECU
 # As we use the CRT heap, the default size is too large and wastes VM.
 #
 # The default heap size is 1MB on Win32.
 # The heap will grow if need be.
 #
 # Set it to 256k.  See bug 127069.
 if CONFIG['OS_ARCH'] == 'WINNT' and not CONFIG['GNU_CC']:
     LDFLAGS += ['/HEAP:0x40000']
+
+FAIL_ON_WARNINGS = True
--- a/ipc/dbus/moz.build
+++ b/ipc/dbus/moz.build
@@ -12,8 +12,10 @@ EXPORTS.mozilla.ipc += [
 SOURCES += [
     'DBusUtils.cpp',
     'RawDBusConnection.cpp',
 ]
 
 include('/ipc/chromium/chromium-config.mozbuild')
 
 FINAL_LIBRARY = 'xul'
+
+FAIL_ON_WARNINGS = True
--- a/ipc/glue/BackgroundImpl.cpp
+++ b/ipc/glue/BackgroundImpl.cpp
@@ -66,21 +66,23 @@ namespace {
 bool
 IsMainProcess()
 {
   static const bool isMainProcess =
     XRE_GetProcessType() == GeckoProcessType_Default;
   return isMainProcess;
 }
 
+#ifdef DEBUG
 bool
 IsChildProcess()
 {
   return !IsMainProcess();
 }
+#endif
 
 void
 AssertIsInMainProcess()
 {
   MOZ_ASSERT(IsMainProcess());
 }
 
 void
--- a/ipc/glue/moz.build
+++ b/ipc/glue/moz.build
@@ -145,8 +145,10 @@ for var in ('MOZ_CHILD_PROCESS_NAME', 'M
 LOCAL_INCLUDES += [
     '/toolkit/crashreporter',
 ]
 
 if CONFIG['MOZ_CONTENT_SANDBOX'] and CONFIG['OS_ARCH'] == 'WINNT':
     LOCAL_INCLUDES += [
         '/security/sandbox/win/src/sandboxbroker',
     ]
+
+FAIL_ON_WARNINGS = True
--- a/ipc/netd/moz.build
+++ b/ipc/netd/moz.build
@@ -10,8 +10,10 @@ EXPORTS.mozilla.ipc += [
 
 SOURCES += [
     'Netd.cpp',
 ]
 
 include('/ipc/chromium/chromium-config.mozbuild')
 
 FINAL_LIBRARY = 'xul'
+
+FAIL_ON_WARNINGS = True
--- a/ipc/nfc/moz.build
+++ b/ipc/nfc/moz.build
@@ -11,8 +11,10 @@ if CONFIG['MOZ_NFC']:
         'Nfc.cpp',
     ]
 
 EXPORT_LIBRARY = True
 
 include('/ipc/chromium/chromium-config.mozbuild')
 
 FINAL_LIBRARY = 'xul'
+
+FAIL_ON_WARNINGS = True
--- a/ipc/ril/moz.build
+++ b/ipc/ril/moz.build
@@ -10,8 +10,10 @@ EXPORTS.mozilla.ipc += [
 
 SOURCES += [
     'Ril.cpp',
 ]
 
 include('/ipc/chromium/chromium-config.mozbuild')
 
 FINAL_LIBRARY = 'xul'
+
+FAIL_ON_WARNINGS = True