Fixed CFFCrosshairOptions::UpdateSliders assert and did some cleanup

That m_pWeapon and iCurrentWeapon has always been incorrectly named, as 0 is actually the Global crosshair. I'll do another pass to clean it up some more.