RESOLVED FIXED 169495
Allow regular expressions to be used when selecting a process name in JSC config file 
https://bugs.webkit.org/show_bug.cgi?id=169495
Summary Allow regular expressions to be used when selecting a process name in JSC con...
Michael Saboff
Reported 2017-03-10 17:13:12 PST
Currently process names filters need to match exactly. It would be better to allow for regular expression matching.
Attachments
Patch (3.19 KB, patch)
2017-03-10 17:16 PST, Michael Saboff 		saam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Michael Saboff
Comment 1 2017-03-10 17:13:30 PST
<rdar://problem/30986750>
Michael Saboff
Comment 2 2017-03-10 17:16:01 PST
Created attachment 304104 [details] Patch
Saam Barati
Comment 3 2017-03-10 17:29:21 PST
Comment on attachment 304104 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=304104&action=review LGTM > Source/JavaScriptCore/runtime/ConfigFile.cpp:148 > + char* stringStart = ++m_srcPtr; Should you `--m_srcPtr` if the below returns nullptr? Or perhaps, just do, stringStart = m_srcPtr + 1, and then modify m_srcPtr on success?
Michael Saboff
Comment 4 2017-03-11 22:16:24 PST
Comment on attachment 304104 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=304104&action=review >> Source/JavaScriptCore/runtime/ConfigFile.cpp:148 >> + char* stringStart = ++m_srcPtr; > > Should you `--m_srcPtr` if the below returns nullptr? > Or perhaps, just do, stringStart = m_srcPtr + 1, and then modify m_srcPtr on success? Good catch. I'll address with "stringStart = m_srcPtr + 1" before landing.
Michael Saboff
Comment 5 2017-03-11 22:27:52 PST
Committed r213768: <http://trac.webkit.org/changeset/213768>
Note You need to log in before you can comment on or make changes to this bug.