[Bug 467729] RFE: Add font autoinstallation support

[Bugzilla@Mozilla]

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) <karlt at mozbugz.karlt.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Attachment|review?(karlt at mozbugz.karlt |review+
     #8564519 Flags|.net)                       |

--- Comment #34 from Karl Tomlinson (:karlt) <karlt at mozbugz.karlt.net> 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.