On Tue, Jul 17, 2018 at 09:13:28PM -0400, Tom Lane wrote:
> Hm, I'd just tried this on my Mac laptop and saw only the one in
> walsender.c. No objection to the other two though, they might be
> relevant on other platforms.
longfin has turned back to green again, so at least this is fixed now.
--
Michael Paquier writes:
> This attempt made longfin happier for the path patched, but there were a
> couple of other ones which were hidden behind. I got them fixed now,
> which should bring the machine back to green.
Hm, I'd just tried this on my Mac laptop and saw only the one in
walsender.c.
On Wed, Jul 18, 2018 at 09:14:14AM +0900, Michael Paquier wrote:
> Thanks for double-checking, pushed.
This attempt made longfin happier for the path patched, but there were a
couple of other ones which were hidden behind. I got them fixed now,
which should bring the machine back to green.
--
Mic
On Tue, Jul 17, 2018 at 08:02:43PM -0400, Tom Lane wrote:
> Sounds right to me.
Thanks for double-checking, pushed.
--
Michael
signature.asc
Description: PGP signature
Michael Paquier writes:
> On Tue, Jul 17, 2018 at 07:23:04PM -0400, Tom Lane wrote:
>> longfin thinks you missed at least one cast ...
> Hm, I checked that. Just casting to (Size) should do it.
Sounds right to me.
regards, tom lane
On Tue, Jul 17, 2018 at 07:23:04PM -0400, Tom Lane wrote:
> Michael Paquier writes:
> > Rework error messages around file handling
>
> longfin thinks you missed at least one cast ...
Hm, I checked that. Just casting to (Size) should do it. I cannot get
a warning with neither clang nor gcc on D
Michael Paquier writes:
> Rework error messages around file handling
longfin thinks you missed at least one cast ...
regards, tom lane
Rework error messages around file handling
Some error messages related to file handling are using the code path
context to define their state. For example, 2PC-related errors are
referring to "two-phase status files", or "relation mapping file" is
used for catalog-to-filenode mapping, however tho