Page MenuHomePhabricator

Remove Profiler::profileIn and ::profileOut, deprecated and unused
Closed, ResolvedPublic

Description

Neither is used, and both were deprecated in 1.33.
See codesearch: https://codesearch.wmflabs.org/search/?q=profile(In%7COut)%5C(&i=nope&files=&repos=

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 573847 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Remove Profiler::profileIn and ::profileOut, deprecated and unused

https://gerrit.wikimedia.org/r/573847

Change 573847 abandoned by DannyS712:
Remove Profiler::profileIn and ::profileOut, deprecated and unused

https://gerrit.wikimedia.org/r/573847

DannyS712 removed a project: Patch-For-Review.
DannyS712 moved this task from Awaiting review and deployment to Others on the User-DannyS712 board.
DannyS712 added a subscriber: Krinkle.

Don't really understand this - leaving for someone else
@Krinkle does this still belong in "Watching"?

Yes, our team is not currently working on it, but are subscribed to get notifications and offer code review if needed.

Change 748253 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] profiler: Remove deprecated profileIn/profileOut

https://gerrit.wikimedia.org/r/748253

Change 748253 merged by jenkins-bot:

[mediawiki/core@master] profiler: Remove deprecated profileIn/profileOut

https://gerrit.wikimedia.org/r/748253

Krinkle assigned this task to Umherirrender.
Krinkle triaged this task as Low priority.