Mistake in r8758

[Zalbor]

With revision 8758, the utils/snit/snit2.tcl file was updated. If ran with this, amsn doesn't work:

Code:

$ ./amsn
Error in startup script: extra characters after close-brace
    while executing
"set command [list  $self  {*}$Snit_optionInfo(configure-$option)  $option]
            "
    (procedure "snit::RT.CacheConfigureCommand" line 36)
    invoked from within
"snit::RT.CacheConfigureCommand  $type $selfns $win $self $option"
    (procedure "::snit::RT.method.configurelist" line 7)
    invoked from within
"::snit::RT.method.configurelist $type $selfns $win $self $args"
    (procedure "::snit::RT.method.configure" line 4)
    invoked from within
"$self configure -width $arrow1width"
    (procedure "::pixmapscrollbar::Snit_constructor" line 154)
    invoked from within
"::pixmapscrollbar::Snit_constructor ::pixmapscrollbar ::pixmapscrollbar::Snit_inst1 .plugins_log.ys .plugins_log.ys -command {.plugins_log.info yview}"
    ("eval" body line 1)
    invoked from within
"eval [linsert $arglist 0  ${type}::Snit_constructor $type $selfns $instance $instance]"
    (procedure "RT.ConstructInstance" line 9)
    invoked from within
"RT.ConstructInstance $type $selfns $name $args"
    (procedure "::snit::RT.widget.typemethod.create" line 53)
    invoked from within
"scrollbar $window.ys -command "$window.info yview""
    (procedure "::pluginslog::draw" line 12)
    invoked from within
"::pluginslog::draw"
    invoked from within
"if { $initialize_amsn == 1 } {
     ::pluginslog::draw
}"
    (file "pluginslog.tcl" line 210)
    invoked from within
"source pluginslog.tcl"
    ("uplevel" body line 27)
    invoked from within
"uplevel \#0 {

        # amsncore.tcl is already loaded but we'll re-source it here in case we manually do reload_files
        source amsncore.tcl
        source audio.tc..."
    (procedure "reload_files" line 2)
    invoked from within
"reload_files"
    (file "./amsn" line 250)

Going back to 8757 it works, with the file's older version.

[kakaroto]

you must use the latest tcl/tk 8.5 release. Tk changed the syntax and this was our fix. anyways, you have to use the latest version otherwise you get a nasty memory leak.

[Zalbor]

Aha, I see. Apparently I had misunderstood, I thought it was 8.5a6 that had the memory leak. Thanks

[Phil]

a6 and all previous alphas are impacted by the memory leak so updating to a6 won't change anything...
But my fix will let some users to use Tcl/Tk CVS which got the memory leak fixed....

[Zalbor]

I see... I'll wait until a7 I think. At least amsn works with a6.

[Phil]

You can wait forever for a7 : it will never out
Next Tcl/Tk 8.5 version will be b1

[Fenix-TX]

You can wait forever for a7 : it will never out
Next Tcl/Tk 8.5 version will be b1

Really?

Extracted from README file:

README:  Tcl
    This is the Tcl 8.5a7 source distribution.
    Tcl/Tk is also available through NetCVS:
   http://tcl.sourceforge.net/
    You can get any source release of Tcl from the file distributions
    link at the above URL.

[billiob]

it is said that the current dev version (called a7) will be the b1.