> On 21 Feb 2022, at 09:30, Kyotaro Horiguchi <horikyota....@gmail.com> wrote:

> (However, I got a mysterious -Wmisleading-indentation warning with this..)
> 
>> describe.c: In function ‘describeOneTableDetails’:
>> describe.c:3420:5: warning: this ‘if’ clause does not guard... 
>> [-Wmisleading-indentation]
>>     if (tableinfo.relam)
>>     ^~
>> describe.c:3423:2: note: ...this statement, but the latter is misleadingly 
>> indented as if it were guarded by the ‘if’
>>  if (res)
>  ^~

I think it's because you've indented your new code differently from the
existing, such that the if (res) clause is indented equally to the previous
pg_free(tableinfo.relam) call, making it look like they are in the same block:

> +    if (tableinfo.relam)
> +        pg_free(tableinfo.relam);
> 
>       if (res)
>               PQclear(res);


--
Daniel Gustafsson               https://vmware.com/



Reply via email to