[XPCOM] Fix the build to be silent by disabling errors originating from
authorKalle Vahlman <kalle.vahlman@movial.com>
Tue, 11 Nov 2008 12:05:00 +0000 (14:05 +0200)
committerKalle Vahlman <kalle.vahlman@movial.com>
Tue, 11 Nov 2008 12:05:00 +0000 (14:05 +0200)
external headers (TODO: check if those are something to worry about) and
fixing the issues that previously were masked by the insane warning
bombardment...

xpcom-dbusservice/DBusMarshaling.cpp
xpcom-dbusservice/DBusMethod.cpp
xpcom-dbusservice/DBusMethod.h
xpcom-dbusservice/DBusService.cpp
xpcom-dbusservice/DBusService.h
xpcom-dbusservice/DBusSignal.cpp
xpcom-dbusservice/DBusSignal.h
xpcom-dbusservice/build.mk

index 71e8f3d..ff7d4cc 100644 (file)
@@ -105,7 +105,6 @@ getSignatureFromJSValue(JSContext *cx, jsval *aValue, nsCString &aResult)
     {
         nsISupports* supports;
         JSClass* clazz;
-        JSObject* parent;
         JSObject* glob = JSVAL_TO_OBJECT(*aValue);
 
         clazz = JS_GET_CLASS(cx, JS_GetParent(cx, glob));
@@ -533,7 +532,7 @@ addVariantToIter(JSContext* cx, nsIVariant *aVariant, DBusMessageIter *aIter, DB
                  * nsIVariants directly, but no API to that effect seems to be
                  * available...
                  */
-                for (int i = 0; i < count; i++)
+                for (PRUint32 i = 0; i < count; i++)
                 {
                     nsCOMPtr<nsIWritableVariant> variant = do_CreateInstance("@mozilla.org/variant;1");
                     
@@ -727,7 +726,7 @@ void addArrayDataToIter(JSContext* cx, void *data_ptr, PRUint32 start, PRUint32
     switch (type)
     {
 #define ADD_DATA \
-            for (int i = start; i < count; i++) \
+            for (PRUint32 i = start; i < count; i++) \
                 dbus_message_iter_append_basic(aIter, aDBusType, data+i)
         case nsIDataType::VTYPE_INT8:
         case nsIDataType::VTYPE_UINT8:
@@ -766,7 +765,7 @@ void addArrayDataToIter(JSContext* cx, void *data_ptr, PRUint32 start, PRUint32
         case nsIDataType::VTYPE_WCHAR_STR:
         {
             PRUnichar **data = (PRUnichar **)data_ptr;
-            for (int i = start; i < count; i++)
+            for (PRUint32 i = start; i < count; i++)
             {
                 const char *val_ptr;
                 nsCAutoString val = NS_ConvertUTF16toUTF8(data[i]);
@@ -784,7 +783,7 @@ void addArrayDataToIter(JSContext* cx, void *data_ptr, PRUint32 start, PRUint32
         case nsIDataType::VTYPE_INTERFACE_IS:
         {
             nsISupports **data = (nsISupports **)data_ptr;
-            for (int i = start; i < count; i++)
+            for (PRUint32 i = start; i < count; i++)
             {
                 nsCOMPtr<nsIVariant> data_variant = do_QueryInterface(data[i]);
                 if (data_variant)
@@ -1086,7 +1085,9 @@ already_AddRefed<nsIWritableVariant> getVariantFromIter(JSContext* cx, DBusMessa
 
                 // Convert to variant and return
                 nsIVariant *var = nsnull;
-                nsresult rs = xpc->JSToVariant(cx, OBJECT_TO_JSVAL(obj), &var);
+                rv = xpc->JSToVariant(cx, OBJECT_TO_JSVAL(obj), &var);
+                if(NS_FAILED(rv))
+                    return nsnull;
                 variant->SetFromVariant(var);
                 var = nsnull;
                 NS_ADDREF(retval = variant);
@@ -1107,7 +1108,7 @@ already_AddRefed<nsIWritableVariant> getVariantFromIter(JSContext* cx, DBusMessa
                 BDBLOG(("    array: %d items\n", item_count));
 
                 nsIVariant **item_array = new nsIVariant*[item_count];
-                for (int i = 0; i < item_count; i++)
+                for (PRUint32 i = 0; i < item_count; i++)
                 {
                     nsCOMPtr<nsIVariant> item = do_QueryElementAt(items, i);
                     item_array[i] = item;
@@ -1117,7 +1118,7 @@ already_AddRefed<nsIWritableVariant> getVariantFromIter(JSContext* cx, DBusMessa
                                     &NS_GET_IID(nsIVariant),
                                     item_count,
                                     item_array);
-                for (int i = 0; i < item_count; i++)
+                for (PRUint32 i = 0; i < item_count; i++)
                     NS_RELEASE(item_array[i]);
                 delete[] item_array;
             }
index 01d9079..c121f16 100644 (file)
@@ -168,7 +168,6 @@ static void
 DoCallBack(DBusMethod *aMethod, DBusMessage *aReply)
 {
     DBusMessageIter iter;
-    int current_type;
     nsCOMPtr<nsIMutableArray> reply_args;
     nsCOMPtr<IDBusMethodCallback> callback;
 
@@ -253,7 +252,7 @@ DBusMethod::DoCall(nsIVariant **aArgs, PRUint32 aCount)
     {
         // FIXME - is it necessary to clear the string?
         signature.Assign("");
-        for (int i = 0; i < aCount; i++)
+        for (PRUint32 i = 0; i < aCount; i++)
         {
             // no method signature specified, guess argument types
             nsCOMPtr<nsIVariant> data = aArgs[i];
index c93ceed..0c0f877 100644 (file)
@@ -42,7 +42,7 @@
     0x2832f621, \
     0xad9b, \
     0x4034, \
-    0x91, 0x0b, 0xcd, 0x8e, 0xea, 0xdf, 0x5c, 0x42 \
+    { 0x91, 0x0b, 0xcd, 0x8e, 0xea, 0xdf, 0x5c, 0x42 } \
 }
 
 class DBusMethod : public IDBusMethod
@@ -68,8 +68,8 @@ protected:
     PRUint32    mBusType;
     nsCString   mDestination;
     nsCString   mObject;
-    nsCString   mInterface;
     nsCString   mMethod;
+    nsCString   mInterface;
     nsCString   mSignature;
     PRBool      mAsync;
     IDBusMethodCallback *mCallback;
index 4e53aa2..3191e1a 100644 (file)
@@ -246,7 +246,7 @@ DBusHandlerResult _signal_filter(DBusConnection *connection,
         BDBLOG(("  got observerList\n"));
         DBusService *service = DBusService::GetSingleton();
         
-        for (PRInt32 i = 0; i < observerList->Length(); ++i) {
+        for (PRUint32 i = 0; i < observerList->Length(); ++i) {
             nsCAutoString t;
             nsCOMPtr<IDBusSignal> signal = do_QueryReferent((*observerList)[i]);
             signal->GetInterfaceName(t);
@@ -285,7 +285,7 @@ DBusHandlerResult _signal_filter(DBusConnection *connection,
             /* arguments are packed as an array into an nsIVariant */
             nsIVariant **callback_args = new nsIVariant*[arg_items];
             nsCOMPtr<nsIWritableVariant> args = do_CreateInstance("@mozilla.org/variant;1");
-            for (int i = 0; i < arg_items; i++)
+            for (PRUint32 i = 0; i < arg_items; i++)
             {
                 nsCOMPtr<nsIVariant> arg = do_QueryElementAt(args_array, i);
                 callback_args[i] = arg;
@@ -295,7 +295,7 @@ DBusHandlerResult _signal_filter(DBusConnection *connection,
                              &NS_GET_IID(nsIVariant),
                              arg_items,
                              callback_args);
-            for (int i = 0; i < arg_items; i++)
+            for (PRUint32 i = 0; i < arg_items; i++)
                 NS_RELEASE(callback_args[i]);
             delete[] callback_args;
 
@@ -456,7 +456,7 @@ void DBusService::RemoveSignalObserver(IDBusSignal *aSignal)
         BDBLOG(("  got observerList\n"));
         nsCOMPtr<nsISupportsWeakReference> weakRefable = do_QueryInterface(aSignal);
         nsWeakPtr weakPtr = getter_AddRefs(NS_GetWeakReference(weakRefable));
-        for (PRInt32 i = 0; i < observerList->Length(); ++i) {
+        for (PRUint32 i = 0; i < observerList->Length(); ++i) {
             nsCAutoString t;
             nsCAutoString ob;
             nsCOMPtr<IDBusSignal> signal = do_QueryReferent((*observerList)[i]);
@@ -469,7 +469,7 @@ void DBusService::RemoveSignalObserver(IDBusSignal *aSignal)
         }
         BDBLOG(("  call observerList->RemoveElement\n"));
         observerList->RemoveElement(weakPtr);
-        for (PRInt32 i = 0; i < observerList->Length(); ++i) {
+        for (PRUint32 i = 0; i < observerList->Length(); ++i) {
             nsCAutoString t;
             nsCAutoString ob;
             nsCOMPtr<IDBusSignal> signal = do_QueryReferent((*observerList)[i]);
index 83647bb..6761133 100644 (file)
@@ -46,7 +46,7 @@ typedef PRUint8 uint8; /* otherwise pldhash.h:199 is sad */
     0xe3b49db1, \
     0x5754, \
     0x4330, \
-    0x92, 0xcd, 0xab, 0xe8, 0xf7, 0xea, 0x54, 0x3d \
+    { 0x92, 0xcd, 0xab, 0xe8, 0xf7, 0xea, 0x54, 0x3d } \
 }
 
 class DBusService : public IDBusService
index fa57679..76bc249 100644 (file)
@@ -127,6 +127,7 @@ DBusSignal::SetEnabled(PRBool aEnabled)
 
     mEnabled = aEnabled;
 
+    return NS_OK;
 }
 
 NS_IMETHODIMP
index 999b977..4655c8b 100644 (file)
@@ -41,7 +41,7 @@
     0xde515b88, \
     0xb8a0, \
     0x416e, \
-    0xb4, 0x38, 0x52, 0x4e, 0xf7, 0x96, 0xfb, 0x13 \
+    { 0xb4, 0x38, 0x52, 0x4e, 0xf7, 0x96, 0xfb, 0x13 } \
 }
 
 class DBusSignal : public IDBusSignal, public nsSupportsWeakReference
index 08b14f4..e334f85 100644 (file)
@@ -5,7 +5,7 @@ PKGS    = dbus-glib-1
 
 # These are wild guesses, but should be close
 GECKO_PREFIX = $(shell pkg-config --variable=prefix mozilla-plugin)
-GECKO_COMPONENT_DIR := $(shell ls -d $(GECKO_PREFIX)/lib/xulrunner-[0-9]* | head -1)/components
+GECKO_COMPONENT_DIR := $(shell ls -d $(GECKO_PREFIX)/lib/xulrunner-[0-9]* 2>/dev/null | head -1)/components
 GECKO_SDK_PATH := $(shell pkg-config --variable=sdkdir mozilla-plugin)
 XPIDL  = $(GECKO_SDK_PATH)/bin/xpidl
 PLUGINDIR := $(GECKO_COMPONENT_DIR)
@@ -18,7 +18,8 @@ CFLAGS        += -I$(GECKO_SDK_PATH)/include \
           -I$(GECKO_SDK_PATH)/include/xpconnect \
           -I$(GECKO_SDK_PATH)/include/js \
           -I$(O)/include \
-          -DNO_NSPR_10_SUPPORT
+          -DNO_NSPR_10_SUPPORT \
+          -Wno-non-virtual-dtor
 
 CFLAGS += -fno-rtti -fno-exceptions -fshort-wchar