Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:62308 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 81870 invoked from network); 20 Aug 2012 20:04:04 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 20 Aug 2012 20:04:04 -0000 Authentication-Results: pb1.pair.com header.from=lars@strojny.net; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=lars@strojny.net; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain strojny.net from 46.4.40.248 cause and error) X-PHP-List-Original-Sender: lars@strojny.net X-Host-Fingerprint: 46.4.40.248 milch.schokokeks.org Received: from [46.4.40.248] ([46.4.40.248:56572] helo=milch.schokokeks.org) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 68/D6-07742-23892305 for ; Mon, 20 Aug 2012 16:04:03 -0400 Received: from [192.168.178.32] (ppp-188-174-111-99.dynamic.mnet-online.de [::ffff:188.174.111.99]) (AUTH: PLAIN lars@schokokeks.org, TLS: TLSv1/SSLv3,128bits,AES128-SHA) by milch.schokokeks.org with ESMTPSA; Mon, 20 Aug 2012 22:03:59 +0200 id 0000000000000022.000000005032982F.0000091A Content-Type: text/plain; charset=windows-1252 Mime-Version: 1.0 (Mac OS X Mail 6.0 \(1485\)) In-Reply-To: <50328249.2020600@sugarcrm.com> Date: Mon, 20 Aug 2012 22:03:59 +0200 Cc: PHP Internals Content-Transfer-Encoding: quoted-printable Message-ID: <2639AB42-2103-471B-948A-6860C310482E@strojny.net> References: <50328249.2020600@sugarcrm.com> To: Stas Malyshev X-Mailer: Apple Mail (2.1485) Subject: Re: [PHP-DEV] Merging fix for quoted_printable_encode() From: lars@strojny.net (Lars Strojny) Hi Stas, I would prefer it int 5.3 as well as it might break mailing systems = using quoted_printable_encode(). I=92ll add the test from the comments = as well. Am 20.08.2012 um 20:30 schrieb Stas Malyshev : > Yes. For 5.3, it does not look like a critical bug, so it shouldn't be > there. Also, the tests are still not there - so they should be added > before the merge can happen.