On Wed, 18 Apr 2012 15:52:26 -0500 "David C. Rankin" <drankinatty@...> wrote: > On 04/18/2012 03:36 PM, David C. Rankin wrote: > > On 04/18/2012 03:26 PM, David C. Rankin wrote: > >> else { > >> = findRange( e->pos() ); > > > > with the 'r' = of course :) > > > > Here is what I have come up with from the additional posts here. I've updated > the patch to bug report 958. The link to the new patch is here: > > http://bugs.pearsoncomputing.net/attachment.cgi?id=549 > > The code reads: > > void K3bAudioEditorWidget::mousePressEvent( TQMouseEvent* e ) > { > Range* r = findRangeEdge( e->pos(), &end ); > m_draggedRange = 0; > m_draggedMarker = 0; > bool end; > > if (r) { > m_draggedRange = r; > m_draggingRangeEnd = end; > setSelectedRange( r ); > } > else { > r = findRange( e->pos() ); > d->movedRange = r; > d->lastMovePosition = posToMsf( e->pos().x() ); > setSelectedRange( r ); > m_draggedMarker = findMarker( e->pos() ); > } > > TQFrame::mousePressEvent(e); > } > > You guys that do the approval before Darrell submits, take a look and let us > know if it is good. I'm on to the next one.. Are you sure that compiles? I would have assumed end needed to be declared before using a reference to it as an argument to findRangeEdge(), which was why I put things in the order that I did. Or maybe I'm wrong.