58 bytes, text/x-review-board-request
One of the things ScopedAutoSECItem is useful for is getting rid of this pattern: > ScopedSECItem item(SECITEM_AllocItem(nullptr, nullptr, 0)); While this pattern works, ScopedAutoSECItem is slightly superior in that it doesn't unnecessarily cause a SECItem to be allocated from the heap. Moreover, this moves us further away from Scoped.h, which is deprecated.
Comment on attachment 8800041 [details] Bug 1308890 - Make use of ScopedAutoSECItem in CryptoKey.cpp. https://reviewboard.mozilla.org/r/85048/#review84218
Attachment #8800041 - Flags: review?(rlb) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/32425af1283d Make use of ScopedAutoSECItem in CryptoKey.cpp. r=rbarnes
You need to log in before you can comment on or make changes to this bug.