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 EB9ADC7D9 for ; Thu, 4 Dec 2014 13:14:47 +0000 (UTC) Received: (qmail 53232 invoked by uid 500); 4 Dec 2014 13:14:47 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 53094 invoked by uid 500); 4 Dec 2014 13:14:47 -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 53050 invoked by uid 99); 4 Dec 2014 13:14:46 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 04 Dec 2014 13:14:46 +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 garydgregory@gmail.com designates 209.85.212.181 as permitted sender) Received: from [209.85.212.181] (HELO mail-wi0-f181.google.com) (209.85.212.181) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 04 Dec 2014 13:14:42 +0000 Received: by mail-wi0-f181.google.com with SMTP id r20so27840983wiv.8 for ; Thu, 04 Dec 2014 05:14:22 -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=ask9EQAkaVazWNwd2PZEzo1w8PuHRjqaWA0aKXxrTFU=; b=wUaxLni3sq1WCjzauax0kMqH9lqrx9QBXq+7MagUEm9tZPTi5VfhYqYPkyFgbVZmyw 4Pt/CrWQdFEEzNCBzoc9ItjWx4oDQ3gr0Sq4oRwRl1Cbh3Nab7/+QlK5r4G9vCIz/xeI 8PH9Z+AorkssGpe9fo/NUkQq5cKXXKNWXIbqC6KLC+zON6UgOkqq1yFMhjerSP3280Sq 6wG9Zkg8i/qEg5Pz6kwlD4ndUtsUgV/ISSufW2rpjnEmIWL505K9KOjcJPc3R4oRBKAO PwmEjvzmDlhxXF8WjE/YgAxdGbXZVHfjjG7xhbBTRuWbd4GuGnHiSxIL+gtVkiKeC+jK PHpg== MIME-Version: 1.0 X-Received: by 10.194.119.193 with SMTP id kw1mr15821263wjb.37.1417698861962; Thu, 04 Dec 2014 05:14:21 -0800 (PST) Received: by 10.27.13.29 with HTTP; Thu, 4 Dec 2014 05:14:21 -0800 (PST) In-Reply-To: References: <20141110125457.C885C23888E4@eris.apache.org> Date: Thu, 4 Dec 2014 08:14:21 -0500 Message-ID: Subject: Re: svn commit: r1637832 - /commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/digest/DigestUtils.java From: Gary Gregory To: Commons Developers List Content-Type: multipart/alternative; boundary=089e0122862610eb96050963bd56 X-Virus-Checked: Checked by ClamAV on apache.org --089e0122862610eb96050963bd56 Content-Type: text/plain; charset=UTF-8 On Wed, Dec 3, 2014 at 11:35 AM, sebb wrote: > On 10 November 2014 at 12:54, wrote: > > Author: ggregory > > Date: Mon Nov 10 12:54:57 2014 > > New Revision: 1637832 > > > > URL: http://svn.apache.org/r1637832 > > Log: > > Move static var decl. > > Why? > > Most components I have worked on have the constants at the start. > The current code has the decl at the start. I can't quite recall what happened with this commit three weeks ago though, odd. Gary > > Modified: > > > commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/digest/DigestUtils.java > > > > Modified: > commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/digest/DigestUtils.java > > URL: > http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/digest/DigestUtils.java?rev=1637832&r1=1637831&r2=1637832&view=diff > > > ============================================================================== > > --- > commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/digest/DigestUtils.java > (original) > > +++ > commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/digest/DigestUtils.java > Mon Nov 10 12:54:57 2014 > > @@ -33,8 +33,6 @@ import org.apache.commons.codec.binary.S > > */ > > public class DigestUtils { > > > > - private static final int STREAM_BUFFER_LENGTH = 1024; > > - > > /** > > * Read through an InputStream and returns the digest for the data > > * > > @@ -816,4 +814,6 @@ public class DigestUtils { > > messageDigest.update(StringUtils.getBytesUtf8(valueToDigest)); > > return messageDigest; > > } > > + > > + private static final int STREAM_BUFFER_LENGTH = 1024; > > } > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > -- E-Mail: garydgregory@gmail.com | ggregory@apache.org Java Persistence with Hibernate, Second Edition JUnit in Action, Second Edition Spring Batch in Action Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory --089e0122862610eb96050963bd56--