605174 - Fix build warnings in dom/

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Mainly uninitialized and unused variables.

Comment 1

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v2

Merged to tip

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v2.1

Merged to tip.

Comment 3

[Ed Morley [:emorley]]

Attachment: Patch v3

Updated to tip & some new warning fixes added.

Peter, ping for review (the previous review has been open for 10 months) - thanks :-)

Comment 4

[Ed Morley [:emorley]]

Just spotted an #ifdef DEBUG instance that can be converted to DebugOnly in one of the dom/ipc/ContentParent.cpp hunks. Will make the change after first pass review.

Comment 5

[Peter Van der Beken [:peterv]]

Comment on attachment 557472 [details] [diff] [review]
Patch v3

Review of attachment 557472 [details] [diff] [review]:
-----------------------------------------------------------------

I don't think we should change for bogus warnings, rest looks ok.

::: dom/base/nsDOMClassInfo.cpp
@@ +7342,5 @@
>  static PRBool
>  GetBindingURL(Element *aElement, nsIDocument *aDocument,
>                nsCSSValue::URL **aResult)
>  {
> +  *aResult = nsnull;

Bogus warning, I don't see why we should change this.

@@ +8448,5 @@
>        return JS_TRUE;
>      }
>    }
>  
> +  nsHTMLDocument* doc = GetDocument(cx, obj);

Follow existing style, don't change this.

@@ +8450,5 @@
>    }
>  
> +  nsHTMLDocument* doc = GetDocument(cx, obj);
> +  nsISupports* result = NULL;
> +  nsWrapperCache* cache = NULL;

Bogus warning, I don't see why we should change this.

::: dom/base/nsDOMException.cpp
@@ +87,5 @@
>      NS_WARNING("Trying to create an exception for the wrong error module."); \
>      return NS_ERROR_FAILURE;                                                 \
>    }                                                                          \
> +  const char* name = NULL;                                                   \
> +  const char* message = NULL;                                                \

Fix this in NSResultToNameAndMessage.

::: dom/base/nsGlobalWindow.cpp
@@ +9208,5 @@
>      ++gRunningTimeoutDepth;
>      ++mTimeoutFiringDepth;
>  
>      PRBool trackNestingLevel = !timeout->mIsInterval;
> +    PRUint32 nestingLevel = 0;

Bogus warning, I don't see why we should change this.

::: dom/base/nsScriptNameSpaceManager.cpp
@@ +294,5 @@
>    nsCAutoString category_entry;
>    const char* if_name;
>    nsCOMPtr<nsISupports> entry;
>    nsCOMPtr<nsIInterfaceInfo> if_info;
> +  PRBool found_old = PR_FALSE, dom_prefix;

Bogus warning, I don't see why we should change this.

::: dom/indexedDB/LazyIdleThread.cpp
@@ +512,5 @@
>  NS_IMETHODIMP
>  LazyIdleThread::AfterProcessNextEvent(nsIThreadInternal* /* aThread */,
>                                        PRUint32 /* aRecursionDepth */)
>  {
> +  bool shouldNotifyIdle = false;

Bogus warning, I don't see why we should change this.

::: dom/ipc/ContentParent.cpp
@@ +488,5 @@
>  
>      nsCOMPtr<nsISimpleEnumerator> enumerator;
> +#ifdef DEBUG
> +    nsresult rv = 
> +#endif

DebugOnly<...>?

::: dom/plugins/ipc/PluginModuleParent.cpp
@@ +618,5 @@
>  
>  PluginIdentifierParent*
>  PluginModuleParent::GetIdentifierForNPIdentifier(NPP npp, NPIdentifier aIdentifier)
>  {
> +    PluginIdentifierParent* ident = NULL;

Bogus warning, I don't see why we should change this.

Comment 6

[Ed Morley [:emorley]]

Attachment: Patch for checkin

Changes made for review comments, carrying forwards r+.

Builds fine locally, sent to try:
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=db82af19db7c

Comment 7

[Ed Morley [:emorley]]

Attachment: Patch for checkin v2

As comment 6 patch, but with dom/plugins/ipc/PluginModuleChild.cpp @@ -254,18 +252,17 @@ PluginModuleChild::Init(const std::strin hunk added to prevent OSX build error + the nsIDOMXULMultSelectCntrlEl.idl hunk removed as it caused win Moth and crashtest failures.

Carrying forwards r+.

Comment 8

[Ed Morley [:emorley]]

https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=84ba319a114a

https://hg.mozilla.org/integration/mozilla-inbound/rev/d4826f400c89

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/d4826f400c89