Committing patches sent by Florian Helmberger. Here are his remarks:
authorBen Faga <faga@cshl.edu>
Mon, 4 Jun 2007 04:01:14 +0000 (04:01 +0000)
committerBen Faga <faga@cshl.edu>
Mon, 4 Jun 2007 04:01:14 +0000 (04:01 +0000)
commit7ed7402c72e81504c8acd4f1bba4837dc69390f0
tree8254af7e22abd7f4648825e82761bbbf87310f5f
parent027cebc7f50e5c669d26f5ed76a0c1dc4f4c8861
Committing patches sent by Florian Helmberger.  Here are his remarks:

Here are two patches for a few bugs in the PostgreSQL parser and producer.

The patch for lib/SQL/Translator/Parser/DBI/PostgreSQL.pm fixes a simple
typo which resulted in the complete absence of the field types and
corrects the way is_nullable is set so the producer now correctly flags
NOT NULL fields.

The patch for lib/SQL/Translator/Producer/PostgreSQL.pm adds missing
newlines to the last line of the 'Table:' and 'Comments:' headers. Also
it fixes an 'use of uninizialised value' warning in 359.

Before the patch the output for command

sqlt --from DBI --dsn dbi:Pg:dbname=testdb --to PostgreSQL \
--db-user postgres

looked like:

====
--
-- Table: mailaliases
--CREATE TABLE "mailaliases" (
"aliasname" (204),
"aliasto" (204),
"domain" (132),
"id" ,
"isdefault" ,
"datecreate" ,
Constraint "foo" UNIQUE ("isdefault")
);
====

After the patch it looks like:

====
--
-- Table: mailaliases
--
CREATE TABLE "mailaliases" (
"aliasname" character varying(204),
"aliasto" character varying(204),
"domain" character varying(132),
"id" int4 NOT NULL,
"isdefault" int4,
"datecreate" text,
Constraint "foo" UNIQUE ("isdefault")
);
====
lib/SQL/Translator/Parser/DBI/PostgreSQL.pm
lib/SQL/Translator/Producer/PostgreSQL.pm