Introduction of Java 8, dependency to guava resolved (issue #40) #43
Conversation
…ompatible with Java 7)
…8 Lambdas ; guava library removed ; Upgrade to Java 8
…is no alternative for javax.annotation.Nullable other than com.google.code.findbugs.annotations)
…ng over tokens and "peeking" the next token without modifying the iterator)
…duces the complexity and is a required step to simplify the iterator handling.
…duces the complexity and is a required step to simplify the iterator handling.
…e TokenIterator#peek(...)
…e TokenIterator#peek(...)
…e TokenIterator#peek(...)
…e TokenIterator#peek(...)
… more to "peek" elements from the iterator, but the new solution is also not perfect and needs some clean-ups...
… more to "peek" elements from the iterator, but the new solution is also not perfect and needs some clean-ups...
Novanic
changed the title
|
There is also a fix for a NPE included, see commit ed475f9. |
|
There is also a refactoring of the "Parser" class included. The class should still be refactored/rewritten in my optinion, because the iterator handling is very confusing and changes can easily result in negative side-effects. The iterator is modified in various methods and depending on a lot of conditions. Now it should hopyfully a little bit better. The PeekingIterator isn't required anymore, the ParserState and the iterator are lesser modified and therefore the complexity should got reduced, but I could not completely get rid of the confusing iterator handling. Please check this class in detail with the review. |
|
I think "Travis CI" should execute an mvn clean ("mvn clean install" instead of "mvn install"). Maybe it has problems with the changed Java version. Locally "mvn clean install" is working. All classes can be compiled and all tests run successfully. |
|
Thanks for sending this. It will take some time to review (and the team is quite busy). I merged the NPE bug fix. |
|
any chance to get this in or is airline considered unmaintained? Should I look for alternatives? I dont wanna be rude, just make an informed decision - I like the library a lot but would love to get rid of guava. |
No description provided.