Bug#687452: devscripts: fails to recognize licenses in (fixed-form) Fortran code

Francesco Poli invernomuto at paranoici.org
Wed Sep 19 17:28:13 UTC 2012


On Wed, 19 Sep 2012 14:32:31 +0200 Benjamin Drung wrote:

> Am Donnerstag, den 13.09.2012, 19:06 +0200 schrieb Francesco Poli:
> > On Thu, 13 Sep 2012 01:30:25 +0200 Benjamin Drung wrote:
[...]
> > > 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. ;)

Whoops, it seems that I misread your question!   :p
Instead of reading "the Fortran comment removal line", I happened to
read "the Fortran comment removal *comment*"...    D'oh!

Now I understand your question correctly (hopefully) and I can reply.

Moving the comment removal statement up to the top of the block is
crucial: it allows the removal of "[cC] " at the beginning of each line
of text

  $content =~ s/^[cC] //gm;

*before* the following statement removes all tabs and all newline
characters:

  $content =~ tr/\t\r\n/ /;

> 
> Moving the changed line of code down, let's the test case fail.

That's expected, for the reason explained above.

> 
> Having the comment on a separate line is fine. I have applied your
> previous patch and added the test case. Thanks for your contribution.

That's really great news!   :-)
Thanks to you for dealing with this bug so quickly and effectively.

Bye.

-- 
 http://www.inventati.org/frx/frx-gpg-key-transition-2010.txt
 New GnuPG key, see the transition document!
..................................................... Francesco Poli .
 GnuPG key fpr == CA01 1147 9CD2 EFDF FB82  3925 3E1C 27E1 1F69 BFFE
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20120919/fbd0dc50/attachment.pgp>


More information about the devscripts-devel mailing list