Remove unreachable statement

[LukasReschke]

@MorrisJobke @schiesbn

[MorrisJobke]

👍

[scrutinizer-notifier]

The inspection completed: No new issues

[ghost]

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/1108/
🚀 Test PASSed. 🚀

[MorrisJobke]

@oparoz @danbartram Easy to review ;)

[oparoz]

Is there some background info somewhere as to what this fixes? ;)

[LukasReschke]

Well, this fixes nothing except that the code will actually never get executed. When an exception is thrown the code after that will not get executed.

Therefore the return false is just unnecessary and confusing for developers.

[MorrisJobke]

@oparoz It was spotted by an static code analyzer. And if you look at the code it totally makes sense to drop the return statement as it is useless.

[danbartram]

👍

[oparoz]

Thanks guys for letting me know what to look for. It was indeed obvious.

[MorrisJobke]

Thanks guys. :)