From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24608 invoked by alias); 23 Oct 2002 20:14:14 -0000 Mailing-List: contact rhdb-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Post: List-Help: , Sender: rhdb-owner@sources.redhat.com Received: (qmail 24599 invoked from network); 23 Oct 2002 20:14:13 -0000 Received: from unknown (HELO smtp012.mail.yahoo.com) (216.136.173.32) by sources.redhat.com with SMTP; 23 Oct 2002 20:14:13 -0000 Received: from pppoe1227.gh.centurytel.net (HELO thor.local) (brett?schwarz@64.91.48.249 with plain) by smtp.mail.vip.sc5.yahoo.com with SMTP; 23 Oct 2002 20:14:13 -0000 Subject: Re: rhdb-admin From: Brett Schwarz To: Fernando Nasser Cc: rhdb@sources.redhat.com In-Reply-To: <3DB6FCDE.2030901@redhat.com> References: <1035397212.6808.92.camel@thor> <3DB6FCDE.2030901@redhat.com> Content-Type: text/plain Content-Transfer-Encoding: 7bit Date: Wed, 23 Oct 2002 13:14:00 -0000 Message-Id: <1035404052.6559.110.camel@thor> Mime-Version: 1.0 X-SW-Source: 2002-q4/txt/msg00005.txt.bz2 On Wed, 2002-10-23 at 12:47, Fernando Nasser wrote: > Brett Schwarz wrote: > > > 2) To load Itcl, Itk, and Iwidgets, you only need to do package require > > Iwidgets ... it loads Itk, and Itcl as well... > > > > Any harm in leaving them there? The packages should all be loaded very > early in the execution anyway and having the explicit require better > documents things. Besides we may depend on a specific version of some > of these packages one day. > No, there is no harm. The package command won't load them again, if they are already loaded, so there is no performance hit... > > > 3) You also gain some by packing widgets together that have the same > > pack options. > > > > I do have the feeling that we have too many packages. Maybe it is a > good idea to simplify this for next version. I was actually talking about the 'pack' geometry manager. For example, if you have several widgets being packed: pack .w1 pack .w2 pack .w3 pack .w4 This can be done as: pack .w1 .w2 .w3 .w4 If all of the pack options are the same (i.e. -side, -expand, etc). Even if one widget differs, it still might be advantageous to do: pack .w1 pack .w2 pack .w3 pack .w4 pack configure .w2 -fill both As far as packages go, you might want to look into tclIndex instead of using the package mechanism for internal "packages". The one advantage is that it "loads on demand", so presumably, your startup time will be less, and then you just take small incremental hits during interaction with the user. However, it is a little more maintenance, since you have to create the tclIndex file whenever you add/remove a proc/method/etc. This is something you might want to investigate...might not make a difference for this app... > > > 4) In some of the classes, the Itcl commands are not fully qualified, > > and rhdb-admin doesn't even startup. I just added namespace import > > ::itcl::* to make it work, but maybe a better idea is to fully qualify > > the Itcl commands, unless you know for sure there won't be any name > > clashing. (I think the one I remember was configbody in > > NewDisjointListBoxWidget class) > > > > We fixed this already and will be checking this and a few other things in > cvs soon. I run into this problem myself when I upgraded my version of > itcl/iwidgets. > Cool... Thanks, --brett -- Brett Schwarz brett_schwarz AT yahoo.com