Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
MIME-Version: 1.0
In-Reply-To: 
 <CAJZRQKyRxXzT+aXpAjgQX8mgiH05TCNXCquUGaVhGVy6dNm_bw@mail.gmail.com>
References: <20150621184450.DE6B2AC094D@hades.apache.org>
	<CAOGo0VZcBiAUb4FVjRk-jzKP5wJDjMYAyDSZKJMMyNmUWFFHGA@mail.gmail.com>
	<CAJZRQKyRxXzT+aXpAjgQX8mgiH05TCNXCquUGaVhGVy6dNm_bw@mail.gmail.com>
Date: Tue, 23 Jun 2015 17:11:27 +0100
Message-ID: 
 <CAOGo0VbnaxjM2S8YhRU=AeV1dNp7+pW+Myg5YqNUGBPJnD_+wQ@mail.gmail.com>
Subject: Re: svn commit: r1686747 [1/3] - in /commons/proper/io/trunk/src:
 main/java/org/apache/commons/io/ main/java/org/apache/commons/io/filefilter/
 main/java/org/apache/commons/io/input/
 main/java/org/apache/commons/io/monitor/
 main/java/org/apache/commons/io/o...
From: sebb <sebbaz@gmail.com>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=UTF-8

On 22 June 2015 at 06:38, Kristian Rosenvold
<kristian.rosenvold@gmail.com> wrote:
> 2015-06-22 2:53 GMT+02:00 sebb <sebbaz@gmail.com>:
>
>> >What was the Findbugs error?
>>
>> Best not to mix these in a single commit.
>>
> Sorry about that.
>
> org.apache.commons.io.output.DeferredFileOutputStream#thresholdReached
> possible file handle leak upon exception
>

Can you update the SVN log message accordingly?

>
>
>>
>> Also looks like many of the checkstyle fixes are just line-wraps; I
>> thought we allowed a longer line length?
>> Screens tend to be used in landscape mode so it's easier to read code
>> if is not wrapped unnecessarily
>>
>
> Most of it is just feeding the OCD. 0 checkstyle errors is the one ring to
> rule them all :)
>
> Would it make sense to change the checkstyle column width to something like
> 160 ?

Or drop the rule entirely?

> Kristian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org