Re: [COMMITTERS] pgsql: Allow ALTER TABLE name {OF type | NOT OF}.

Cédric Villemain <cedric.villemain.debian@gmail.com>

From: Cédric Villemain <cedric.villemain.debian@gmail.com>

To: Alvaro Herrera <alvherre@commandprompt.com>

Cc: Robert Haas <rhaas@postgresql.org>, Pg Hackers <pgsql-hackers@postgresql.org>

Date: 2011-05-28T07:05:17Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Allow ALTER TABLE name {OF type | NOT OF}.
- 68739ba856c5 9.1.0 cited

2011/5/28 Alvaro Herrera <alvherre@commandprompt.com>:
> Excerpts from Cédric Villemain's message of vie may 27 18:37:05 -0400 2011:
>> 2011/4/21 Robert Haas <rhaas@postgresql.org>:
>
>> > Modified Files
>> > --------------
>> > doc/src/sgml/ref/alter_table.sgml         |   26 +++
>> > src/backend/commands/tablecmds.c          |  277 +++++++++++++++++++++++++++--
>>
>> I noticed 2 warnings on unused variables from gcc in tablecmds.c
>> The attached patch fix that by removing those 2 variables.
>
> My compiler wasn't complaining, but since the variable is clearly unused
> I went ahead and pushed this.  Thanks

I have a recent gcc wich probably help here:
gcc-4.6.real (Debian 4.6.0-7) 4.6.1 20110507 (prerelease)


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



-- 
Cédric Villemain               2ndQuadrant
http://2ndQuadrant.fr/     PostgreSQL : Expertise, Formation et Support