[prev in list] [next in list] [prev in thread] [next in thread] 

List:       postgresql-general
Subject:    Re: \if, \elseif, \else, \endif (was Re: [HACKERS] PSQL commands: \quit_if, \quit_unless)
From:       Corey Huinker <corey.huinker () gmail ! com>
Date:       2017-01-31 17:16:51
Message-ID: CADkLM=c8M9LDBdvijq=yXrS4ar2KLoq7uoYs0JQFmh0d3w2hmg () mail ! gmail ! com
[Download RAW message or body]

On Tue, Jan 31, 2017 at 1:04 AM, Fabien COELHO <coelho@cri.ensmp.fr> wrote:

>
> This is code lifted from variable.c's ParseVariableBool().  When the other
>>> patch for "psql hooks" is committed (the one that detects when the string
>>> wasn't a valid boolean), this code will go away and we'll just use
>>> ParseVariableBool() again.
>>>
>>
> The ParseVariableBool function has been updated, and the new version is
> much cleaner, including all fixes that I suggested in your copy, so you can
> use it in your patch.
>
> --
> Fabien.
>

I see there's still a lot of activity in the thread, I can't tell if it's
directly related to ParseVariableBool() or in the way it is called. Should
I wait for the dust to settle over there?

[Attachment #3 (text/html)]

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 31, 2017 \
at 1:04 AM, Fabien COELHO <span dir="ltr">&lt;<a href="mailto:coelho@cri.ensmp.fr" \
target="_blank">coelho@cri.ensmp.fr</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><span class=""><br> <blockquote class="gmail_quote" \
style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> This is code lifted from variable.c&#39;s \
ParseVariableBool().   When the other<br> patch for &quot;psql hooks&quot; is \
committed (the one that detects when the string<br> wasn&#39;t a valid boolean), this \
code will go away and we&#39;ll just use<br> ParseVariableBool() again.<br>
</blockquote></blockquote>
<br></span>
The ParseVariableBool function has been updated, and the new version is much cleaner, \
including all fixes that I suggested in your copy, so you can use it in your \
patch.<span class="HOEnZb"><font color="#888888"><br> <br>
-- <br>
Fabien.<br>
</font></span></blockquote></div><br>I see there&#39;s still a lot of activity in the \
thread, I can&#39;t tell if it&#39;s directly related to ParseVariableBool() or in \
the way it is called. Should I wait for the dust to settle over \
there?<br><br><br></div></div>



[prev in list] [next in list] [prev in thread] [next in thread]