Re: UniqueKey v2 - Mailing list pgsql-hackers

From zhihuifan1213@163.com
Subject Re: UniqueKey v2
Date
Msg-id 7mla7cnmue6q.fsf@e18c07352.et15sqa
Whole thread Raw
In response to Re: UniqueKey v2  (jian he <jian.universality@gmail.com>)
Responses Re: UniqueKey v2
List pgsql-hackers
jian he <jian.universality@gmail.com> writes:

Hi jian,

> hi.
> After `git am`, I still cannot build.
>
> ../../Desktop/pg_sources/main/postgres/src/backend/optimizer/path/uniquekey.c:125:45:
> error: variable ‘var’ set but not used
> [-Werror=unused-but-set-variable]
>   125 |                         Var                *var;
>       |                                             ^~~

Thanks for this report, looks clang 11 can't capture this error.  I have
switched to clang 17 which would report this issue at the first place.

>
> You also need to change src/backend/optimizer/path/meson.build.

Great thanks.

>
> git apply failed.
>
> git am warning:
> Applying: uniquekey on base relation and used it for mark-distinct-as-op.
> .git/rebase-apply/patch:876: new blank line at EOF.
> +
> warning: 1 line adds whitespace errors.
>
> I think you can use `git diff --check`
> (https://git-scm.com/docs/git-diff) to check for whitespace related
> errors.

thanks for the really good suggestion.  Here is the newer version:



-- 
Best Regards
Andy Fan

Attachment

pgsql-hackers by date:

Previous
From: Jeff Davis
Date:
Subject: Re: broken master regress tests
Next
From: Jeff Davis
Date:
Subject: Re: Pre-proposal: unicode normalized text