Re: [HACKERS] Missing newlines in error messages
От | Magnus Hagander |
---|---|
Тема | Re: [HACKERS] Missing newlines in error messages |
Дата | |
Msg-id | CABUevEzwmv5bvxTiE8nTkL2NLabf-nrXR3LTZn5K3sOvhONyVw@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [HACKERS] Missing newlines in error messages (Tom Lane <tgl@sss.pgh.pa.us>) |
Список | pgsql-hackers |
On Thu, Dec 15, 2016 at 4:38 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Magnus Hagander <magnus@hagander.net> writes:
> While poking around the pg_basebackup code, I noticed there are a lot of
> error messages are missing the \n at the end. Most of them are very
> unlikely to happen, but in general I believe all error messages we print to
> stderr from those binaries should have a newline at the end.
> Is that wrong? :)
I think you missed the fact that PQerrorMessage's result will already
end with a newline. So most (not all) of these look OK as they stand.
Ha, I knew there was something. But somehow I forgot about that one. I was wondering why nobody had found it before :)
And yeah, that leaves just the one error that I actually bumped into. I'll go ahead and apply that one alone then.
Pah. Less coding on jetlag.
Thanks!
В списке pgsql-hackers по дате отправления: