Opened 15 months ago

Last modified 14 months ago

#6823 closed change

Remove unnecessary usage of the u flag in regular expressions — at Version 1

Reported by: mjethani Assignee: mjethani
Priority: P1 Milestone:
Module: Core Keywords:
Cc: kzar Blocked By:
Blocking: #6821 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29843588/

Description

Background

See #6821 for the background.

In a couple of places we are using the u flag in regular expressions where it is not really needed (it does not affect \s\).

What to change

In lib/snippets.js and lib/common.js remove the unnecessary usage of the u flag in the two places where it is being used.

Hints for testers

Same as #6781 and #6619.

Change History (1)

comment:1 Changed 15 months ago by mjethani

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.