r5446 - trunk/eda/fped

werner at docs.openmoko.org werner at docs.openmoko.org
Sat Aug 15 00:12:17 CEST 2009


Author: werner
Date: 2009-08-15 00:12:16 +0200 (Sat, 15 Aug 2009)
New Revision: 5446

Modified:
   trunk/eda/fped/fpd.y
   trunk/eda/fped/gui_frame.c
Log:
- KiCad is very liberal when it comes to valid part names, and so are we now
  (i.e., we now accept anything but control and non-ASCII characters)



Modified: trunk/eda/fped/fpd.y
===================================================================
--- trunk/eda/fped/fpd.y	2009-08-14 15:53:21 UTC (rev 5445)
+++ trunk/eda/fped/fpd.y	2009-08-14 22:12:16 UTC (rev 5446)
@@ -207,7 +207,7 @@
 				YYABORT;
 			}
 			for (p = $2; *p; *p++)
-				if (!is_id_char(*p, 0)) {
+				if (*p < 32 || *p > 126) {
 					yyerrorf("invalid part name");
 					YYABORT;
 				}

Modified: trunk/eda/fped/gui_frame.c
===================================================================
--- trunk/eda/fped/gui_frame.c	2009-08-14 15:53:21 UTC (rev 5445)
+++ trunk/eda/fped/gui_frame.c	2009-08-14 22:12:16 UTC (rev 5446)
@@ -1120,9 +1120,11 @@
 {
 	if (!*s)
 		return 0;
-	while (*s)
-		if (!is_id_char(*s++, 0))
+	while (*s) {
+		if (*s < 32 || *s > 126)
 			return 0;
+		s++;
+	}
 	return 1;
 }
 




More information about the commitlog mailing list