Replace StringTokenizer

[oscargus]

See JabRef#1877 and the link in the comment http://stackoverflow.com/questions/691184/scanner-vs-stringtokenizer-vs-string-split

[Siedlerchr]

I just learned the Regex/Matcher Split can also be used in combination with streams:
http://winterbe.com/posts/2015/03/25/java8-examples-string-number-math-files/

[koppor]

One can search for StringTokenizer using Ctrl+Shift+F.

[PFvanBeerendonk]

Would like to take this issue.

[koppor]

Assigned!

As a general advice for newcomers: check out Contributing for a start. Also, guidelines for setting up a local workspace is worth having a look at.

Feel free to ask here at GitHub, if you have any issue related questions. If you have questions about how to setup your workspace use JabRef's Gitter chat. Try to open a (draft) pull-request early on, so that people can see you are working on the issue and so that they can see the direction the pull request is heading towards. This way, you will likely receive valuable feedback.

[PFvanBeerendonk]

I have some minor questions.

1. I am considering not replacing QuotedStringTokenizer since it is its own class, and does not use deprecated class StringTokenizer.
2. All changes are internal, and all internal parts should already be tested. So I assume I do not need to add test cases (since I'm not adding or changing functionality?)

[Siedlerchr]

2. Well, if there are no tests cases you should add one!

[PFvanBeerendonk]

PR is ready

[koppor]

Thank you for investigating this. Comments on the PR.