[2849] in BarnOwl Developers

home help back first fref pref prev next nref lref last post

Re: [PATCH] Don't use error_message if we are building without Zephyr.

daemon@ATHENA.MIT.EDU (Nelson Elhage)
Tue Jan 4 11:58:00 2011

In-Reply-To: <alpine.DEB.1.10.1101041113040.15170@dr-wily.mit.edu>
Date: Tue, 4 Jan 2011 11:57:58 -0500
From: Nelson Elhage <nelhage@nelhage.com>
To: David A Benjamin <davidben@mit.edu>
Cc: Nelson Elhage <nelhage@mit.edu>, barnowl-dev@mit.edu,
        Nelson Elhage <nelhage@ksplice.com>

--000e0cd32cf0ac1a8e0499082b12
Content-Type: text/plain; charset=UTF-8

Oh hey. I had "#ifdef it out", but the # ended up at start-of-line and was
interpreted as a comment.
On Jan 4, 2011 11:13 AM, "David A Benjamin" <davidben@mit.edu> wrote:
> LGTM.
>
> (I assume the commit message was supposed to be longer? :-D)
>
> David
>
> On Tue, 4 Jan 2011, Nelson Elhage wrote:
>
>> That code path will never actually be executed if we don't have Zephyr,
so let's
>> ---
>> functions.c | 5 +++++
>> 1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/functions.c b/functions.c
>> index 1f2aa12..2619977 100644
>> --- a/functions.c
>> +++ b/functions.c
>> @@ -375,6 +375,7 @@ void owl_callback_zwrite(owl_editwin *e) {
>> /* send, log and display an outgoing zephyr. If 'msg' is NULL
>> * the message is expected to be set from the zwrite line itself
>> */
>> +#ifdef HAVE_LIBZEPHYR
>> void owl_function_zwrite(owl_zwrite *z, const char *msg)
>> {
>> owl_message *m;
>> @@ -409,6 +410,10 @@ void owl_function_zwrite(owl_zwrite *z, const char
*msg)
>> }
>> }
>> }
>> +#else
>> +void owl_function_zwrite(owl_zwrite *z, const char *msg) {
>> +}
>> +#endif
>>
>> /* send, log and display an outgoing zcrypt zephyr. If 'msg' is NULL
>> * the message is expected to be set from the zwrite line itself
>> --
>> 1.7.3.4
>>
>>

--000e0cd32cf0ac1a8e0499082b12
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p>Oh hey. I had &quot;#ifdef it out&quot;, but the # ended up at start-of-=
line and was interpreted as a comment.</p>
<div class=3D"gmail_quote">On Jan 4, 2011 11:13 AM, &quot;David A Benjamin&=
quot; &lt;<a href=3D"mailto:davidben@mit.edu">davidben@mit.edu</a>&gt; wrot=
e:<br type=3D"attribution">&gt; LGTM.<br>&gt; <br>&gt; (I assume the commit=
 message was supposed to be longer? :-D)<br>
&gt; <br>&gt; David<br>&gt; <br>&gt; On Tue, 4 Jan 2011, Nelson Elhage wrot=
e:<br>&gt; <br>&gt;&gt; That code path will never actually be executed if w=
e don&#39;t have Zephyr, so let&#39;s<br>&gt;&gt; ---<br>&gt;&gt; functions=
.c |    5 +++++<br>
&gt;&gt; 1 files changed, 5 insertions(+), 0 deletions(-)<br>&gt;&gt;<br>&g=
t;&gt; diff --git a/functions.c b/functions.c<br>&gt;&gt; index 1f2aa12..26=
19977 100644<br>&gt;&gt; --- a/functions.c<br>&gt;&gt; +++ b/functions.c<br=
>
&gt;&gt; @@ -375,6 +375,7 @@ void owl_callback_zwrite(owl_editwin *e) {<br>=
&gt;&gt; /* send, log and display an outgoing zephyr.  If &#39;msg&#39; is =
NULL<br>&gt;&gt;  * the message is expected to be set from the zwrite line =
itself<br>
&gt;&gt;  */<br>&gt;&gt; +#ifdef HAVE_LIBZEPHYR<br>&gt;&gt; void owl_functi=
on_zwrite(owl_zwrite *z, const char *msg)<br>&gt;&gt; {<br>&gt;&gt;   owl_m=
essage *m;<br>&gt;&gt; @@ -409,6 +410,10 @@ void owl_function_zwrite(owl_zw=
rite *z, const char *msg)<br>
&gt;&gt;     }<br>&gt;&gt;   }<br>&gt;&gt; }<br>&gt;&gt; +#else<br>&gt;&gt;=
 +void owl_function_zwrite(owl_zwrite *z, const char *msg) {<br>&gt;&gt; +}=
<br>&gt;&gt; +#endif<br>&gt;&gt;<br>&gt;&gt; /* send, log and display an ou=
tgoing zcrypt zephyr.  If &#39;msg&#39; is NULL<br>
&gt;&gt;  * the message is expected to be set from the zwrite line itself<b=
r>&gt;&gt; -- <br>&gt;&gt; 1.7.3.4<br>&gt;&gt;<br>&gt;&gt;<br></div>

--000e0cd32cf0ac1a8e0499082b12--

home help back first fref pref prev next nref lref last post