On Sun, Mar 12, 2023 at 08:59:44PM -0700, Andrey Borodin wrote:
> I've tried this approach, but could not come up with sufficiently
> different error messages...
>
>> Wouldn't it be better to have a couple of regression
>> tests, as well?
> Added two tests.
It should have three tests with one for ERANGE on top of the other
two. Passing down a value like "10e400" should be enough to cause
strtod() to fail, as far as I know.
+ if (sleep == 0)
+ continue;
While on it, forgot to comment on this one.. Indeed, this choice to
authorize 0 and not wait between two commands is more natural.
I have tweaked things as bit as of the attached, and ran pgindent.
What do you think?
--
Michael