Hey,
that's cool, thanx Jonne for being able to spare some time to work on this. And 
sorry Miguel for 
not having the time myself to dedicate to this TIP.
I have to admit, last time I read your mail and the TIP, I didn't quite 
understand the 
difference between the current behavior of tcl/tk and the proposited behavior 
by the tip. I 
guess what I assumed from the tip is that it's the expected behavior and I 
guess it just fixes 
tcl which doesn't follow the 'guidelines'...
Anyways, Jonne, I don't think this should go into 0.96, maybe back port the 
changes between 
trunk and 0.96, but I don't think it's critical for 0.96, because :
1 - there might be tooo many code to review and it would be impossible to do 
all the test cases 
to see if anything breaks
2 - it could take time and we could postpone the release and in the end it 
won't affect us 
(people not using the version with the tip, or does who do will be able to use 
SVN anyways) 
3 - We still need Miguel to give an answer concerning when will this tip be 
imlpemented in the 
main branch.

about the error you talked about with the initialize_amsn, look carefully at 
the reload_files 
proc, it does :
uplevel #0 {
  source ...
  source ...
  ...
}

so all the sourcing is done in the global namespace, so $initialize_amsn should 
be access as is, 
no ened to fully qualify it since we're in the global namespace already...
at least, that's what I think, mabye Miguel can enlighten us on this subject as 
I may have not 
understood correctly what the changes this tip brings..

thx

KKRT

On Mon, Oct 16, 2006 at 12:07:07PM -0300, miguel wrote:
> Jonne Zutt wrote:
> > It's quite easy to make these changes.
> > I made about 40 changes to amsn already (these can be committed as they
> > work with and without the Tip being implemented).
> 
> Yes, that's the idea.
> 
> > However, Tip 278 also breaks (at least) BWidget.
> > In widget.tcl, one can find this:
> > 
> >   proc Widget::init { class path options } {
> >     ...
> >     set Widget::_class($path) $class
> > 
> > Which needs to be ...
> >     set ::Widget::_class($path) $class
> > or perhaps ...
> >     set _class($path) $class
> > 
> > in the future.
> > 
> > I'll just change utils/BWidget too, doesn't matter of course.
> 
> Could you please forward the info to the bwidget maintainers?
> 
> Thx again
> Miguel
> 
> -------------------------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job easier
> Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> _______________________________________________
> Amsn-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/amsn-devel

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Amsn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to