70bac87fed
Most of the warnings we don't really care about, and those are simply white-listed using inline comments; however two cases prompted actual code changes: - In `src/display/pattern_helper.js` the branch in question is indeed unreachable, and should thus be safe to remove. (This code originated in PR 4192, which is now over seven years ago.) - In `test/test.js`, the function in question indeed doesn't accept any arguments. (The patch also re-formats a string just above, which didn't seem worthy of a separated patch.) This now leaves only *one* warning in the LGTM report, however that one is a false positive that we'll need to report upstream. |
||
|---|---|---|
| .. | ||
| accessibility_spec.js | ||
| annotation_spec.js | ||
| find_spec.js | ||
| scripting_spec.js | ||
| test_utils.js | ||