Re: system catalog relation of a table and a serial sequence - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: system catalog relation of a table and a serial sequence
Date
Msg-id 200203080055.g280tgx23186@candle.pha.pa.us
Whole thread Raw
In response to Re: system catalog relation of a table and a serial sequence  (Brent Verner <brent@rcfile.org>)
List pgsql-patches
Brent, do you have a new, final patch that you want to submit for this?


---------------------------------------------------------------------------

Brent Verner wrote:
> [2001-12-16 06:30] Brent Verner said:
> | [2001-12-16 00:42] Tom Lane said:
> | | Brent Verner <brent@rcfile.org> writes:
> | | > How about strtok_r?  I /really/ like the fact that strtok will
> | | > eat either of the tokens ['"] that might be around the sequence
> | | > name... just call me lazy :-)
> | |
> | | That behavior creates one of the "corner cases" I was alluding to.
> | | Shall I leave the difficulty as an exercise for the student?
> |
> | Ok... I ended up working longer than I'd thought :-)
> |
> | * no strtok were used in this patch. ;-)
> | * Handles both serial-sequences and user-sequences referenced in
> |   nextval(...) default column defs.
> | * Loop over tables so we can check wildcard table name in the future
> |   per your suggestion.  I've only noted a TODO: regarding the wildcard
> |   matching.
> | * Instead of using a NULL terminated char** array to hold the collected
> |   sequence names, I put in a simple strarray ADT -- mostly so I could
> |   have the strarrayContains() test to call from the conditional around
> |   dumpSequence().  If this is just dumb, I'll replace it with a simple
> |   char** implementation.  Did I overlook some utility funcs in the
> |   PG source that already does this?  If so, I'll gladly use those.
> | * Patch is really attached :-P
>
> This patch needs a fix already...  I just realized (while playing with
> this code in a different context) that I forgot to change the malloc
> line in strarrayInit() after typedef'ing strarray as pointer to struct,
> instead of just the struct.
>
> -  strarray _ary = (strarray)malloc(sizeof(strarray));
> +  strarray _ary = (strarray)malloc(sizeof(struct strarray));
>
> cheers.
>   brent
>
> --
> "Develop your talent, man, and leave the world something. Records are
> really gifts from people. To think that an artist would love you enough
> to share his music with anyone is a beautiful thing."  -- Duane Allman
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo@postgresql.org so that your
> message can get through to the mailing list cleanly
>

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

pgsql-patches by date:

Previous
From: Neil Conway
Date:
Subject: Re: Basic DOMAIN Support
Next
From: "Rod Taylor"
Date:
Subject: Re: Basic DOMAIN Support