Re: Add --check option to pgindent

Поиск
Список
Период
Сортировка
От Tristan Partin
Тема Re: Add --check option to pgindent
Дата
Msg-id CXP0TUOU4C1N.Z7Q508WQ4XJP@neon.tech
обсуждение исходный текст
Ответ на Re: Add --check option to pgindent  (Jelte Fennema-Nio <postgres@jeltef.nl>)
Ответы Re: Add --check option to pgindent  (Daniel Gustafsson <daniel@yesql.se>)
Список pgsql-hackers
On Fri Dec 15, 2023 at 8:18 AM CST, Jelte Fennema-Nio wrote:
> This part of the first patch seems incorrect, i.e. same condition in
> the if as elsif
>
> -       if ($silent_diff)
> +       if ($check)
> +       {
> +           print show_diff($source, $source_filename);
> +           exit 2;
> +       }
> +       elsif ($check)
>         {
>             exit 2;
>         }

Weird how I was able to screw that up so bad :). Thanks! Here is a v3.
--
Tristan Partin
Neon (https://neon.tech)

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Josef Šimánek
Дата:
Сообщение: Re: [PATCH] Add --syntax to postgres for SQL syntax checking
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Clean up find_typedefs and add support for Mac