noprompt / frak

Transform collections of strings into regular expressions.
1.13k stars 39 forks source link

Add missing escape #12

Closed guns closed 10 years ago

guns commented 10 years ago

Hi @noprompt,

We're looking to expand usage of frak in vim-clojure-static and I found these two little bugs in the process.

BTW, would it be possible to either have access to escape-chars or provide a custom escape function? I'd like to be able to escape vim-specific special characters like <, >, @, and more. There are lots of ways to implement this (dynamic var, explicit parameter, callback, etc), so I didn't prepare a patch.

Thanks!

noprompt commented 10 years ago

Good catch. I want to say I noticed this bug a little while ago but never got around to patching it. Thanks for fixing this. :smile:

BTW, would it be possible to either have access to escape-chars or provide a custom escape function?

This is definitely ok with me. If you'd like to draw up something and open another issue that'd be great.