Hi,

Can a gatekeeper review the patch to bug 716? Thank you very much. The bug
716 is caused by the special character '+', which is legal in file names but
illegal in identifiers. The original code only replaces the '-' by '_'. This
patch will replace all these kind of characters.

Index: common/ipc_compile.cxx
===================================================================
--- common/ipc_compile.cxx      (revision 2052)
+++ common/ipc_compile.cxx      (working copy)
@@ -208,7 +208,7 @@
     */
 static char* proper_name(char *name){
     for(char *i = name; *i; i++) {
-      if ( *i == '-' ) *i = '_';
+      if ( !isalnum(*i) ) *i = '_';
     }
     return name;
 }
Index: common/ipc_pic.cxx
===================================================================
--- common/ipc_pic.cxx  (revision 2052)
+++ common/ipc_pic.cxx  (working copy)
@@ -63,7 +63,7 @@

   int i;
   for(i=0; bname[i] != '\0'; i++) {
-    if ( bname[i] == '-' ) IPA_Object_Name[i] = '_';
+    if ( !isalnum(bname[i]) ) IPA_Object_Name[i] = '_';
     else IPA_Object_Name[i] = bname[i];
   }
   IPA_Object_Name[i] = '\0';


-- 
Regards,
Lai Jian-Xin
------------------------------------------------------------------------------
Gaining the trust of online customers is vital for the success of any company
that requires sensitive data to be transmitted over the Web.   Learn how to 
best implement a security strategy that keeps consumers' information secure 
and instills the confidence they need to proceed with transactions.
http://p.sf.net/sfu/oracle-sfdevnl 
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to