Minor bug fix in PropertiesUtils

[elepedus]

Incorrect conditional logic was causing prefixes to be removed when keepPrefix was true and kept when it was false.

[AngledLuffa]

Looks legit. I needed to switch the truth value on one of the uses
elsewhere in corenlp to make it work. Thanks!

On Sun, Oct 25, 2015 at 12:59 PM, Edmond Lepedus [email protected]
wrote:

Incorrect conditional logic was causing prefixes to be removed when

keepPrefix was true and kept when it was false.

You can view, comment on, or merge this pull request online at:

#97
Commit Summary

• Add .gitignore
• Fix bug which was preventing the removal of property prefixes

File Changes

• M .gitignore
  https://github.com/stanfordnlp/CoreNLP/pull/97/files#diff-0 (6)
• M src/edu/stanford/nlp/util/PropertiesUtils.java
  https://github.com/stanfordnlp/CoreNLP/pull/97/files#diff-1 (4)

Patch Links:

• https://github.com/stanfordnlp/CoreNLP/pull/97.patch
• https://github.com/stanfordnlp/CoreNLP/pull/97.diff

—
Reply to this email directly or view it on GitHub
#97.

[gangeli]

Closed with commit 7ffd9ba