Created attachment 306913 [details] [diff] [review] The fix Following on from dwitte's comment in bug 406198, and also the realisation now that we can pre-set an nsCOMArray's size (maybe not on the stack, but at least set it in one go), I think it would be better to change the ArcLabelsOut functions of nsAbDirectoryDataSource and nsSubscribeDataSource to use nsCOMArray rather than nsIMutableArray. Patch attached.
Comment on attachment 306913 [details] [diff] [review] The fix I don't know how long we intend to keep these datasources ;-) >+ nsCOMArray<nsIRDFResource> array(hasChildren ? 5 : 6); This is the wrong way around ;-) but I'd just allocate 6 all the time anyway.
Checked in -> fixed.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.