From 24a60c22cd156a0c572ecbf6de767b989ec7a897 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arnim=20L=C3=A4uger?= <arniml@users.sourceforge.net>
Date: Fri, 22 Aug 2008 20:58:36 +0000
Subject: [PATCH] enhanced error reporting

git-svn-id: https://urjtag.svn.sourceforge.net/svnroot/urjtag/trunk@1343 b68d4a1b-bc3d-0410-92ed-d4ac073336b7
---
 jtag/ChangeLog             | 13 +++++++++
 jtag/src/bsdl/vhdl_bison.y | 56 +++++++++++++++++++-------------------
 2 files changed, 41 insertions(+), 28 deletions(-)

diff --git a/jtag/ChangeLog b/jtag/ChangeLog
index 2d0714c2..d12a0830 100644
--- a/jtag/ChangeLog
+++ b/jtag/ChangeLog
@@ -1,3 +1,16 @@
+2008-08-22  Arnim Laeuger  <arniml@users.sourceforge.net>
+
+   * src/bsdl/vhdl_bison.y: enhanced error reporting
+
+   * configure.ac, data/Makefile.am, src/Makefile.am,
+     src/bsdl/Makefile.am, src/bsdl/bsdl.c, src/bsdl/bsdl_bison.y,
+     src/bsdl/bsdl_flex.l, src/bsdl/bsdl_sem.c, src/bsdl/bsdl_sysdep.h,
+     data/bsdl/STD_1532_2001, data/bsdl/STD_1532_2002, src/bsdl/bsdl_msg.h,
+     src/bsdl/bsdl_parser.h, src/bsdl/bsdl_types.h, src/bsdl/vhdl_bison.y,
+     src/bsdl/vhdl_flex.l, src/bsdl/vhdl_parser.h, src/bsdl/bsdl_local.h:
+     merged from branches/bsdl 1304:1340
+     rewrite of BSDL parser
+
 2008-07-15  Arnim Laeuger  <arniml@users.sourceforge.net>
 
    * doc/UrJTAG.txt, src/cmd/dr.c (cmd_dr_help):
diff --git a/jtag/src/bsdl/vhdl_bison.y b/jtag/src/bsdl/vhdl_bison.y
index 038a9451..3850b5e8 100644
--- a/jtag/src/bsdl/vhdl_bison.y
+++ b/jtag/src/bsdl/vhdl_bison.y
@@ -485,19 +485,25 @@ VHDL_Constant_Part : IDENTIFIER COLON PIN_MAP_STRING COLON_EQUAL
 		     // { set_attr_const( priv_data, $1, strdup( "PIN_MAP_STRING" ) ); }
                      { free( $1 ); }
 ;
-VHDL_Attribute     : VHDL_Attr_Boolean
-                   | VHDL_Attr_Decimal
-                   | VHDL_Attr_Real
-                   | VHDL_Attr_String
-                   | VHDL_Attr_PhysicalPinMap
+VHDL_Attribute     : ATTRIBUTE VHDL_Attribute_Types
 ;
-VHDL_Attr_Boolean  : ATTRIBUTE IDENTIFIER OF IDENTIFIER
-                     COLON SIGNAL IS Boolean SEMICOLON
+VHDL_Attribute_Types : VHDL_Attr_Boolean
+                     | VHDL_Attr_Decimal
+                     | VHDL_Attr_Real
+                     | VHDL_Attr_String
+                     | VHDL_Attr_PhysicalPinMap
+                     | error
+                       {
+                         Print_Error( priv_data, _("Error in Attribute specification") );
+                         BUMP_ERROR; YYABORT;
+                       }
+;
+VHDL_Attr_Boolean  : IDENTIFIER OF IDENTIFIER COLON SIGNAL IS Boolean SEMICOLON
                      { 
-                       //set_attr_bool( priv_data, $2, $8 );
-                       //free( $4 );
+                       //set_attr_bool( priv_data, $1, $7 );
+                       //free( $3 );
                        /* skip boolean attributes for the time being */
-                       free( $2 ); free( $4 );
+                       free( $1 ); free( $3 );
 		     }
 ;
 Boolean            : TRUE
@@ -505,36 +511,30 @@ Boolean            : TRUE
                    | FALSE
                      { $$ = 0; }
 ;
-VHDL_Attr_Decimal : ATTRIBUTE IDENTIFIER OF IDENTIFIER
-                    COLON ENTITY IS DECIMAL_NUMBER SEMICOLON
+VHDL_Attr_Decimal : IDENTIFIER OF IDENTIFIER COLON ENTITY IS DECIMAL_NUMBER SEMICOLON
                     {
-                      set_attr_decimal( priv_data, $2, $8 );
-                      free( $4 );
-                      /* skip decimal attributes for the time being */
-                      //free( $2 ); free( $4 );
+                      set_attr_decimal( priv_data, $1, $7 );
+                      free( $3 );
 		    }
 ;
-VHDL_Attr_Real   : ATTRIBUTE IDENTIFIER OF IDENTIFIER COLON SIGNAL
-                   IS LPAREN REAL_NUMBER COMMA Stop RPAREN SEMICOLON
+VHDL_Attr_Real   : IDENTIFIER OF IDENTIFIER COLON SIGNAL IS LPAREN REAL_NUMBER COMMA Stop RPAREN SEMICOLON
                    {
-		     //set_attr_real( priv_data, $2, $9 );
-		     //free( $4 );
+		     //set_attr_real( priv_data, $1, $8 );
+		     //free( $3 );
                      /* skip real attributes for the time being */
-                     free( $2 ); free( $4 ); free( $9 );
+                     free( $1 ); free( $3 ); free( $8 );
 		   }
 ;
 Stop             : LOW | BOTH
 ;
-VHDL_Attr_String : ATTRIBUTE IDENTIFIER OF IDENTIFIER
-                   COLON ENTITY IS Quoted_String SEMICOLON
+VHDL_Attr_String : IDENTIFIER OF IDENTIFIER COLON ENTITY IS Quoted_String SEMICOLON
                    {
-		     set_attr_string( priv_data, $2, strdup( priv_data->buffer ) );
-		     free( $4 );
+		     set_attr_string( priv_data, $1, strdup( priv_data->buffer ) );
+		     free( $3 );
 		   }
 ;
-VHDL_Attr_PhysicalPinMap : ATTRIBUTE IDENTIFIER OF IDENTIFIER
-                           COLON ENTITY IS PHYSICAL_PIN_MAP SEMICOLON
-                           { free( $2 ); free( $4 ); }
+VHDL_Attr_PhysicalPinMap : IDENTIFIER OF IDENTIFIER COLON ENTITY IS PHYSICAL_PIN_MAP SEMICOLON
+                           { free( $1 ); free( $3 ); }
 ;
 Quoted_String    : QUOTED_STRING
                    {