interface.ccl parser reports error for extra spaces after in function alias block

Comments (8)

1. 

   Erik Schnetter

   • changed status to open
   • removed comment

   Please test this patch:

   Index: interface_parser.pl

   --- interface_parser.pl (revision 4788) +++ interface_parser.pl (working copy) @@ -746,7 +746,7 @@ $funcname = $1; $provided_by = $2;

   - if($provided_by =~ m/(.*)\s*LANGUAGE\s*(.+)/i) + if($provided_by =~ m/(.*)\s*LANGUAGE\s*(.+)\s*/i) { $provided_by = $1; $provided_by_language = "\U$2";

   • 2012-02-05T18:40:27+00:00
2. 

   Jian Tao reporter

   • removed comment

   tried this and couple hacks myself, but still not working.

   • 2012-02-10T16:25:06+00:00
3. 

   Erik Schnetter

   • removed comment

   Right. The pattern should end in

   LANGUAGE\s+(.*\S)\s*/i

   (which is still not quite right, because no space in front of LANGUAGE is required).

   • 2012-02-10T18:12:21+00:00
4. 

   Frank Löffler

   • changed status to open
   • removed comment

   • 2012-03-08T23:39:42+00:00
5. 

   Erik Schnetter

   • changed status to open
   • removed comment

   This is the patch I am suggesting:

   Index: interface_parser.pl

   --- interface_parser.pl (revision 4788) +++ interface_parser.pl (working copy) @@ -746,7 +746,7 @@ $funcname = $1; $provided_by = $2;

   - if($provided_by =~ m/(.*)\s*LANGUAGE\s*(.+)/i) + if($provided_by =~ m/(.*)\s* LANGUAGE\s+(.*\S)\s*/i) { $provided_by = $1; $provided_by_language = "\U$2";

   • 2012-03-09T06:46:07+00:00
6. 

   Jian Tao reporter

   • removed comment

   It works for me. Thanks !

   • 2012-03-09T10:48:11+00:00
7. 

   Erik Schnetter

   • changed status to resolved
   • removed comment

   Corrected and applied. The regexp is now

   if($provided_by =~ m/^(.*)\s+LANGUAGE\s+(.*\S)\s*$/i)

   • 2012-03-12T12:42:17+00:00
8. 

   Roland Haas

   • edited description
   • changed status to closed

   • 2019-02-21T20:21:38+00:00
9. Log in to comment