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 38FEEE55F for ; Thu, 22 Nov 2012 08:16:42 +0000 (UTC) Received: (qmail 91580 invoked by uid 500); 22 Nov 2012 08:16:41 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 91267 invoked by uid 500); 22 Nov 2012 08:16:40 -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 91203 invoked by uid 99); 22 Nov 2012 08:16:38 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 22 Nov 2012 08:16:38 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of beneritter@gmail.com designates 209.85.223.171 as permitted sender) Received: from [209.85.223.171] (HELO mail-ie0-f171.google.com) (209.85.223.171) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 22 Nov 2012 08:16:32 +0000 Received: by mail-ie0-f171.google.com with SMTP id 17so7703242iea.30 for ; Thu, 22 Nov 2012 00:16:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=1jhGWaq6LB7YzgP1CqSJLFcqS2/+cSwzjjLwgvivPGc=; b=ZmEQkLUzYT34jjKnwQHegl662mengT341QMv0KQ1nIpOHJAC3SvnQOkjMxcoSjoqfW FgNmUBzDrq4C39iCaXQCX3GppodjzxCoOonfgmuAU6BzJB6krGKZtk+09cCl9+zyHQQa GtJY6BMYxN9/Go/18pa+fb6K0H387wHnTBEyM4LYMoVRBrqE5l1JElxEGTSn1vIKdCoc /7ylP2GSMzr4oEHtHmZG3S61d2dD7L7KyAQeAcjIFD6L1RhME0OG3zeRGgPe9nTI9vj9 2NGWyPniGtwzRybNSz+X19SqjTK9d4vJ1cxzY9jc6uUgpMKmnb40WZvE0BmnKvBiYryf JMVw== MIME-Version: 1.0 Received: by 10.50.57.200 with SMTP id k8mr2434790igq.29.1353572172186; Thu, 22 Nov 2012 00:16:12 -0800 (PST) Received: by 10.64.100.165 with HTTP; Thu, 22 Nov 2012 00:16:11 -0800 (PST) In-Reply-To: References: <50ABFD1D.4020205@apache.org> <-4391643847510383239@unknownmsgid> Date: Thu, 22 Nov 2012 09:16:11 +0100 Message-ID: Subject: Re: [CSV] Discussion about the new CSVFormatBuilder From: Benedikt Ritter To: Commons Developers List Content-Type: multipart/alternative; boundary=14dae9340f8b8042a904cf111407 X-Virus-Checked: Checked by ClamAV on apache.org --14dae9340f8b8042a904cf111407 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Maybe we first have to decide if we want validation of CSVFormats at construction time or not. If not, the changes of CSV-68 can be reverted. Benedikt 2012/11/21 James Carman > I don't really have a problem with the extra call to build() before > you have something useful. It does give us the ability to do > validation on the object before you build it. If we choose not to do > the validation at this time, that's fine, but if we ever do choose to > add that in the future, we don't have to break API backward > compatibility to do so. > > On Tue, Nov 20, 2012 at 5:57 PM, Gary Gregory > wrote: > > Ok this is good. Let's see some healthy debating. :) > > > > What is the alternate API? > > > > To me the bother is the extra build() call, but that's the pattern. > > > > Could an alt API be used and co-exist? > > > > Is making the ctor an option? It would have to do some validation. > > > > Gary > > > > On Nov 20, 2012, at 16:59, Emmanuel Bourg wrote: > > > >> Le 20/11/2012 20:01, Benedikt Ritter a =E9crit : > >> > >>> Please share your thoughts about the builder. > >> > >> Sorry Benedikt but I have to say I really don't like this design. I > >> prefer a simpler API for the reasons you mentioned in the disadvantage= s. > >> The minor improvements from the developer's point of view are much les= s > >> important than the ease of use from user's point of view. > >> > >> Emmanuel Bourg > >> > >> > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > > For additional commands, e-mail: dev-help@commons.apache.org > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > --14dae9340f8b8042a904cf111407--