Newsgroups: php.internals,php.internals Path: news.php.net Xref: news.php.net php.internals:105434 php.internals:105435 Return-Path: Delivered-To: mailing list internals@lists.php.net Received: (qmail 42250 invoked from network); 25 Apr 2019 10:56:58 -0000 Received: from unknown (HELO smtp.opensides.be) (195.154.20.141) by pb1.pair.com with SMTP; 25 Apr 2019 10:56:58 -0000 Received: from localhost (localhost [127.0.0.1]) by smtp.opensides.be (Postfix) with ESMTP id 60BF22277CA for ; Thu, 25 Apr 2019 09:57:48 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at opensides.be Received: from smtp.opensides.be ([127.0.0.1]) by localhost (smtp.opensides.be [127.0.0.1]) (amavisd-new, port 10024) with LMTP id K5sYqJaemZ4n for ; Thu, 25 Apr 2019 09:57:45 +0200 (CEST) Received: from mcmic-probook.opensides.be (63.120.199.77.rev.sfr.net [77.199.120.63]) by smtp.opensides.be (Postfix) with ESMTPSA id 8B4592277C9 for ; Thu, 25 Apr 2019 09:57:45 +0200 (CEST) To: internals@lists.php.net Date: Thu, 25 Apr 2019 09:57:29 +0200 Message-ID: <2122186.CDTRH7ZZW5@mcmic-probook> Organization: OpenSides User-Agent: KMail/5.2.3 (Linux/4.9.0-8-amd64; KDE/5.28.0; x86_64; ; ) In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" Subject: Re: [PHP-DEV] [RFC] [VOTE] Deprecate PHP's short open tags From: come@opensides.be (=?ISO-8859-1?Q?C=F4me?= Chilliet) Le jeudi 25 avril 2019, 00:11:42 CEST Thomas Hruska a =C3=A9crit : > So I built a tool that meets the above minimum criteria and have already= =20 > run it against a fairly extensive codebase (and have shared the tool=20 > with a few concerned folks - you've already seen its output here on-list): >=20 > https://github.com/cubiclesoft/php-short-open-tag-finder/ =C2=AB// (C) 2019 CubicleSoft. All Rights Reserved.=C2=BB does not meet my= minimum criteria. > Tools that just walk down a directory tree and bulk=20 > modify by default, [=E2=80=A6] should all be rejected from=20 > consideration. =20 I do not understand why you have a problem with this, the rest of your emai= l seems to suggest you are using git. Just run the tool and then look at the diff before committing anything. This makes *way* more sense than having to manually review each change one = by one. C=C3=B4me