From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3921 invoked by alias); 2 Jan 2003 23:54:59 -0000 Mailing-List: contact guile-gtk-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: guile-gtk-owner@sources.redhat.com Received: (qmail 3911 invoked from network); 2 Jan 2003 23:54:56 -0000 Received: from unknown (HELO mail.tiscali.cz) (213.235.135.70) by 209.249.29.67 with SMTP; 2 Jan 2003 23:54:56 -0000 Received: from hobitin.ucw.cz (212.11.98.85) by mail.tiscali.cz (6.0.044) id 3DC10A6300C3AED7; Fri, 3 Jan 2003 00:52:39 +0100 Received: from 0rfelyus by hobitin.ucw.cz with local (Exim 3.36 #1 (Debian)) id 18UEuy-0003Ek-00; Fri, 03 Jan 2003 00:38:28 +0100 To: guile-gtk@sources.redhat.com Cc: Kevin Ryde Subject: Re: GtkAdjustment setters References: <87adioh45k.fsf@zip.com.au> From: Daniel Skarda <0rfelyus@ucw.cz> Date: Thu, 02 Jan 2003 23:54:00 -0000 In-Reply-To: <87adioh45k.fsf@zip.com.au> (Kevin Ryde's message of "Mon, 30 Dec 2002 08:05:43 +1000") Message-ID: User-Agent: Gnus/5.090008 (Oort Gnus v0.08) Emacs/20.7 (i386-debian-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2003-q1/txt/msg00000.txt.bz2 Hello, thank you for your patch - I am going to commit it to CVS in few minutes (unfortunately I am working offline, so maybe somebody already did it). There is only one issue I do not like about the patch - it creates a setter gtk-adjustment-set-value!, while we already have gtk-adjustment-set-value (without exclamation mark) binding for Gtk function. Moreover, exclamation marks are not used in guile-gtk at all (so ! in gtk-adjustment setters could be unwanted surprise to guile-gtk users) - but this is not problem of your patch but it is more guile-gtk problem. So I removed *-set-value! setter from your patch in favour of Gtk *-set-value function. 0. ps: BTW - it is strange that "setter" feature has not been used. I am afraid which Gtk features does not have their GuileGtk companions yet... It seems we have to check it very soon.