Thanks, patch applied with minor tweaking re: trailing spaces as discussed on IM, and documentation.
Regards, Dave. > -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of > Magnus Hagander > Sent: 08 January 2006 15:05 > To: pgadmin-hackers@postgresql.org > Subject: Re: [pgadmin-hackers] Query tool: Autocompletion > > Dave tells me the list silently eats large mails, so here is the mail > again without the patch. The patch is now on > http://www.hagander.net/pgsql/patches/pgadmin_autocomplete.patch > > //Magnus > > > > -----Original Message----- > > From: Magnus Hagander > > Sent: Saturday, January 07, 2006 6:33 PM > > To: pgadmin-hackers@postgresql.org > > Subject: Query tool: Autocompletion > > > > Hi! > > > > Attached is a first attempt at autocompletion for the SQL > > Query tool. It probably needs some more work, but it's a start :-) > > > > It's based on the tab completion in psql. The idea is to pull > > the latest and greatest tab-complete.c from psql, then run it > > through a perlscript that picks out the tasty parts. > > > > > > Anyway. Attached is the patch. I've also included a > > tab-complete.inc from the current Due to a bug in the Visual > > C++ compiler, it *has* to be compiled as "C" not as "C++". > > Thus, a bit if ugly glue is required between those two worlds > > :-)psql - to make it build directly (without adding a build > > dependency on perl), this one is what should probably go in > > svn, and then be manually sycned onw and then from psql. > > "tabcomplete.c" needs to be added to the build project (I'm > > devving in Visual Studio 2003, so I can't modify the .dsw > > from there easily). > > > > I've only tested this on Windows so far. Wouldn't surprise me > > if some minor work is needed to build on *nix. > > > > I'm also unsure if I can get away with the easy way I pass > > strings in and out of wx (encoding issues?). It works in my > > testcases, but I'm not familiar enough to be sure if it > always does. > > > > Finally, I've added a screenshot for those who don't want to > > rebuild :) > > > > So. Thoughts, and comments? > > > > //Magnus > > > > ---------------------------(end of > broadcast)--------------------------- > TIP 5: don't forget to increase your free space map settings > ---------------------------(end of broadcast)--------------------------- TIP 6: explain analyze is your friend
