1455563 - Audit code coverage reports and improve testing or remove unused code

Status: RESOLVED FIXED

(Core :: JavaScript: GC, enhancement, P3)

Description

[Jon Coppeard (:jonco)]

Code coverage results are available for the codeline:

https://coveralls.io/github/marco-c/gecko-dev

This tells us what was exercised during testing.  We should check js/src/gc for untested code and either ensure that it does get tested or remove it if it's not needed.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The meta keyword is there, the bug doesn't depend on other bugs and there is no activity for 12 months.
:jonco, maybe it's time to close this bug?

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The meta keyword is there, the bug doesn't depend on other bugs and there is no activity for 12 months.
:jonco, maybe it's time to close this bug?

Comment 3

[Jon Coppeard (:jonco)]

Code coverage has moved here: https://coverage.moz.tools/

Comment 4

[Jon Coppeard (:jonco)]

I looked at files with 80% coverage or less and found a couple of things but nothing major. We seem to have decent coverage.

Comment 5

[Jon Coppeard (:jonco)]

Attachment: Bug 1455563 - Remove some dead code from the GC r?sfink

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e2ba76aefa65
Remove some dead code from the GC r=sfink

Comment 7

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/e2ba76aefa65