X-Authentication-Warning: delorie.com: mail set sender to geda-user-bounces using -f
X-Recipient: geda-user AT delorie DOT com
Date: Sun, 8 Jul 2012 11:26:20 -0700
From: Andrew Poelstra <asp11 AT sfu DOT ca>
To: geda-user AT delorie DOT com
Subject: Re: [geda-user] [PATCH] gtk hid file chooser default path (corrected)
Message-ID: <20120708182619.GI1637@malakian.lan>
References: <4FF9CF99 DOT 1070901 AT cox DOT net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4FF9CF99.1070901@cox.net>
User-Agent: Mutt/1.5.20 (2009-12-10)
Reply-To: geda-user AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: geda-user AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

On Sun, Jul 08, 2012 at 11:21:13AM -0700, Eric Brombaugh wrote:
> Here's the patch with Patrick and Andrew's suggestions/corrections.
> Limited testing, works for me, use with caution, void where
> prohibited, etc.
> 
> Caveat - the "File->Load Vendor Resource File" menu still has the
> problem. Apparently it doesn't use ghid_dialog_file_select_open()
> function call and in a cursory check I didn't see where it was
> implemented. I don't use that menu item so it's not an issue for me.
>

"Load Vendor Resource File" calls the LoadVendorFrom() action,
which in turn calls gui->fileselect(), which points to ghid_fileselect,
which is implemented at src/hid/gtk/gui-dialog.c:526.

This goes through the same default-directory dance as the others.

I don't use it either (I'm not even sure what it does), but it
would be good for consistency's sake to fix it as well.

-- 
Andrew Poelstra
Email: asp11 at sfu.ca OR apoelstra at wpsoftware.net
Web:   http://www.wpsoftware.net/andrew
"You shouldn't trust every quote you read on the Internet." -- Socrates