[2.1] Prevent infinite loop on empty xml elements

[DeepDiver1975]

port of #158 to 2.1

[codecov]

Codecov Report

Merging #161 into 2.1 will decrease coverage by 1.33%.
The diff coverage is 62.5%.

@@             Coverage Diff              @@
##                2.1     #161      +/-   ##
============================================
- Coverage     98.82%   97.48%   -1.34%     
  Complexity      112      112              
============================================
  Files            13       13              
  Lines           426      438      +12     
============================================
+ Hits            421      427       +6     
- Misses            5       11       +6

Impacted Files	Coverage Δ	Complexity Δ
lib/Deserializer/functions.php	87.5% <62.5%> (-6.62%)	0 <0> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5f510f...e15454e. Read the comment docs.

[DeepDiver1975]

https://github.com/sabre-io/xml/releases/tag/2.1.2

[evert]

This feels a bit hacky. Makes me wonder if this is there is a cleaner way to do this. Would for instance just a single $reader->next() do the job (removing $reader->read()?)

[DeepDiver1975]

THX for your feedback @evert

I can have a look at this ....
Since I already tagged the versions are you okay in accepting this implementation as is?

[evert]

Yea np. If it works it works. It was a mild concern. The tests pass which is the most important =)