johnturek / advanced-find

Automatically exported from code.google.com/p/advanced-find
0 stars 0 forks source link

Cannot find certain texts containing "$" #48

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Not using Regular expression
2. Search all documents in directory
3. Directory include sub-folder
4. Use filter like "*.rb"
4. Search for certain texts that are directly copied from the files opened in 
Gedit (ie, these texts do exist!):

$1
$!.record
$(

What is the expected output? What do you see instead?
Should return list of matching results. Instead, "0 hits in 0 files".

What version of the product are you using? On what operating system?
3.3.2 on ArchLinux with kernel 3.1.8-1, Gedit 3.2.6.

Please provide any additional information below.

Original issue reported on code.google.com by n...@gust.com on 9 Jan 2012 at 11:07

GoogleCodeExporter commented 9 years ago
I will fix this at next release.

Original comment by swatch.c...@gmail.com on 10 Jan 2012 at 12:36

GoogleCodeExporter commented 9 years ago
Fixed at v3.3.3.
Thanks for your reply.

Original comment by swatch.c...@gmail.com on 12 Jan 2012 at 11:49

GoogleCodeExporter commented 9 years ago
Thank you for quick fix. However, just find similar bug. Will create a new 
issue.

Original comment by n...@gust.com on 13 Jan 2012 at 12:22

GoogleCodeExporter commented 9 years ago
Testing on 3.3.4, found one pattern still broke, which is very commonly used in 
Javascript files:

$('

However, the similar one worked well:
$("

Original comment by n...@gust.com on 17 Jan 2012 at 12:12

GoogleCodeExporter commented 9 years ago
Try v3.3.5.
I think the problem is solved.
Thanks for your replying.

Original comment by swatch.c...@gmail.com on 17 Jan 2012 at 11:52

GoogleCodeExporter commented 9 years ago
Confirmed! Thank you very much.

Original comment by n...@gust.com on 18 Jan 2012 at 5:07

GoogleCodeExporter commented 9 years ago

Original comment by swatch.c...@gmail.com on 18 Jan 2012 at 11:37