Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-31 Thread Devrim GÜNDÜZ
Hi,

On Wed, 2012-05-30 at 22:03 -0400, Bruce Momjian wrote:
 A control variable was added in this commit:
 
 commit db84ba65ab5c0ad0b34d68ab5a687bc5f4ca3ba6
 Author: Peter Eisentraut pete...@gmx.net

Thanks Bruce, apparently I missed it.

Regards,
-- 
Devrim GÜNDÜZ
Principal Systems Engineer @ EnterpriseDB: http://www.enterprisedb.com
PostgreSQL Danışmanı/Consultant, Red Hat Certified Engineer
Community: devrim~PostgreSQL.org, devrim.gunduz~linux.org.tr
http://www.gunduz.org  Twitter: http://twitter.com/devrimgunduz


signature.asc
Description: This is a digitally signed message part


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-30 Thread Devrim GÜNDÜZ
Hi,

On Mon, 2012-05-07 at 13:22 -0400, Robert Haas wrote:
 On Sat, May 5, 2012 at 9:03 AM, Bruce Momjian br...@momjian.us
 wrote:
  On Fri, May 04, 2012 at 08:46:28PM +0300, Peter Eisentraut wrote:
  On tor, 2012-05-03 at 15:47 -0400, Bruce Momjian wrote:
   Peter, where are we on this?
 
  I hadn't received any clear feedback, but if no one objects, I can
  commit it.
 
  I think there were enough people that wanted some kind of control in
  this area.  I did give you feedback on the patch.
 
 Yes, there were significantly more votes for reverting this than
 keeping it.  So we at least need to have a setting.

Can we do something about this before beta2 is bundled? I would like it
to be reverted though, rather than having a setting.

Regards,
-- 
Devrim GÜNDÜZ
Principal Systems Engineer @ EnterpriseDB: http://www.enterprisedb.com
PostgreSQL Danışmanı/Consultant, Red Hat Certified Engineer
Community: devrim~PostgreSQL.org, devrim.gunduz~linux.org.tr
http://www.gunduz.org  Twitter: http://twitter.com/devrimgunduz


signature.asc
Description: This is a digitally signed message part


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-30 Thread Bruce Momjian
On Thu, May 31, 2012 at 04:06:50AM +0300, Devrim Gunduz wrote:
 Hi,
 
 On Mon, 2012-05-07 at 13:22 -0400, Robert Haas wrote:
  On Sat, May 5, 2012 at 9:03 AM, Bruce Momjian br...@momjian.us
  wrote:
   On Fri, May 04, 2012 at 08:46:28PM +0300, Peter Eisentraut wrote:
   On tor, 2012-05-03 at 15:47 -0400, Bruce Momjian wrote:
Peter, where are we on this?
  
   I hadn't received any clear feedback, but if no one objects, I can
   commit it.
  
   I think there were enough people that wanted some kind of control in
   this area.  I did give you feedback on the patch.
  
  Yes, there were significantly more votes for reverting this than
  keeping it.  So we at least need to have a setting.
 
 Can we do something about this before beta2 is bundled? I would like it
 to be reverted though, rather than having a setting.

A control variable was added in this commit:

commit db84ba65ab5c0ad0b34d68ab5a687bc5f4ca3ba6
Author: Peter Eisentraut pete...@gmx.net
Date:   Tue May 8 21:03:45 2012 +0300

psql: Add variable to control keyword case in tab completion

This adds the variable COMP_KEYWORD_CASE, which controls in what 
case
keywords are completed.  This is partially to let users configure 
the
change from commit 69f4f1c3576abc535871c6cfa95539e32a36120f, but it
also offers more behaviors than were available before.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-07 Thread Robert Haas
On Sat, May 5, 2012 at 9:03 AM, Bruce Momjian br...@momjian.us wrote:
 On Fri, May 04, 2012 at 08:46:28PM +0300, Peter Eisentraut wrote:
 On tor, 2012-05-03 at 15:47 -0400, Bruce Momjian wrote:
  Peter, where are we on this?

 I hadn't received any clear feedback, but if no one objects, I can
 commit it.

 I think there were enough people that wanted some kind of control in
 this area.  I did give you feedback on the patch.

Yes, there were significantly more votes for reverting this than
keeping it.  So we at least need to have a setting.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-05 Thread Bruce Momjian
On Fri, May 04, 2012 at 08:46:28PM +0300, Peter Eisentraut wrote:
 On tor, 2012-05-03 at 15:47 -0400, Bruce Momjian wrote:
  Peter, where are we on this?
 
 I hadn't received any clear feedback, but if no one objects, I can
 commit it.

I think there were enough people that wanted some kind of control in
this area.  I did give you feedback on the patch.

---

  
  On Fri, Mar 30, 2012 at 08:16:59PM +0300, Peter Eisentraut wrote:
   On fre, 2012-03-23 at 07:52 -0700, David Fetter wrote:
On Thu, Mar 22, 2012 at 06:05:30PM -0400, Andrew Dunstan wrote:
 On 03/22/2012 05:49 PM, Bruce Momjian wrote:
 Robert Haas and I are disappointed by this change.  I liked the
 fact that I could post nice-looking SQL queries without having to
 use my capslock key (which I use as a second control key).  Any
 chance of reverting this change?
 
 
 Should it be governed by a setting?

Something like (upper|lower|preserve) ?
   
   How about this patch then?  (There are actually four possible settings,
   see patch.)
 
 

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-04 Thread Peter Eisentraut
On tor, 2012-05-03 at 15:47 -0400, Bruce Momjian wrote:
 Peter, where are we on this?

I hadn't received any clear feedback, but if no one objects, I can
commit it.

 ---
 
 On Fri, Mar 30, 2012 at 08:16:59PM +0300, Peter Eisentraut wrote:
  On fre, 2012-03-23 at 07:52 -0700, David Fetter wrote:
   On Thu, Mar 22, 2012 at 06:05:30PM -0400, Andrew Dunstan wrote:
On 03/22/2012 05:49 PM, Bruce Momjian wrote:
Robert Haas and I are disappointed by this change.  I liked the
fact that I could post nice-looking SQL queries without having to
use my capslock key (which I use as a second control key).  Any
chance of reverting this change?


Should it be governed by a setting?
   
   Something like (upper|lower|preserve) ?
  
  How about this patch then?  (There are actually four possible settings,
  see patch.)



-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-05-03 Thread Bruce Momjian

Peter, where are we on this?

---

On Fri, Mar 30, 2012 at 08:16:59PM +0300, Peter Eisentraut wrote:
 On fre, 2012-03-23 at 07:52 -0700, David Fetter wrote:
  On Thu, Mar 22, 2012 at 06:05:30PM -0400, Andrew Dunstan wrote:
   On 03/22/2012 05:49 PM, Bruce Momjian wrote:
   Robert Haas and I are disappointed by this change.  I liked the
   fact that I could post nice-looking SQL queries without having to
   use my capslock key (which I use as a second control key).  Any
   chance of reverting this change?
   
   
   Should it be governed by a setting?
  
  Something like (upper|lower|preserve) ?
 
 How about this patch then?  (There are actually four possible settings,
 see patch.)
 

 diff --git i/doc/src/sgml/ref/psql-ref.sgml w/doc/src/sgml/ref/psql-ref.sgml
 index b849101..be9d37d 100644
 --- i/doc/src/sgml/ref/psql-ref.sgml
 +++ w/doc/src/sgml/ref/psql-ref.sgml
 @@ -2652,6 +2652,22 @@ bar
/varlistentry
  
varlistentry
 +termvarnameCOMP_KEYWORD_CASE/varname/term
 +listitem
 +para
 +Determines which letter case to use when completing an SQL key word.
 +If set to literallower/literal or literalupper/literal, the
 +completed word will be in lower or upper case, respectively.  If set
 +to literalpreserve-lower/literal
 +or literalpreserve-upper/literal (the default), the completed 
 word
 +will be in the case of the word already entered, but words being
 +completed without anything entered will be in lower or upper case,
 +respectively.
 +/para
 +/listitem
 +  /varlistentry
 +
 +  varlistentry
  termvarnameDBNAME/varname/term
  listitem
  para
 diff --git i/src/bin/psql/tab-complete.c w/src/bin/psql/tab-complete.c
 index 6f481bb..00d87d5 100644
 --- i/src/bin/psql/tab-complete.c
 +++ w/src/bin/psql/tab-complete.c
 @@ -682,7 +682,7 @@ static char **complete_from_variables(char *text,
   const char *prefix, const char 
 *suffix);
  static char *complete_from_files(const char *text, int state);
  
 -static char *pg_strdup_same_case(const char *s, const char *ref);
 +static char *pg_strdup_keyword_case(const char *s, const char *ref);
  static PGresult *exec_query(const char *query);
  
  static void get_previous_words(int point, char **previous_words, int nwords);
 @@ -3048,7 +3048,7 @@ create_or_drop_command_generator(const char *text, int 
 state, bits32 excluded)
   {
   if ((pg_strncasecmp(name, text, string_length) == 0) 
   !(words_after_create[list_index - 1].flags  excluded))
 - return pg_strdup_same_case(name, text);
 + return pg_strdup_keyword_case(name, text);
   }
   /* if nothing matches, return NULL */
   return NULL;
 @@ -3335,9 +3335,9 @@ complete_from_list(const char *text, int state)
   if (completion_case_sensitive)
   return pg_strdup(item);
   else
 - /* If case insensitive matching was requested 
 initially, return
 -  * it in the case of what was already entered. 
 */
 - return pg_strdup_same_case(item, text);
 + /* If case insensitive matching was requested 
 initially, adjust
 +  * the case according to setting. */
 + return pg_strdup_keyword_case(item, text);
   }
   }
  
 @@ -3374,9 +3374,9 @@ complete_from_const(const char *text, int state)
   if (completion_case_sensitive)
   return pg_strdup(completion_charp);
   else
 - /* If case insensitive matching was requested 
 initially, return it
 -  * in the case of what was already entered. */
 - return pg_strdup_same_case(completion_charp, text);
 + /* If case insensitive matching was requested 
 initially, adjust the
 +  * case according to setting. */
 + return pg_strdup_keyword_case(completion_charp, text);
   }
   else
   return NULL;
 @@ -3484,27 +3484,48 @@ complete_from_files(const char *text, int state)
  
  
  /*
 - * Make a pg_strdup copy of s and convert it to the same case as ref.
 + * Make a pg_strdup copy of s and convert the case according to
 + * COMP_KEYWORD_CASE variable, using ref as the text that was already 
 entered.
   */
  static char *
 -pg_strdup_same_case(const char *s, const char *ref)
 +pg_strdup_keyword_case(const char *s, const char *ref)
  {
   char *ret, *p;
   unsigned char first = ref[0];
 + int tocase;
 + const char *varval;
 +
 + varval = GetVariable(pset.vars, COMP_KEYWORD_CASE);
 + 

Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-04-10 Thread Bruce Momjian
On Fri, Mar 30, 2012 at 08:16:59PM +0300, Peter Eisentraut wrote:
 On fre, 2012-03-23 at 07:52 -0700, David Fetter wrote:
  On Thu, Mar 22, 2012 at 06:05:30PM -0400, Andrew Dunstan wrote:
   On 03/22/2012 05:49 PM, Bruce Momjian wrote:
   Robert Haas and I are disappointed by this change.  I liked the
   fact that I could post nice-looking SQL queries without having to
   use my capslock key (which I use as a second control key).  Any
   chance of reverting this change?
   
   
   Should it be governed by a setting?
  
  Something like (upper|lower|preserve) ?
 
 How about this patch then?  (There are actually four possible settings,
 see patch.)
 

Yes, very nice.  I found the preserve modes confusing, but I see the
problem that saying preserve doesn't tell us how to expand a word
where we didn't type anything.

 +Determines which letter case to use when completing an SQL key word.
 +If set to literallower/literal or literalupper/literal, the
 +completed word will be in lower or upper case, respectively.  If set
 +to literalpreserve-lower/literal
 +or literalpreserve-upper/literal (the default), the completed 
 word
 +will be in the case of the word already entered, but words being
  -
The being word above got me confused;  I would remove it.

 +completed without anything entered will be in lower or upper case,
 +respectively.

without anything - with nothing ?

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-30 Thread Peter Eisentraut
On fre, 2012-03-23 at 07:52 -0700, David Fetter wrote:
 On Thu, Mar 22, 2012 at 06:05:30PM -0400, Andrew Dunstan wrote:
  On 03/22/2012 05:49 PM, Bruce Momjian wrote:
  Robert Haas and I are disappointed by this change.  I liked the
  fact that I could post nice-looking SQL queries without having to
  use my capslock key (which I use as a second control key).  Any
  chance of reverting this change?
  
  
  Should it be governed by a setting?
 
 Something like (upper|lower|preserve) ?

How about this patch then?  (There are actually four possible settings,
see patch.)

diff --git i/doc/src/sgml/ref/psql-ref.sgml w/doc/src/sgml/ref/psql-ref.sgml
index b849101..be9d37d 100644
--- i/doc/src/sgml/ref/psql-ref.sgml
+++ w/doc/src/sgml/ref/psql-ref.sgml
@@ -2652,6 +2652,22 @@ bar
   /varlistentry
 
   varlistentry
+termvarnameCOMP_KEYWORD_CASE/varname/term
+listitem
+para
+Determines which letter case to use when completing an SQL key word.
+If set to literallower/literal or literalupper/literal, the
+completed word will be in lower or upper case, respectively.  If set
+to literalpreserve-lower/literal
+or literalpreserve-upper/literal (the default), the completed word
+will be in the case of the word already entered, but words being
+completed without anything entered will be in lower or upper case,
+respectively.
+/para
+/listitem
+  /varlistentry
+
+  varlistentry
 termvarnameDBNAME/varname/term
 listitem
 para
diff --git i/src/bin/psql/tab-complete.c w/src/bin/psql/tab-complete.c
index 6f481bb..00d87d5 100644
--- i/src/bin/psql/tab-complete.c
+++ w/src/bin/psql/tab-complete.c
@@ -682,7 +682,7 @@ static char **complete_from_variables(char *text,
 		const char *prefix, const char *suffix);
 static char *complete_from_files(const char *text, int state);
 
-static char *pg_strdup_same_case(const char *s, const char *ref);
+static char *pg_strdup_keyword_case(const char *s, const char *ref);
 static PGresult *exec_query(const char *query);
 
 static void get_previous_words(int point, char **previous_words, int nwords);
@@ -3048,7 +3048,7 @@ create_or_drop_command_generator(const char *text, int state, bits32 excluded)
 	{
 		if ((pg_strncasecmp(name, text, string_length) == 0) 
 			!(words_after_create[list_index - 1].flags  excluded))
-			return pg_strdup_same_case(name, text);
+			return pg_strdup_keyword_case(name, text);
 	}
 	/* if nothing matches, return NULL */
 	return NULL;
@@ -3335,9 +3335,9 @@ complete_from_list(const char *text, int state)
 			if (completion_case_sensitive)
 return pg_strdup(item);
 			else
-/* If case insensitive matching was requested initially, return
- * it in the case of what was already entered. */
-return pg_strdup_same_case(item, text);
+/* If case insensitive matching was requested initially, adjust
+ * the case according to setting. */
+return pg_strdup_keyword_case(item, text);
 		}
 	}
 
@@ -3374,9 +3374,9 @@ complete_from_const(const char *text, int state)
 		if (completion_case_sensitive)
 			return pg_strdup(completion_charp);
 		else
-			/* If case insensitive matching was requested initially, return it
-			 * in the case of what was already entered. */
-			return pg_strdup_same_case(completion_charp, text);
+			/* If case insensitive matching was requested initially, adjust the
+			 * case according to setting. */
+			return pg_strdup_keyword_case(completion_charp, text);
 	}
 	else
 		return NULL;
@@ -3484,27 +3484,48 @@ complete_from_files(const char *text, int state)
 
 
 /*
- * Make a pg_strdup copy of s and convert it to the same case as ref.
+ * Make a pg_strdup copy of s and convert the case according to
+ * COMP_KEYWORD_CASE variable, using ref as the text that was already entered.
  */
 static char *
-pg_strdup_same_case(const char *s, const char *ref)
+pg_strdup_keyword_case(const char *s, const char *ref)
 {
 	char *ret, *p;
 	unsigned char first = ref[0];
+	int		tocase;
+	const char *varval;
+
+	varval = GetVariable(pset.vars, COMP_KEYWORD_CASE);
+	if (!varval)
+		tocase = 0;
+	else if (strcmp(varval, lower) == 0)
+		tocase = -2;
+	else if (strcmp(varval, preserve-lower) == 0)
+		tocase = -1;
+	else if (strcmp(varval, preserve-upper) == 0)
+		tocase = +1;
+	else if (strcmp(varval, upper) == 0)
+		tocase = +2;
+	else
+		tocase = 0;
 
-	if (isalpha(first))
-	{
-		ret = pg_strdup(s);
-		if (islower(first))
-			for (p = ret; *p; p++)
-*p = pg_tolower((unsigned char) *p);
-		else
-			for (p = ret; *p; p++)
-*p = pg_toupper((unsigned char) *p);
-		return ret;
-	}
+	/* default */
+	if (tocase == 0)
+		tocase = +1;
+
+	ret = pg_strdup(s);
+
+	if (tocase == -2
+		|| ((tocase == -1 || tocase == +1)  islower(first))
+		|| (tocase == -1  !isalpha(first))
+		)
+		for (p = ret; *p; p++)
+			*p = pg_tolower((unsigned char) *p);
 	else
-		return pg_strdup(s);
+		for (p = ret; *p; p++)
+			*p 

Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread Alvaro Herrera

Excerpts from Andrew Dunstan's message of jue mar 22 19:05:30 -0300 2012:
 
 On 03/22/2012 05:49 PM, Bruce Momjian wrote:
 
  Robert Haas and I are disappointed by this change.  I liked the fact
  that I could post nice-looking SQL queries without having to use my
  capslock key (which I use as a second control key).  Any chance of
  reverting this change?
 
 
 Should it be governed by a setting?

A \set variable perhaps?  +1  Would the old behavior be the default?

-- 
Álvaro Herrera alvhe...@commandprompt.com
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread David Fetter
On Thu, Mar 22, 2012 at 06:05:30PM -0400, Andrew Dunstan wrote:
 On 03/22/2012 05:49 PM, Bruce Momjian wrote:
 Robert Haas and I are disappointed by this change.  I liked the
 fact that I could post nice-looking SQL queries without having to
 use my capslock key (which I use as a second control key).  Any
 chance of reverting this change?
 
 
 Should it be governed by a setting?

Something like (upper|lower|preserve) ?

Cheers,
David.
-- 
David Fetter da...@fetter.org http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter  XMPP: david.fet...@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread David Fetter
On Fri, Mar 23, 2012 at 11:51:16AM -0300, Alvaro Herrera wrote:
 
 Excerpts from Andrew Dunstan's message of jue mar 22 19:05:30 -0300 2012:
  
  On 03/22/2012 05:49 PM, Bruce Momjian wrote:
  
   Robert Haas and I are disappointed by this change.  I liked the
   fact that I could post nice-looking SQL queries without having
   to use my capslock key (which I use as a second control key).
   Any chance of reverting this change?
  
  
  Should it be governed by a setting?
 
 A \set variable perhaps?  +1  Would the old behavior be the default?

+1 for defaulting to the old behavior.

Cheers,
David.
-- 
David Fetter da...@fetter.org http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter  XMPP: david.fet...@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread Peter Geoghegan
On 22 March 2012 22:05, Andrew Dunstan and...@dunslane.net wrote:


 On 03/22/2012 05:49 PM, Bruce Momjian wrote:


 Robert Haas and I are disappointed by this change.  I liked the fact
 that I could post nice-looking SQL queries without having to use my
 capslock key (which I use as a second control key).  Any chance of
 reverting this change?


 Should it be governed by a setting?

Perhaps, but I find the behaviour that was introduced by Peter's patch
to be a more preferable default.

-- 
Peter Geoghegan       http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training and Services

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread Andrew Dunstan



On 03/23/2012 11:07 AM, Peter Geoghegan wrote:

On 22 March 2012 22:05, Andrew Dunstanand...@dunslane.net  wrote:


On 03/22/2012 05:49 PM, Bruce Momjian wrote:


Robert Haas and I are disappointed by this change.  I liked the fact
that I could post nice-looking SQL queries without having to use my
capslock key (which I use as a second control key).  Any chance of
reverting this change?


Should it be governed by a setting?

Perhaps, but I find the behaviour that was introduced by Peter's patch
to be a more preferable default.



Upper casing SQL keywords is a common style, which is used in lots of 
our code (e.g. regression tests, psql queries, pg_dump). I think the 
default should match what is in effect our house style, and what we have 
historically done.


cheers

andrew



--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread Tom Lane
Peter Geoghegan pe...@2ndquadrant.com writes:
 On 22 March 2012 22:05, Andrew Dunstan and...@dunslane.net wrote:
 Should it be governed by a setting?

 Perhaps, but I find the behaviour that was introduced by Peter's patch
 to be a more preferable default.

FWIW, I like the new behavior better too.  I'm not particularly a
fan of all-caps.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-23 Thread Peter Geoghegan
On 23 March 2012 15:13, Andrew Dunstan and...@dunslane.net wrote:
 Upper casing SQL keywords is a common style, which is used in lots of our
 code (e.g. regression tests, psql queries, pg_dump). I think the default
 should match what is in effect our house style, and what we have
 historically done.

The code doesn't give preferential treatment to lower-case code - it
merely puts it on an even footing. I would agree with your position if
the change assumed that the user always wanted to use lower-case SQL,
but it does not. Rather, it intelligently infers what the user wants.

-- 
Peter Geoghegan       http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training and Services

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-22 Thread Andres Freund
On Thursday, March 22, 2012 10:49:55 PM Bruce Momjian wrote:
 Postgres 9.2 has been modified so psql no longer uppercases SQL keywords
 when using tab completation, by this commit:
 
   commit 69f4f1c3576abc535871c6cfa95539e32a36120f
   Author: Peter Eisentraut pete...@gmx.net
   Date:   Wed Feb 1 20:16:40 2012 +0200
 
   psql: Case preserving completion of SQL key words

 Robert Haas and I are disappointed by this change.  I liked the fact
 that I could post nice-looking SQL queries without having to use my
 capslock key (which I use as a second control key).  Any chance of
 reverting this change?
Seconded.

Andres

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Uppercase tab completion keywords in psql?

2012-03-22 Thread Andrew Dunstan



On 03/22/2012 05:49 PM, Bruce Momjian wrote:


Robert Haas and I are disappointed by this change.  I liked the fact
that I could post nice-looking SQL queries without having to use my
capslock key (which I use as a second control key).  Any chance of
reverting this change?



Should it be governed by a setting?

cheers

andrew

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers