Opened 16 months ago

#6267 new change

[emscripten] Get rid of global storage of regular expressions in native implementation

Reported by: sergz Assignee:
Priority: P3 Milestone:
Module: Core Keywords:
Cc: hfiguiere Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

It's a continuation from https://codereview.adblockplus.org/29661608/diff/29663576/compiled/library.cpp#newcode45.

What to change

Introduce an opaque structure representing a regular expression (e.g. by typedefs switched on by #ifdef conditions), which is just and integer in the case of JS and std::wregex in the case of C++, therefore the methods working with it should be adjusted and global regexPool and regexPoolMutex should be removed.

Change History (0)

Note: See TracTickets for help on using tickets.