Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 365806

Summary: Use common regex util
Product: [ECD] Orion Reporter: Mark Macdonald <mamacdon>
Component: ClientAssignee: Mark Macdonald <mamacdon>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 0.3   
Target Milestone: 0.4 M1   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Mark Macdonald CLA 2011-12-06 15:30:53 EST
While fixing Bug 364481 I added a library for dealing with regex stuff: "regex.js". I see at least 2 other places in the code that are using copied "escapeRegex" functions. They need to be calling the library function instead.
Comment 1 Mark Macdonald CLA 2011-12-06 15:55:51 EST
Hmm, fixing this would make editorFeatures.js pull in regex.js just to support case-insensitive search in incremental find. Anyone consuming the editor via the <script> method will need to write out the extra dependency. 

We might want to avoid this by splitting up editorFeatures into smaller units of functionality, which consumers could add (or not) piece-by-piece.