1503839 - Port Bug 1503207 - Remove nsWeakPtr.h

Status: RESOLVED FIXED

(MailNews Core :: General, task)

Description

[Jorg K (CEST = GMT+2)]

Looks like we need to do this:

-#include "nsWeakReference.h"
-#include "nsWeakPtr.h"
+#include "nsIWeakReferenceUtils.h"

Comment 1

[Jorg K (CEST = GMT+2)]

Attachment: 1503839-weak.patch

This compiles so far.

Comment 2

[Jorg K (CEST = GMT+2)]

Attachment: 1503839-weak.patch (v2)

Removed a few necessary includes. Aceman can tell us whether more could have been removed.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/de28835aa5f1
Port bug 1503207: replace include of nsWeakPtr.h with nsIWeakReferenceUtils.h, use nsWeakPtr. rs=bustage-fix

Comment 4

[:aceman]

Attachment: 1503839.patch weak2

Found some more possible clean up.

Comment 5

[Jorg K (CEST = GMT+2)]

Comment on attachment 9021955 [details] [diff] [review]
1503839.patch weak2

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

I can do the adjustment before landing.

::: ldap/xpcom/public/nsILDAPMessageListener.idl
@@ +13,1 @@
>   * A callback interface to be implemented by any objects that want to 

Let's lose this hunk. We can take care of white-space in another bug.

::: mailnews/addrbook/src/nsAbLDAPDirectoryQuery.h
@@ +9,5 @@
>  #include "nsIAbDirectoryQuery.h"
>  #include "nsILDAPConnection.h"
>  #include "nsILDAPMessageListener.h"
>  #include "nsILDAPURL.h"
> +#include "nsWeakReference.h" // this is needed, but why?

Let's lose this comment, or research it and document it.

Comment 6

[:aceman]

I did research it, but don't know why it is needed. The compiler claims the nsILDAPCOnnection class in incomplete. But I do not see why, it does not have any weak references that are obviously seen.
So I add the comment so that somebody may be more successful in researching it further. But you can drop it if you want.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/3cb2a9dc80d3
clean up some more ns*WeakReference*.h includes. r=jorgk