Am Donnerstag, den 13.09.2012, 19:06 +0200 schrieb Francesco Poli: > On Thu, 13 Sep 2012 01:30:25 +0200 Benjamin Drung wrote: > > > Am Donnerstag, den 13.09.2012, 00:08 +0200 schrieb Francesco Poli: > [...] > > > OK, this is my attempt to fix the bug. > > > > > > I tested a modified licensecheck on the minimal test case that I > > > provided and on a directory tree containing about 11400 files to scan. > > > It seems that It Works For Me™. > [...] > > > > > > Take into account that my Perl knowledge is just a smattering > > > and it is rusty, too... > > > > My Perl foo isn't that good either. > > > > Why did you move the Fortran comment removal line up to the top of the > > block? > > For consistency with the other comment about C / C++ ...
I was referring to the line of code, not the comment. ;) Moving the changed line of code down, let's the test case fail. Having the comment on a separate line is fine. I have applied your previous patch and added the test case. Thanks for your contribution. -- Benjamin Drung Debian & Ubuntu Developer
signature.asc
Description: This is a digitally signed message part
