it would make more sense to me to have a flag hasError instead of using expected cases how the BaseTrackerRecHit is created
"default"-constructed 
```
btrh = BaseTrackerRecHit(DetId id, GeomDet const * idet, trackerHitRTTI::RTTI rt)
```
will return hasPositionAndError true, which is not what we want