Tcl Source Code

Artifact [ec67d8ac8a]
Login

Artifact ec67d8ac8a1154a88c1b90809b50453b890943f0a54495b18c72f6344696ee6a:

Ticket change [ec67d8ac8a] - Ticket [cc1e91552c] Regression in list expansion status still Open with 3 other changes by sebres 2019-01-11 12:07:02.
D 2019-01-11T12:07:02.972
J icomment Well,\sactually\sI\sdoubt\sthat\sthe\sproblem\sof\sFrançois\swould\sbe\scaused\sdirectly\sby\sthis\sfix\s(possibly\sjust\san\safter-effect),\sbecause\sif\sI'm\snot\swrong\sit\scan\scrash\sin\stwo\scases:\r\n\r\n1.\sin\scall\sof\sListObjIsCanonical,\sbut\sit\sis\sa\snonsense,\sbecause\seither\svaluePtr\sis\sinaccessible\sand\sdon't\sthink\sso,\sbecause\s[/artifact/5718eaeee4fa7747?ln=5222|TclListObjGetElements]\sabove\sby\sretrieving\sof\slist\slength\s(objc),\sor\sby\scheck\sof\sthe\scanonical\sflag\s(canonicalFlag)\sof\slist\sand\sit\swould\sbe\seven\sweird,\seven\sbecause\sTclListObjGetElements\salways\sconvert\svaluePtr\sto\sthe\slist\s(tclListType)\swith\sproper\slist\srepresentation.\r\n\r\n2.\swithin\sgoto,\sbut\sit\sis\salso\shard\sto\simagine\s(below\sINST_LIST_RANGE_IMM\sdoes\sthis\salso\s[/artifact/5718eaeee4fa7747?ln=5260|for\sseveral\scases]\s(e.\sg.\sif\sindices\sare\sout\sof\srange).\r\n\r\nSo\smy\sassumption\sthere\sare\sthe\spossible\safter-effects\slike:\r\n\r\n-\sRef-Counting\sissues,\sfor\sexample\snow\sit\sreturns\snew\sobject\s(refCount=0),\swhere\spreviously\srefCount\swas\s>=1.\r\nIn\sthis\scase\sthe\sTk\smiss\sthe\srefCount\sincrement\ssomewhere.\r\n\r\n-\sType-safety\slike\sconversion\sto\sanother\stype\sin\sthe\sin-between\soperations\s(hard\sto\simagine,\snow\sas\swell\sas\spreviously\sit\sis/was\salways\sa\slist\sin\sthis\scase).\r\n\r\n-\sAnother\sUB.\r\n\r\nAnyway,\sI'll\stry\sto\sfind\sthe\sreason.\r\n\r\n@François,\sis\sthere\sany\stest-case\sI\scould\suse\sto\sreproduce\sthe\scrash?\sCan\syou\spossibly\sprovide\sa\scall-stack\sby\sthe\ssegfault.
J login sebres
J mimetype text/x-fossil-wiki
K cc1e91552c0ca1817292d951f8c694b28c2b2251
U sebres
Z 77315361208c5ef0ae00faf7464b662a