As of today's master, RHistImpl::GetBinUncertainty and RHistImpl::GetBinContent are implemented rather differently:
- One is partially implemented at the RHistImplBase layer, the other is fully implemented at the RHistImpl layer.
- One is checking that GetBinIndex returns a positive bin index (where a negative index seems to indicate a bin lookup error), the other doesn't.
The former inconsistency merely makes the code harder to read, the latter inconsistency is probably an error handling bug. Overall, it seems a bit of implementation homogeneization wouldn't hurt there.