Return-Path: X-Original-To: apmail-commons-dev-archive@www.apache.org Delivered-To: apmail-commons-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id BB1D6CB25 for ; Sun, 27 May 2012 19:21:46 +0000 (UTC) Received: (qmail 46461 invoked by uid 500); 27 May 2012 19:21:46 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 46355 invoked by uid 500); 27 May 2012 19:21:46 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 46346 invoked by uid 99); 27 May 2012 19:21:46 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 27 May 2012 19:21:46 +0000 X-ASF-Spam-Status: No, hits=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of SRS0=tlTk=D7=m4x.org=sebastien.brisard@bounces.m4x.org designates 129.104.30.34 as permitted sender) Received: from [129.104.30.34] (HELO mx1.polytechnique.org) (129.104.30.34) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 27 May 2012 19:21:36 +0000 Received: from mail-ob0-f171.google.com (mail-ob0-f171.google.com [209.85.214.171]) (using TLSv1 with cipher RC4-MD5 (128/128 bits)) (No client certificate requested) by ssl.polytechnique.org (Postfix) with ESMTPSA id 34F0314045565 for ; Sun, 27 May 2012 21:21:15 +0200 (CEST) Received: by obfk16 with SMTP id k16so5949949obf.30 for ; Sun, 27 May 2012 12:21:14 -0700 (PDT) MIME-Version: 1.0 Received: by 10.50.17.226 with SMTP id r2mr3007690igd.47.1338146473992; Sun, 27 May 2012 12:21:13 -0700 (PDT) Received: by 10.64.72.167 with HTTP; Sun, 27 May 2012 12:21:13 -0700 (PDT) In-Reply-To: <4FC2507E.6010303@free.fr> References: <4FBE8EED.5080107@free.fr> <4FC2507E.6010303@free.fr> Date: Sun, 27 May 2012 21:21:13 +0200 Message-ID: Subject: Re: [math] LocalizedFormats.ALPHA and BETA no longer used From: =?ISO-8859-1?Q?S=E9bastien_Brisard?= To: Commons Developers List Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-AV-Checked: ClamAV using ClamSMTP at svoboda.polytechnique.org (Sun May 27 21:21:15 2012 +0200 (CEST)) X-Org-Mail: sebastien.brisard.1997@polytechnique.org X-Old-Spam-Flag: No, tests=bogofilter, spamicity=0.000070, queueID=8453214045568 Hi, 2012/5/27 Luc Maisonobe : > Le 27/05/2012 16:25, Jochen Wiedmann a =E9crit : >> On Thu, May 24, 2012 at 8:41 PM, Luc Maisonobe w= rote: >>> Le 24/05/2012 21:27, S=E9bastien Brisard a =E9crit : >>>> Hi, >>>> commit 1342404 solves MATH-791. It turns out that >>>> LocalizedFormats.ALPHA and BETA are no longer used. I wanted to remove >>>> them, but that breaks the compatibility. How should we do that >>>> cleanly? >>> >>> Just remove the entries from the enum and temove the corresponding entr= y >>> in the translation properties file. >> >> Why remove them at all? I understand the desire, it meets my sense for >> clean code. But I learned to fight against these desires from time to >> time. > > These parameters are mainly for internal use, we don't consider them as > part of the public API. As we remove their internal use, it seems good > to be consistent and remove everything at once. > I fully agree. > > Do you have an idea about another use that could justify we retain them ? > It's probably just as well to restore these constants if we find a use for them at a later point (that would be trivial). So, do we agree that these constants should go? How should I proceed? The clean way (deprecation in 3.1, and complete removal in 4.0), or the brutal way (complete removal in 3.1) ? Best regards, S=E9bastien --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org