Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiline search using regex fails to return all results #19217

Closed
paymand opened this issue Jan 24, 2017 · 5 comments
Closed

Multiline search using regex fails to return all results #19217

paymand opened this issue Jan 24, 2017 · 5 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality search Search widget and operation issues

Comments

@paymand
Copy link

paymand commented Jan 24, 2017

  • VSCode Version: Code 1.8.1 (ee428b0, 2016-12-19T14:49:23.350Z)
  • OS Version: Windows_NT ia32 6.1.7601
  • Extensions:
Extension Author Version

Hi,
I'm trying to find all occurences of:
''LTS+/\r&
''IGNORED\r&

I use the following regex:

''LTS\+/\\r&[\r\n].*''IGNORED.*[\r\n]

which seems to work in sublime text, for instance.
On VSCode, it only returns one hit.

Steps to Reproduce:

@mousetraps mousetraps added the search Search widget and operation issues label Jan 24, 2017
@roblourens
Copy link
Member

roblourens commented Jan 24, 2017

Our text search in general only works line-by-line at the moment

@roblourens roblourens added the feature-request Request for new features or functionality label Jan 24, 2017
@paymand
Copy link
Author

paymand commented Jan 25, 2017

Oh, well. At least I know why it isn't working :)
👍 for the feature request.

@danwulff
Copy link

Atom does not have multi-line regex searching atm it seems: atom/find-and-replace#303

But I wouldn't mind seeing it implemented. I personally don't care for line breaks in the middle of code but I could see it being useful for others.

@roblourens
Copy link
Member

Actually there is an existing feature request for this: #13155

@roblourens roblourens added the *duplicate Issue identified as a duplicate of another issue(s) label Feb 8, 2017
@steviesama
Copy link

steviesama commented Aug 1, 2017

@danwulff I don't know if it still isn't supposed to have multiline regex, but I have gotten it to work. Here is where I posted about it with explanation and screen shots:

https://github.com/atom/find-and-replace/issues/303#issuecomment-319508730

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality search Search widget and operation issues
Projects
None yet
Development

No branches or pull requests

5 participants