Closed Bug 643283 Opened 15 years ago Closed 15 years ago

Make SmartPointer the one TraceLocation override

Categories

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

defect

Tracking

(Not tracked)

RESOLVED WONTFIX
Q3 11 - Serrano

People

(Reporter: treilly, Unassigned)

Details

No description provided.
Right now we have overrides for GCWriteBarrier,GCWriteBarrierRC,GCMember,atomWBCore and now all these classes inherit from SmartPointer. Could we move value() and location() to SmartPointer and clean things up? Would help the player, right now there's smart pointer style classes that are inheriting from GCInlineObject and SmartPointer and hooked in via GC_STRUCTURE, using GC_POINTER and making a TraceLocation override for SmartPointer would be cleaner.
Summary: Make SmartPointer the one TraceLocation override. → Make SmartPointer the one TraceLocation override
I think this sounds promising but I'd prefer not to do it prematurely, nor for data that are merely smart pointer "style" - are there reasonable requirements we could place on subclasses of SmartPointer, and are there ways we could check those requirements?
Patch to bug 643413 will remove SmartPointer again.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.