From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 To: Jjgod Jiang , notmuch@notmuchmail.org In-Reply-To: References: 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." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Nov 2009 22:27:37 -0000 On Wed, 18 Nov 2009 11:50:17 +0800, Jjgod Jiang 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