On Wednesday 22 October 2003 10:12 am, Chris Cannam wrote: > EventView::m_listSelection is a bit of an oddity. It's written in > various places, but the only place it's actually read (applyLayout()) > looks like it could as well be done with a local variable. > > Have I overlooked a situation in which there can actually be an > effective selection on the event list that isn't visible in the list > widget, or can we do away with this one?
No. There's stuff there that needs to act where current cursor position is when it isn't necessarily selected. So the code you're looking at is there to fix the event list to work in a reasonably sensible manner (for no selection at all). While it could probably done better you'd be breaking some behaviour by getting rid of this. So please don't touch it cos it took quite some effort to get working. R ------------------------------------------------------- This SF.net email is sponsored by OSDN developer relations Here's your chance to show off your extensive product knowledge We want to know what you know. Tell us and you have a chance to win $100 http://www.zoomerang.com/survey.zgi?HRPT1X3RYQNC5V4MLNSV3E54 _______________________________________________ Rosegarden-devel mailing list [EMAIL PROTECTED] - use the link below to unsubscribe https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
