Pull to refresh

Comments 12

На фото: третье исправление сверху, это как вообще? Снова Tab VS. Space?
Да и следующее Size*тпаолвптвыв*elementSize, что с этой строкой? Куда её ровняли?

Вот так поступают те кто в недавнем топике кричали Space используют трУсы не умеющие настивать IDE под себя?
Надо спросить коммиттера (Tom Lane), но скорее всего это автоматическое исправление утилитой pgindent. Официально в проекте есть один правильный способ форматировать код — с ее помощью. Но иногда во время разработки код форматируют «на глазок», а потом перед релизом комиттеры прогоняют pgindent по всему коду.
Ясно, спасибо что разъяснили, буду знать про pgindent.
Но вот здесь у меня тоже возник вопрос:
| char *qual_value;
| ParseState *qual_pstate = make_parsestate(NULL);
|
| /* parsestate is built just to build the range table */
| qual_pstate = make_parsestate(NULL);

пока не увидел в оригинале

| @@ -1081,7 +1081,7 @@ AlterPolicy(AlterPolicyStmt *stmt)
| if (!attr_isnull)
| {
| char *qual_value;
| — ParseState *qual_pstate = make_parsestate(NULL);
| + ParseState *qual_pstate;
|
| /* parsestate is built just to build the range table */
| qual_pstate = make_parsestate(NULL);
| @@ -1122,7 +1122,7 @@ AlterPolicy(AlterPolicyStmt *stmt)
| if (!attr_isnull)
| {
| char *with_check_value;
| — ParseState *with_check_pstate = make_parsestate(NULL);
| + ParseState *with_check_pstate;
|
| /* parsestate is built just to build the range table */
| with_check_pstate = make_parsestate(NULL);
Искренне извините за вертикальный слэш :(
Это невозможно не то что читать, даже видеть: глаза выпадают.
Впредь, никогда не буду так форматировать коммент!
Нередко опечатки появляются по той причине, что перед принятием патчей коммиттеры немного, совсем чуть-чуть, переписывают их. В результате получается совершенно новый код, который никто до этого не вычитывал.

А разве нельзя сделать так, чтобы при подаче коммита делался хеш, затем, при принятии, делался хеш и после принятия, если хеши не совпали, об этом сразу появлялось уведомление и можно было бы посмотреть разность и варианты?

Почему автора не записывают в поле "author" в коммите? Обидно как-то.

Если честно, я даже не знаю, как это делается и возможно ли в git. Наверное просто потому что неудобно, проще в commit message указать. Плюс иной раз ревьюверы и тестировщики делают больше, чем сам автор.

Автор и коммитер в git — отдельные сущности. При коммите можно указать другого автора.
Собственно это и было добавлено для того, что бы можно было указать истинного автора при пересылке патчей через почту.

и возможно ли в git

так делается, например, в pgadmin-е.


проще в commit message указать

да, но иногда забывают. потом в соседнем письме в pgsql-commiters указывают автора)

так делается, например, в pgadmin-е.

и в самом Git'е
Подскажите — есть ли встроенная функция, наподобие pg_reload_conf, или способ перезапускать службу СУБД из psql? (про "\!" я знаю, но хотелось бы какой-нибудь встроенный метод, не завязанный на расположение внешних файлов).
Sign up to leave a comment.