Return-Path: Delivered-To: apmail-lucene-java-dev-archive@www.apache.org Received: (qmail 33161 invoked from network); 10 Oct 2009 19:28:15 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 10 Oct 2009 19:28:15 -0000 Received: (qmail 31985 invoked by uid 500); 10 Oct 2009 19:28:15 -0000 Delivered-To: apmail-lucene-java-dev-archive@lucene.apache.org Received: (qmail 31897 invoked by uid 500); 10 Oct 2009 19:28:14 -0000 Mailing-List: contact java-dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: java-dev@lucene.apache.org Delivered-To: mailing list java-dev@lucene.apache.org Received: (qmail 31886 invoked by uid 99); 10 Oct 2009 19:28:14 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 10 Oct 2009 19:28:14 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of simon.willnauer@googlemail.com designates 209.85.220.206 as permitted sender) Received: from [209.85.220.206] (HELO mail-fx0-f206.google.com) (209.85.220.206) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 10 Oct 2009 19:28:04 +0000 Received: by fxm2 with SMTP id 2so7102538fxm.28 for ; Sat, 10 Oct 2009 12:26:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=gamma; h=domainkey-signature:mime-version:received:reply-to:date:message-id :subject:from:to:content-type; bh=l1cHX/tUkt6EAaT33HFb3mJNFBO+O8bqlBUosqcAKO8=; b=UcKBjrEAOCcCT667wBEF/2UJ32DPXZbhPnsZltwrvqgxuXIw+QZ7cZcpH3WyiOOgsG BLJIZvp8OcUx4vHfxxWyhNTCS5A4P154X9BR+hk5M9kzkaMVpRCFDsRGhs0WrhJqIY8M XkE4rbXY9sSuNMZAmxB1+FaaUUR7J9kYB/j3E= DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=mime-version:reply-to:date:message-id:subject:from:to:content-type; b=rL5ZV+irX8fdVclhCHQX2hj8S6p1DqYm27JUFICHL7d+n/Bqwj6fNWE8g8s2ZSrF7Z Dsw95MtB/CFrlegVZBG7iVJHY2A6fDp/CdzcRskoZ53gZXzT0ng4TqDd/Lv2jxi7e+Vo IMMYYwmsduuYJlTB4EyiPfHrHF/kfNT6FjqSs= MIME-Version: 1.0 Received: by 10.239.182.137 with SMTP id q9mr233922hbg.72.1255202803374; Sat, 10 Oct 2009 12:26:43 -0700 (PDT) Reply-To: simon.willnauer@gmail.com Date: Sat, 10 Oct 2009 21:26:43 +0200 Message-ID: Subject: Change HashSet to Set in WordlistLoader - BackCompat Issue From: Simon Willnauer To: java-dev@lucene.apache.org Content-Type: text/plain; charset=UTF-8 X-Virus-Checked: Checked by ClamAV on apache.org Hey there, in the context of https://issues.apache.org/jira/browse/LUCENE-1967 I was looking at org.apache.lucene.analysis.WordlistLoader. I noticed that all static methods return HashSet / HashMap instead of their corresponding interfaces Map / Set. As I'm going to add another static helper to this class I was wondering if we should change the concrete impl. to interface return values. I guess this would break back-compat. for 3.0 so would it make sense to mark the current static methods as deprecated and add new ones or should we just keep it as it is while not being best practice though. simon --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org For additional commands, e-mail: java-dev-help@lucene.apache.org