Re: tablecmds.c/MergeAttributes() cleanup

On 29.08.23 19:45, Nathan Bossart wrote:
> On Tue, Aug 29, 2023 at 10:43:39AM +0200, Peter Eisentraut wrote:
>> I have committed a few more patches from this series that were already
>> agreed upon. The remaining ones are rebased and reordered a bit, attached.
>
> My compiler is complaining about 1fa9241b:
>
> ../postgresql/src/backend/commands/sequence.c: In function ‘DefineSequence’:
> ../postgresql/src/backend/commands/sequence.c:196:21: error: ‘coldef’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
> 196 | stmt->tableElts = lappend(stmt->tableElts, coldef);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> This went away when I added a default case that ERROR'd or initialized
> coldef to NULL.

fixed