RE: [PATCH] Style: fix function declaration

Поиск
Список
Период
Сортировка
От Ranier Vilela
Тема RE: [PATCH] Style: fix function declaration
Дата
Msg-id BN8PR18MB291654BFF79E6CCC6E3A6818E34A0@BN8PR18MB2916.namprd18.prod.outlook.com
обсуждение исходный текст
Ответ на [PATCH] Style: fix function declaration  (Ranier Vilela <ranier_gyn@hotmail.com>)
Список pgsql-hackers
Hi,

>I see no problem with fixing this kind of inconsistency for
>readability, so applied the change.
Thank you.

>Anyway, when sending a patch there are a couple of things which can
>make the life of people looking at what you send easier:
>https://wiki.postgresql.org/wiki/Submitting_a_Patch
Yes, I will read.

>One problem that I noted with the patch sent on this thread is that it
>does not directly apply on the git repository.  Folks on -hackers are
>mainly used to diffs generated by git.  So first I would recommend
>that you set up a git repository of the tree, say with that:
>git clone https://git.postgresql.org/git/postgresql.git
I will make use.

>Generating a patch can be done with git in a couple of ways from the
>cloned repository, say:
>1) git diff
>2) git format-patch
>Both can be applied with a simple "patch -p1" command or even the more
>advanced "git am", still the latter is kind of picky.
Thanks for the hints.

>The code of Postgres is complex, so usually there are reasons why
>things are done the way they are, and it is important to not be afraid
>to ask questions.  Also, making the subject of the emails you send
>explicative enough is important.  Please note pgsql-hackers has a lot
>of traffic, and this helps some people in filtering out threads they
>are not interested in.
Well, my experience with pgsql-hackers, haven't been good.
Ask questions, about the code, have not had good acceptance..

Best regards.
Ranier Vilela

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

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: [PATCH] Style: fix function declaration
Следующее
От: Kyotaro Horiguchi
Дата:
Сообщение: Re: [HACKERS] WAL logging problem in 9.4.3?