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

[#2066] avoid NPE in regex matcher #1022

Merged
merged 1 commit into from
Nov 11, 2016

Conversation

flybyray
Copy link
Contributor

@flybyray flybyray commented Nov 5, 2016

I think it is enough to check only once for null before calling the regex matcher.

This fix is needed if you really use "open file from errors pages" (a Play! developer feature):

# Open file from errors pages
# ~~~~~
# If your text editor supports opening files by URL, Play! will
# dynamically link error pages to files 
#
# Example, for textmate:
# play.editor=txmt://open?url=file://%s&line=%s

Here is a nice summary how to use it with several os: How open files in ide from debugger

@xael-fry xael-fry merged commit 6061c12 into playframework:master Nov 11, 2016
@xael-fry
Copy link
Member

Merged and pushed in master and 1.4.x
Thanks

@xael-fry xael-fry added this to the 1.4.4 milestone Dec 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants