Skip to content

Noissue - Include blank keyword in regular expression

Manish Jethani requested to merge skipintro/adblockpluscore:simplify-keyword-candidates into next

Issues

None.

Description

This is based on !47 (closed)

There's no need to check for null and push an empty string onto the candidate list. We can get the same result by appending a |$ to the regular expression.

Performance is the same or better.

Checklist

  • Passes all tests
  • Passes linting

@hfiguiere

Merge request reports