Re: [PATCH] ALTER tab completion

Hi,

Thank you for the comments!

I made following updates:

>> -ALTER DEFAULT PRIVILEGES: missing FOR USER
> 
> FOR ROLE is an equivalent.  That does not seem mandatory to me.
I deleted the completion for "FOR USER".

>> -ALTER VIEW: no completion after ALTER COLUMN column_name
> +   /* ALTER VIEW xxx ALTER yyy */
> +   else if (Matches("ALTER", "VIEW", MatchAny, "ALTER", MatchAny))
> +       COMPLETE_WITH("SET DEFAULT", "DROP DEFAULT");
> It may be cleaner to group this one with "ALTER VIEW xxx ALTER yyy"
> two blocks above.
I put them back to back so that it looks cleaner.

>> -ALTER SEQUENCE: missing AS
> +   /* ALTER SEQUENCE <name> AS */
> +   else if (TailMatches("ALTER", "SEQUENCE", MatchAny, "AS"))
> +       COMPLETE_WITH("smallint", "integer", "bigint");
> Re-quoting Horiguchi-san, that should be COMPLETE_WITH_CS() to keep
> these completions in lower case.
That's what it's for.
I used COMPLETE_WITH_CS instead of COMPLETE_WITH.

Best wishes,

-- 
Ken Kato
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION