Return-Path: X-Original-To: notmuch@notmuchmail.org Delivered-To: notmuch@notmuchmail.org Received: from localhost (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id 9321E4196F2 for ; Tue, 13 Apr 2010 08:36:07 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: -2.89 X-Spam-Level: X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5 tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01] autolearn=ham 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 BvXboGAabGJN; Tue, 13 Apr 2010 08:36:07 -0700 (PDT) Received: from yoom.home.cworth.org (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id DD01F431FC1; Tue, 13 Apr 2010 08:36:06 -0700 (PDT) Received: by yoom.home.cworth.org (Postfix, from userid 1000) id 755FD5883A6; Tue, 13 Apr 2010 08:36:06 -0700 (PDT) From: Carl Worth To: Gregor Hoffleit , notmuch Subject: Re: [notmuch] [PATCH] json_quote_str should handle non-ASCII characters In-Reply-To: <1267697893-sup-4538@sam.mediasupervision.de> References: <1267697893-sup-4538@sam.mediasupervision.de> Date: Tue, 13 Apr 2010 08:36:06 -0700 Message-ID: <87r5mje55l.fsf@yoom.home.cworth.org> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature" X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.13 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: Tue, 13 Apr 2010 15:36:07 -0000 --=-=-= Content-Transfer-Encoding: quoted-printable On Thu, 04 Mar 2010 11:40:03 +0100, Gregor Hoffleit wr= ote: > The current code in json_quote_str() only accepts strict printable ASCII > code points (i.e. 32-127), all other code points are dropped from the > JSON output. >=20 > This patch accepts code points 32-255. Thanks, Gregor! I've pushed this patch out now. And I noted in our TODO file that we really should add some tests to the test suite to exercise our --format=3Djson code paths. It would be nice to ensure we don't have any regressions in this area later. =2DCarl --=-=-= Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iD8DBQFLxI9m6JDdNq8qSWgRAunuAJ0bt1/+r0wREeN91xaPBIaCQR4KuQCeN3da optKGG7AVQ6vq9Uepc/slaQ= =2Csh -----END PGP SIGNATURE----- --=-=-=--