Patches item #1571568, was opened at 2006-10-05 09:43
Message generated for change (Comment added) made by andreas_kupries
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=310894&aid=1571568&group_id=10894

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: 07. Variables
Group: TIP Implementation
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Andreas Kupries (andreas_kupries)
Assigned to: Andreas Kupries (andreas_kupries)
Summary: [TIP 280] extended 'info frame'

Initial Comment:
Implementation of 'info frame' going beyond #211.
Derived from the 'info linenum' of #86.

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

> Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-11-28 13:21

Message:
Logged In: YES 
user_id=75003
Originator: YES

Updated to Head for both branches, added documentation to 8.5 branch.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-11-24 12:28

Message:
Logged In: YES 
user_id=75003
Originator: YES

Updated the patches to CVS head of 8.4/8.5 as of Nov 24.
Resolved the conflicts which occured for 8.5 due to code changes.
Fixed the line numbers in the info.test file.


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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-11-21 12:26

Message:
Logged In: YES 
user_id=75003
Originator: YES

NOTES.

The Tcl 8.5 patch is unconditional.

The Tcl 8.4 patch implements the functionality conditionally.
Define the CPP macro "TCL_TIP280" to activate the functionality.


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

Comment By: Eckhard Lehmann (ecky-l)
Date: 2006-11-01 23:47

Message:
Logged In: YES 
user_id=1345135

some tests for [info frame] still fail... Presumably due to
line movements in the test scripts. Otherwise it works fine
in 8.5a6, as far as I could see.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-30 12:26

Message:
Logged In: YES 
user_id=75003

Next 8.5 (base4). Fixed crashes and test failures. This one
should be good for testing.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-27 17:35

Message:
Logged In: YES 
user_id=75003

Next 8.5 (base3). New: Completed handling of 'apply' (aka
lambda). Interpreted dift for/with/filter-script, bcc dict
for, bcc switch. First handling of {expand}. NOTE!! This is
known to crash tests in compile.test, and basic.test (It
passes info.test). Patch has been added more to have it safe
somewhere else than on my machine, than for testing and use.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-26 16:20

Message:
Logged In: YES 
user_id=75003
Originator: YES

Next 8.5 patch (base2). Updated test cases (tcltest
locations changed slightly), additional comments. Test cases
for {expand} added. Two of them known to fail, demonstrating
that {expand}{literal list} is not tracking line numbers
inside. Yet.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-26 11:40

Message:
Logged In: YES 
user_id=75003

Next set of patches, base8 for 8.4, base1 for 8.5. Fixed a
bug in the new code handling interpreted switch, forgot a
qualifying condition when releasing data from the context.
DecrRefcount on a path tcl_obj which was no such, but a
bytecode*, this smashed its interphandle.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-25 16:07

Message:
Logged In: YES 
user_id=75003

First (base0) 8.5 patch. Fails 4 test cases, otherwise ok.
Equivalent to 8.4 base7 patch. This patch does not yet deal
with {expand} yet, nor compiled 'switch', nor 'dict for'
etc. I.e. 8.5 specific pieces have not been done yet.


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

Comment By: Donal K. Fellows (dkf)
Date: 2006-10-25 16:03

Message:
Logged In: YES 
user_id=79902

As the author of the [switch] compiler, I can say that yes,
it's doing things that are horrific. I learned the hard way
(through many crashes in testing) that tokens *must* refer
to the position of the token string within the overall
script or things go badly askew. The way I ended up doing
this is definitely a hack, but it is a safe one since any
time the compiler can't be sure, it just falls back to the
interpreted code (throwing up its hands in horror in the
process).

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-25 15:44

Message:
Logged In: YES 
user_id=75003

Next 8.4 (base7). Implemented line tracking for 'switch',
additional tests for that, and ensured that dynamic procs
return type 'proc'. Switch is a bit of a beast with its
single and multi-word modes. In the single-word mode we have
to track lines inside the list holding the patterns and
branch scripts. Unavoidable even if the data is later not
used. In 8.5 modifying the script compile function is ...
difficult. A draft forward port has been done, but fails
parts of the testsuite ...

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-23 17:07

Message:
Logged In: YES 
user_id=75003

It will be relevant for the 8.5 implementation. Should use
'memset', then set all the non empty fields.

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

Comment By: miguel sofer (msofer)
Date: 2006-10-23 16:55

Message:
Logged In: YES 
user_id=148712

The comments reminds me: TclApplyObjCmd also does not
initialise procPtr-> cmdPtr properly, the only non-trash
field is nsPtr. Not sure if this is relevant.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-23 16:42

Message:
Logged In: YES 
user_id=75003

Next (base6). Fix bug in Proc setup and cleanup for
precompiled bytecode (tbcload). Setup assumed that 'line'
infomration was not NULL, but can happen, i.e. for
precompiled bc. Cleanup crashed because iPtr was assumed to
be valid. Is not for precompiled bc. This also exposed a bug
in tbcload, ExtractFromProcBody (file cmpRead, line 2567)
does not fully initialize the Proc structures it allocates.
Bogus non-NULL proc-> iPtr cannot be checked for validity.
The 'cmdPtr' field is not initialized either. Working on
tbcload patch ...

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-19 17:49

Message:
Logged In: YES 
user_id=75003

Next (base5). Fixed the majority of the knownBug test cases,
i.e. 'if 1 $body' etc. Currently the only remaining knownBug
is the handling of cont. lines in {...}-words.


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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-19 14:30

Message:
Logged In: YES 
user_id=75003

Next (base4). Moved the normalization from 'info frame' to
'source'. Otherwise the 'file' entry in the result can be
wrong for relative paths (pwd changed between the calls of
'source' and 'info frame'). Thanks to Don Porter for
pointing this out. Should have no impact on performance due
to 'source' already doing normalization for reading the
file, and the result of that getting cached. Also removed
some debugging cruft from compile.test ... File now
unchanged again.

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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-19 13:54

Message:
Logged In: YES 
user_id=75003

Next patch (280 base3). normalized 'file' entry, bugfix in
use of path which seg.faulted when using 'info frame 0' at
the console of an interactive shell. Additional testcases,
mostly 'knownBug'.


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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-19 09:58

Message:
Logged In: YES 
user_id=75003

Uploading new patch (280 base2). Fixed the use of newer gcc
features which crept in and break with older versions of
gcc, or other native compilers (c++ var decl in the middle
of a block, mixing macro usage with #ifdef).


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

Comment By: Andreas Kupries (andreas_kupries)
Date: 2006-10-18 13:48

Message:
Logged In: YES 
user_id=75003

Uploading a new patch implementing the TIP in 8.4. changes:
TIP reference 86/211 ->  280, code is conditional
(TCL_TIP280),  bug fixes (mem leaks).


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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=310894&aid=1571568&group_id=10894

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Tcl-Bugs mailing list
Tcl-Bugs@[...].net
https://lists.sourceforge.net/lists/listinfo/tcl-bugs