diff --git a/ChangeLog b/ChangeLog index 98364be66..7ce881307 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2004-09-05 10:30 Matt Rice + + * Source/NSTableView.m: [NSTableView mouseDown:] removed code which + copied and subsequently DESTROYs dataCell. This was causing a + SIGSEGV. + 2004-09-05 06:23 Matt Rice * Source/GSServicesManager.m: new method -activateIgnoringOtherApps: diff --git a/Source/NSTableView.m b/Source/NSTableView.m index 829146bba..e4b3d7e21 100644 --- a/Source/NSTableView.m +++ b/Source/NSTableView.m @@ -3460,9 +3460,7 @@ inline float computePeriod(NSPoint mouseLocationWin, // Prepare the cell tb = [_tableColumns objectAtIndex: _clickedColumn]; - // NB: need to be released when no longer used - // Not sure if we really need a copy here. - cell = [[tb dataCellForRow: _clickedRow] copy]; + cell = [tb dataCellForRow: _clickedRow]; [cell setObjectValue: [self _objectValueForTableColumn: tb row: _clickedRow]]; cellFrame = [self frameOfCellAtColumn: _clickedColumn @@ -3498,8 +3496,6 @@ inline float computePeriod(NSPoint mouseLocationWin, [cell setHighlighted: NO]; [self setNeedsDisplayInRect: cellFrame]; lastEvent = [NSApp currentEvent]; - - DESTROY(cell); } while (done != YES)