Changes between Version 3 and Version 4 of DevelopersFAQ

Timestamp:

Aug 5, 2007, 7:47:07 PM (18 years ago)

Author:

guest

Comment:

--

DevelopersFAQ

@@ -68 +68 @@
 enough.
 
+
+=== Is the code below OK? Did I account for the terminating nulls correctly? ===
+
+{{{
+//BldFullPathName returns the fullpath of a file or Nullstr
+CHAR *FullPathName = BldFullPathName(CHAR *pathname, CHAR
+*filename)
+{
+
+CHAR newname[CCHMAXPATH] = Nullstr;
+INT c = 0;
+
+c = strlen(pathname);
+if (c > 0) {
+memcpy(newname, pathname, c + 1);
+if (newname[c] != '\\')
+newname[c++] = '\\';
+}
+strcpy(newname + c, filename);
+return newname;
+}
+
+//BldQuotedFullPathName returns the quoted fullpath of a file or ""
+CHAR *FullPathName = BldQuotedFullPathName(CHAR *pathname,
+CHAR *filename)
+{
+
+CHAR newname[CCHMAXPATH] = '\"';
+INT c = 0;
+
+c = strlen(pathname);
+if (c > 0) {
+memcpy(newname + 1, pathname, c + 2);
+if (newname[c + 1] != '\\')
+newname[c + 2] = '\\';
+strcpy(newname + c + 3, filename);
+}
+else
+strcpy(newname + 1, filename)
+strcat(newname, '\"')
+return newname;
+}
+}}}
+
+Not quite. The caller must pass a pointer to the buffer. Keep in mind
+that local variable disappear when the function returns.
+
+
+{{{
+PSZ BldFullPathName(PSZ fullPathName, PSZ dirname, PSZ filename);
+}}}
+
+
+
+{{{
+CHAR newname[CCHMAXPATH] = Nullstr;
+}}}
+
+
+This will give you an error because NullStr is a pointer.
+
+
+{{{
+CHAR newname[CCHMAXPATH];
+}}}
+
+
+Is sufficent since you are always going to copy something. The return is
+
+
+{{{
+return fullPathName
+}}}
+
+
+with the appropriate name changes.
+
+
+{{{
+//BldQuotedFullPathName returns the quoted fullpath of a file or "" CHAR
+}}}
+
+
+The quotes need to be optional and the return buffer needs to be passed as
+above.
+
+
+{{{
+PSZ BldQuotedFullPathName(PSZ fullPathName, PSZ pathname, PSZ filename)
+}}}
+
+
+You also need to add the calls to needs_quotes() a stuff the quotes only
+if needed. This is what the existing inline code does.
+
 ----