Re: Bogus sizing parameters in some AllocSetContextCreate calls

Robert Haas <robertmhaas@gmail.com> writes:
> Also, I think we ought to replace this code in aset.c:

>     initBlockSize = MAXALIGN(initBlockSize);
>     if (initBlockSize < 1024)
>         initBlockSize = 1024;
>     maxBlockSize = MAXALIGN(maxBlockSize);

> With this:

>     Assert(initBlockSize >= 1024 && initBlockSize == MAXALIGN(initBlockSize));
>     Assert(maxBlockSize == MAXALIGN(maxBlockSize));

Good idea --- if we'd had it that way, these errors would never have
gotten committed in the first place.  I'm for doing that only in HEAD
though.
        regards, tom lane