Fix #264: Distance inherent data provider should not fail

• Remove unused error handling that was not used because check_inherent is not implemented in our case.
• Remove the error return for the inherent data provider, as it is optional and should not fail under any circumstances.

assigned to @bgallois

added 1 commit

• f48a5ecf - inherent data provider cannot fail

Compare with previous version

added C-binary label

added RN-binary label

changed the description

requested review from @HugoTrentesaux

marked this merge request as ready

approved this merge request

requested review from @tuxmain and removed review request for @HugoTrentesaux

changed milestone to %client-0.10.0

I like the variable shadowing with early return pattern. It makes indentation more predictable and logic blocs more visible.

You can also use let...else instead of shadowing with let; match but that becomes a matter of personal taste.

approved this merge request

added 3 commits

• a0afb078 - 1 commit from branch master
• f4f23dfd - remove unused errors
• b270ecb9 - inherent data provider cannot fail

Compare with previous version

merged

mentioned in commit e3298309