[Tcl-bugs] [ tktoolkit-Bugs-1630262 ] segfault when deleting a line which is a -startline

[SourceForge.net <no...@so...>]

Bugs item #1630262, was opened at 2007-01-07 19:43
Message generated for change (Settings changed) made by fvogelnew1
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112997&aid=1630262&group_id=12997

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: 18. [text]
>Group: current: 8.5.10
Status: Open
Resolution: None
>Priority: 7
Private: No
Submitted By: Tim Baker (treectrl)
>Assigned to: Francois VOGEL (fvogelnew1)
Summary: segfault when deleting a line which is a -startline

Initial Comment:
text .t
.t insert end "1\n2\n3\n"
.t peer create .peer
.t configure -startline 3
.peer delete 2.0 end


I tracked this down to tkTextBTree.c:AdjustStartEndRefs(),
change 2 lines from this...

treePtr->startEndRef[count] = treePtr->sharedTextPtr->peers;

to this ...

treePtr->startEndRef[count] = textPtr;

----------------------------------------------------------------------

Comment By: Francois VOGEL (fvogelnew1)
Date: 2012-01-31 14:28

Message:
I have proposed a patch which is available for review in branch bug-1630262


----------------------------------------------------------------------

Comment By: Francois VOGEL (fvogelnew1)
Date: 2012-01-23 02:13

Message:
The proposed change indeed prevents the crash for the given test case.
It has however the side effect of resetting -startline of .t to {}, which
is debatable.

Anyway I still get a crash when running the following slightly modified
test case:

text .t
.t peer create .pt
pack .t .pt
.t configure -height 12
.pt configure -height 12
for {set i 1} {$i<10} {incr i} {
  .t insert end "Line $i\n"
}
.t configure -startline 5
.pt configure -startline 3
.pt delete 2.0 end


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112997&aid=1630262&group_id=12997