Closed Bug 1675776 Opened 5 years ago Closed 5 years ago

Document `createServicesCache`

Categories

(Core :: XPCOM, enhancement)

enhancement

Tracking

()

RESOLVED WONTFIX

People

(Reporter: mbrodesser, Unassigned)

References

Details

This method is meant. To someone unfamiliar with it, it's unclear what it returns.

:kmag: if it makes sense, could you please add some documentation for this?

Flags: needinfo?(kmaglione+bmo)

This method is temporary, and is only ever meant to be used in one place. It will go away with bug 1667455, so I don't think it's worth documenting in the mean time.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(kmaglione+bmo)
Resolution: --- → WONTFIX
See Also: → 1667455

(In reply to Kris Maglione [:kmag] from comment #2)

This method is temporary, and is only ever meant to be used in one place. It will go away with bug 1667455, so I don't think it's worth documenting in the mean time.

Wasn't aware of that, thanks. The problem with "temporary" code-states is that sometimes they become permanent. Searching for "temporary" on searchfox yields some examples. Since, in this case, there's the ticket for bug 1667455, it hopefully doesn't happen.

You need to log in before you can comment on or make changes to this bug.