Consistent use of unsigned or signed integers in SciFi hit container
The SciFi hit container uses a lot of unsigned integers. This should be changed to signed integers where reasonable. See here the discussion. While at it, fix:
-
signed vs unsigned -
prefer pass by value -
fix unused argument in fast binary search unwind function