Really, you should leave this here.  it's _really_ bad form to have global (and globally-accessible) values.  If you need to pass it to your plot prefs dialog, just pass it as &m_sfLapOpts.  If LapPainter needs to use it, it is already accessible via the lap options interface.  Since you kept the name as m_sfLapOpts, it shouldn't be too hard to switch it back to being owned by CMainUI rather than a global.

http://en.wikipedia.org/wiki/Global_variable