Comments 12
На фото: третье исправление сверху, это как вообще? Снова Tab VS. Space?
Да и следующее Size*тпаолвптвыв*elementSize, что с этой строкой? Куда её ровняли?
Вот так поступают те кто в недавнем топике кричали Space используют трУсы не умеющие настивать IDE под себя?
Да и следующее Size*тпаолвптвыв*elementSize, что с этой строкой? Куда её ровняли?
Вот так поступают те кто в недавнем топике кричали Space используют трУсы не умеющие настивать IDE под себя?
0
Надо спросить коммиттера (Tom Lane), но скорее всего это автоматическое исправление утилитой pgindent. Официально в проекте есть один правильный способ форматировать код — с ее помощью. Но иногда во время разработки код форматируют «на глазок», а потом перед релизом комиттеры прогоняют pgindent по всему коду.
+1
Ясно, спасибо что разъяснили, буду знать про 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);
Но вот здесь у меня тоже возник вопрос:
| 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);
0
Нередко опечатки появляются по той причине, что перед принятием патчей коммиттеры немного, совсем чуть-чуть, переписывают их. В результате получается совершенно новый код, который никто до этого не вычитывал.
А разве нельзя сделать так, чтобы при подаче коммита делался хеш, затем, при принятии, делался хеш и после принятия, если хеши не совпали, об этом сразу появлялось уведомление и можно было бы посмотреть разность и варианты?
0
Почему автора не записывают в поле "author" в коммите? Обидно как-то.
+3
Если честно, я даже не знаю, как это делается и возможно ли в git. Наверное просто потому что неудобно, проще в commit message указать. Плюс иной раз ревьюверы и тестировщики делают больше, чем сам автор.
0
Автор и коммитер в git — отдельные сущности. При коммите можно указать другого автора.
Собственно это и было добавлено для того, что бы можно было указать истинного автора при пересылке патчей через почту.
+2
и возможно ли в git
так делается, например, в pgadmin-е.
проще в commit message указать
да, но иногда забывают. потом в соседнем письме в pgsql-commiters указывают автора)
+1
Подскажите — есть ли встроенная функция, наподобие pg_reload_conf, или способ перезапускать службу СУБД из psql? (про "\!" я знаю, но хотелось бы какой-нибудь встроенный метод, не завязанный на расположение внешних файлов).
0
Sign up to leave a comment.
Контрибьютим в PostgreSQL: примеры реальных патчей, часть 1 из N