geoRssConvertor: add missing return statement

[pjonsson]

What this PR does

Adds a missing (?) return statement. If the intention is to return undefined when falling out of the loop, the result variable is unused and should be removed.

Test me

Not sure how to test this.

Checklist

• There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
• I've updated relevant documentation in doc/.
• I've updated CHANGES.md with what I changed.
• I've provided instructions in the PR description on how to test this PR.

[pjonsson]

@zoran995 this code is from 2020 in commit 8ebc61a, do you remember if your intention was to return undefined when falling out of the loop?

[pjonsson]

@zoran995 seems I pinged you in the middle of the holidays so not sure if you saw this, do you have any recollection of whether the fall-through in this function (but not the others) was intentional?

[zoran995]

Honestly, I don't recall anymore; it seems like a forgotten. Why else would someone create an object and return it? Ideally, we need to find a sample, test it and make sure it works correctly