Rename ArenaFalseCleaner to PORT_FreeArena_false

RESOLVED FIXED in mozilla32

Status

()

Core
Security: PSM
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: cviecco, Assigned: cviecco)

Tracking

unspecified
mozilla32
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Missunderstood a review comment, so we need to rename a function to maintain coding style. See https://bugzilla.mozilla.org/show_bug.cgi?id=1000548#c9.
(Assignee)

Updated

4 years ago
Assignee: nobody → cviecco
(Assignee)

Comment 1

4 years ago
Created attachment 8435982 [details] [diff] [review]
rename-plaerna-fix
(Assignee)

Comment 2

4 years ago
Comment on attachment 8435982 [details] [diff] [review]
rename-plaerna-fix

Review of attachment 8435982 [details] [diff] [review]:
-----------------------------------------------------------------

::: security/pkix/include/pkix/pkixtypes.h
@@ +39,1 @@
>    // PL_FreeArenaPool can't be used because it doesn't actually free the

PORT_FreeArena_false is kind of bad the you would expect PORT_functions to be NSPR functions. Maybe instead: mzPkix_FreeArena_false ?
Attachment #8435982 - Flags: review?(dkeeler)
Comment on attachment 8435982 [details] [diff] [review]
rename-plaerna-fix

Review of attachment 8435982 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for making this change.

::: security/pkix/include/pkix/pkixtypes.h
@@ +39,1 @@
>    // PL_FreeArenaPool can't be used because it doesn't actually free the

I agree it's not the best naming convention. Let's go with PORT_FreeArena_false to be consistent for now. We can change this and the ones in mozilla::psm if we come up with something better later.
Attachment #8435982 - Flags: review?(dkeeler) → review+
https://hg.mozilla.org/mozilla-central/rev/33e00e8b000e
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.