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