Do not reply to this email. You can add comments to this bug at
https://bugzilla.mozilla.org/show_bug.cgi?id=467729

Karl Tomlinson (:karlt) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Attachment|review?([email protected] |review+
     #8564519 Flags|.net)                       |

--- Comment #34 from Karl Tomlinson (:karlt) <[email protected]> 
2015-02-15 18:26:59 PST ---
Comment on attachment 8564519
  --> https://bugzilla.mozilla.org/attachment.cgi?id=8564519
Add a PackageKit XPCOM API

>+  nsAutoArrayPtr<gchar*> packages(new gchar*[arrayLength + 1]);
>+  NS_ENSURE_TRUE(packages, NS_ERROR_OUT_OF_MEMORY);

No need for NS_ENSURE_TRUE here as |new| will not return null.  (It would throw
an exception, which would abort the program, if it failed.  The nothrow form of
new could be used, but I expect this won't be called with large arrays, and so
we can keep things simple here.)

-- 
Configure bugmail: https://bugzilla.mozilla.org/userprefs.cgi?tab=email

-------------------------------
Product/Component: Firefox :: General



------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
fonts-bugs mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/fonts-bugs
http://fonts.fedoraproject.org/

Reply via email to