Re: tasklist/checkstyle version dep

On Wed, 11 Feb 2004 14:08:55 +0100, Petr Kuzel wrote:

> Petr Kuzel wrote:
> 
>> John Richardson wrote:
>> 
>>> On Mon, 17 Nov 2003 04:46:54 -0500, Jesse Glick wrote:
>>> 
>>>> Tor Norbye wrote:
>>>>> Hmm, I wonder if they've fixed the reason why I "abandoned" the
>>>>> integration with checkstyle in the first place: I could only find a way
>>>>> to run their checkers on Files, not InputStream / Reader / Document /
>>>>> something dynamic.  For now, the checkstyle integration is pretty
>>>>> rough.
>>>> 
>>>> Release notes for 3.2 don't mention anything like that. Did you file an
>>>> RFE or patch for them?
>>> 
>>> FWIW, I did some digging and it turns out that some of the checks really
>>> do need a java.io.File because they access the file via a
>>> RandomAccessFile, etc...
>>> 
>>> I inquired about hacking in InputStreams on the checkstyle developer
>>> list. Basically I was throwing out the idea of creating an abstraction
>>> over File
>>> which would include both File and InputStreams.  This didn't go over too
>>> well.
>>> 
>>> Is Jesse's suggestion of File.createTempFile() that much of a headache?
>> 
>> In development time few lines of code. In runtime probably
>> several hundreds milliseconds. It's probably OK for user's action
>> but unsuitable for background scanning.

OK, but several hundreds of milliseconds is probably small compared to the
time it takes to actually run the checks.  

> 
> Another approach could be that checkstyle accepts CharSequence.
> In standalone mode it can get FileChannel then ByteBuffer and then
> CharBuffer.

I think they're still supporting JDK 1.3, so NIO might be out of the
question.

> 
>   Cc.