Better var name and added a comment

jaegertracing · Sep 26, 2017 · 99c2d42 · 99c2d42
1 parent ae5702f
commit 99c2d42
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/components/SearchTracePage/SearchDropdownInput.js b/src/components/SearchTracePage/SearchDropdownInput.js
@@ -48,9 +48,9 @@ export default class SearchDropdownInput extends Component {
   }
   onSearch(_, searchText) {
     const { items, maxResults } = this.props;
-    const rxStr = regexpEscape(searchText);
-    const rx = new RegExp(rxStr, 'i');
-    return items.filter(v => rx.test(v.text)).slice(0, maxResults);
+    const regexStr = regexpEscape(searchText);
+    const regex = new RegExp(regexStr, 'i');
+    return items.filter(v => regex.test(v.text)).slice(0, maxResults);
   }
   render() {
     const { input: { value, onChange } } = this.props;

diff --git a/src/utils/regexp-escape.js b/src/utils/regexp-escape.js
@@ -18,6 +18,9 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 // THE SOFTWARE.
 
+/**
+ * Escape the meta-caharacters used in regular expressions.
+ */
 export default function regexpEscape(s) {
   return s.replace(/[-/\\^$*+?.()|[\]{}]/g, '\\$&');
 }