From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <cworth@cworth.org>
Received: from localhost (localhost [127.0.0.1])
	by olra.theworths.org (Postfix) with ESMTP id 4A34F431FC0;
	Wed, 18 Nov 2009 14:27:37 -0800 (PST)
X-Virus-Scanned: Debian amavisd-new at olra.theworths.org
Received: from olra.theworths.org ([127.0.0.1])
	by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id 2H3gZ5qNkzph; Wed, 18 Nov 2009 14:27:36 -0800 (PST)
Received: from cworth.org (localhost [127.0.0.1])
	by olra.theworths.org (Postfix) with ESMTP id E3B13431FBC;
	Wed, 18 Nov 2009 14:27:35 -0800 (PST)
From: Carl Worth <cworth@cworth.org>
To: Jjgod Jiang <gzjjgod@gmail.com>, notmuch@notmuchmail.org
In-Reply-To: <ddd65cda0911171950o4eea4389v86de9525e46052d3@mail.gmail.com>
References: <ddd65cda0911171950o4eea4389v86de9525e46052d3@mail.gmail.com>
Date: Wed, 18 Nov 2009 23:27:20 +0100
Message-ID: <874oorfozb.fsf@yoom.home.cworth.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: Mac OS X/Darwin compatibility issues
X-BeenThere: notmuch@notmuchmail.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "Use and development of the notmuch mail system."
	<notmuch.notmuchmail.org>
List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,
	<mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>
List-Archive: <http://notmuchmail.org/pipermail/notmuch>
List-Post: <mailto:notmuch@notmuchmail.org>
List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>
List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,
	<mailto:notmuch-request@notmuchmail.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Nov 2009 22:27:37 -0000

On Wed, 18 Nov 2009 11:50:17 +0800, Jjgod Jiang <gzjjgod@gmail.com> wrote:
> 1. g++ reports 'warning: command line option "-Wmissing-declarations"
> is valid for C/ObjC but not for C++'

In cairo, what we do is have a huge list of desired warnings and then
test each one in turn with a test-compile in the configure script.
That's probably what we should do here, (I recently established a
Makefile.config for this purpose that should be generated by the
configure script.)

> 2.
> notmuch-reply.c: In function =E2=80=98address_is_users=E2=80=99:
> notmuch-reply.c:87: warning: passing argument 2 of
> =E2=80=98notmuch_config_get_user_other_email=E2=80=99 from incompatible p=
ointer type

Thanks for the report. I think I saw a fix for this in a commit from
Chris Wilson that's in my queue. (Though from a quick glance I'm not
sure where the size_t type got started in the code and propagated
throught this part).

> 3. Several errors about missing GNU extensions like getline() and strndup=
():
>=20
> warning: implicit declaration of function =E2=80=98getline=E2=80=99
> error: =E2=80=98strndup=E2=80=99 was not declared in this scope
>=20
> We can implement these with fgets() and strncpy() though.

Yes. I knew I was "cheating" by using some GNU extensions here. I'm
happy to accept portability patches for these things, but it's hard for
me to get excited about writing them myself.

Care to take a whack at these?

-Carl