[Viplugin-devel] Re: JUnit Test writer for Eclipse based

[Michael B. <zed...@sa...>]

Date: Mon, 8 Dec 2003 13:47:44 +0100
To: "Brian M. Kelley" <bk...@sb...>
Subject: Re: JUnit Test writer for Eclipse based

Hi Brian!

I've reviewed and applied your patch! Looks good! There have been 2
annyoing things tough!

1. The patch was incredibly big, because of whitespace issues. Eclipse
has some problems even when I click on "Ignore Whitespace".

2. For a reason I cannot see you renamed the TextModificator instance to
t instead of _tm. I've reverted this change.

There are always problems with applying patches that change something at
the end of a file. "Only-God-knows" why Eclipse can't handle this, but that's
not your fault ;)

I've commited everything to CVS, so don't forget to update your working
copy.

Thank you & Keep up the good work,
    Michael