657927 - GC::SetFinalize should be removed, and GC::ClearFinalized should be made private

Status: RESOLVED FIXED

(Tamarin Graveyard :: Garbage Collection (mmGC), defect)

Description

[Lars T Hansen]

There are exactly three users of GC::SetFinalize [sic]:

- RegExpObject::RegExpObject calls it on "this", in two cases
- PlayerScriptObject::PlayerScriptObject calls it on "this"

In all cases the call is redundant, because the objects being constructed are RCObjects and hence always finalized.

There is exactly one user of GC::ClearFinalized [sic]:

- GC::AbortFree calls it to clear the finalized bit.

Since the only client is in the GC, the API can be made private.

With regard to IsFinalized, it is only used from within Tamarin, but it seems benign to leave it public for now (it's used by Sampler).

Comment 1

[Lars T Hansen]

Attachment: Patch

Removes the calls to SetFinalize; removes SetFinalize; makes ClearFinalized private.

Comment 2

[Tamarin Bot]

changeset: 6318:6665a32e4dae
user:      Lars T Hansen <[e-mail]>
summary:   Fix 657927 - GC::SetFinalize should be removed, and GC::ClearFinalized should be made private (r=fklockii)

http://hg.mozilla.org/tamarin-redux/rev/6665a32e4dae