P4Available is probable DUC in several targets

RESOLVED WONTFIX

Status

--
trivial
RESOLVED WONTFIX
8 years ago
a month ago

People

(Reporter: pnkfelix, Unassigned)

Tracking

unspecified
Bug Flags:
flashplayer-qrb +

Details

Brent and Felix were wondering yesterday about what android should do for its P4Available definition.  They were struck by the fact that the function was defined for several targets, but never referenced in the AVM source tree (AFATCT).

Bug 504202, comment 4 identifies a historical use of P4Available; that's one known invocation, which was removed in TR changeset 3824:8ee29cea9d4a

It now seems likely that P4Available is DUC and can be removed.
(In reply to comment #0)
> Bug 504202, comment 4 identifies a historical use of P4Available; that's one
> known invocation, which was removed in TR changeset 3824:8ee29cea9d4a

TR changeset 3824 is:
  http://hg.mozilla.org/tamarin-redux/rev/8ee29cea9d4a
Incidentally the real benefit of getting rid of P4Available is that it seems likely that we could also remove the various platform *cpuid.cpp files along with it.

(Removing those files does introduce the usual project file maintenance pain but surely we're better off without it all if we can get away with it.)

Updated

7 years ago
Flags: flashplayer-qrb+
Tamarin is a dead project now. Mass WONTFIX.
Status: NEW → RESOLVED
Last Resolved: a month ago
Resolution: --- → WONTFIX
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
You need to log in before you can comment on or make changes to this bug.