[Bug 732572]

Karlt 732572 at bugs.launchpad.net
Tue Nov 22 03:41:51 UTC 2011


Comment on attachment 575899
Bug 635918 Part 1 - Make nsISound::Play use libcanberra on Linux rather than esound (v3)

I didn't know the nsLocalFile implementation was thread safe, but yay it is:
http://mxr.mozilla.org/mozilla-central/source/xpcom/io/nsLocalFileUnix.cpp#287

>+    nsILocalFile* forget() {
>+        nsILocalFile *tmp;
>+        mFile.forget(&tmp);
>+        return tmp;
>+    }

Usually we have already_AddRefed<T> return values for forget() methods in
scoped refcounting systems so this is a bit different if the result is ever
assigned to another similar auto-refcount object.  The return value is in fact
not used, so use "mFile.forget()" to simplify things and either return its
result directly or make the function return type void.

>+    operator void*() { return mFile; }

Making this "operator nsILocalFile*()" would be more explicit.

-- 
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to Mozilla.
https://bugs.launchpad.net/bugs/732572

Title:
  New Mail Notification Sound does not play in Natty

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/732572/+subscriptions




More information about the Ubuntu-mozillateam-bugs mailing list