Update the logic in consumeAll to be more readable

[catri0na]

I attempted to make this more readable:

• Removed while true
• Added a bool flag so we no when to exit the parser loop
• moved returns outside of the while loop

Please note I have not really tested this outside of the building the app and seeing a flag nor have a benchmarked I am putting up this PR so you can see my approach

[benpious]

I'm going to close this as it's over six months old and out of date. But feel free to reopen if you want to update it.