Re: Add header support to text format and matching feature - Mailing list pgsql-hackers

From Rémi Lapeyre
Subject Re: Add header support to text format and matching feature
Date
Msg-id 4009F36F-7D63-4E76-8CFE-776A5ED71BB9@lenstra.fr
Whole thread Raw
In response to Re: Add header support to text format and matching feature  (vignesh C <vignesh21@gmail.com>)
Responses Re: Add header support to text format and matching feature  ("Daniel Verite" <daniel@manitou-mail.org>)
List pgsql-hackers
Thanks for the feedback,

> There is one warning present in the changes:
> copy.c: In function ‘ProcessCopyOptions’:
> copy.c:1208:5: warning: ISO C90 forbids mixed declarations and code
> [-Wdeclaration-after-statement]
>     char    *sval = defGetString(defel);
>

Weirdly this is not caught by clang, even with -Wdeclaration-after-statement.

Here’s a new version that fix all the issues.

Rémi


Attachment

pgsql-hackers by date:

Previous
From: Ranier Vilela
Date:
Subject: CID 1428952 (#1 of 1): Out-of-bounds access (OVERRUN) (src/backend/commands/async.c)
Next
From: Rémi Lapeyre
Date:
Subject: Re: WIP: System Versioned Temporal Table