Changeset c510a7 for molecuilder/src/config.cpp

Timestamp:

Apr 29, 2008, 12:19:51 PM (17 years ago)

Author:

Frederik Heber <heber@…>

Children:

e936b3

Parents:

307fd1

Message:

char lengths of 255 and MAXDUMMYSTRING replaced with define MAXSTRINGSIZE in molecuilder and pcp

File:

• molecuilder/src/config.cpp (modified) (3 diffs)

molecuilder/src/config.cpp ¶

@@ -13 +13 @@
 config::config()
 {
-  mainname = (char *) MallocString(sizeof(char)*255,"config constructor: mainname");
-  defaultpath = (char *) MallocString(sizeof(char)*255,"config constructor: mainname");
-  pseudopotpath = (char *) MallocString(sizeof(char)*255,"config constructor: mainname");
-  configpath = (char *) MallocString(sizeof(char)*255,"config constructor: mainname");
-  configname = (char *) MallocString(sizeof(char)*255,"config constructor: mainname");
+  mainname = (char *) MallocString(sizeof(char)*MAXSTRINGSIZE,"config constructor: mainname");
+  defaultpath = (char *) MallocString(sizeof(char)*MAXSTRINGSIZE,"config constructor: mainname");
+  pseudopotpath = (char *) MallocString(sizeof(char)*MAXSTRINGSIZE,"config constructor: mainname");
+  configpath = (char *) MallocString(sizeof(char)*MAXSTRINGSIZE,"config constructor: mainname");
+  configname = (char *) MallocString(sizeof(char)*MAXSTRINGSIZE,"config constructor: mainname");
   strcpy(mainname,"pcp");
   strcpy(defaultpath,"not specified");
@@ -414 +414 @@
   char *ptr = NULL;
   int last = -1;
-  for(int i=0;i<255;i++) {
+  for(int i=0;i<MAXSTRINGSIZE;i++) {
     if (filename[i] == '/')
       last = i;
@@ -1168 +1168 @@
                 case(string_type):
                   if (value != NULL) {
-                    if (maxlength == -1) maxlength = strlen((char *)value); // get maximum size of string array
+                    //if (maxlength == -1) maxlength = strlen((char *)value); // get maximum size of string array
+                    maxlength = MAXSTRINGSIZE;
                     length = maxlength > (dummy-dummy1) ? (dummy-dummy1) : maxlength; // cap at maximum
                     strncpy((char *)value, dummy1, length);  // copy as much