I've been going over the source code and noticed a few things that are a bit of headscratcher. Meaning I think they're bugs or at least an oversight.
As of right now, I have 2 that I found:
It was confirmed on GitHub that the return of this is never used and the method should be re-written
On line 83, a variable valueType
is made. It is never used in the while-loop below it, only to eventually hit line 150, where it is checked for null and will always be null.
What should happen in this case?