ldforge: comparison src/file.cpp

-:4e686b771996
+:ac2d3e8dd110
 }
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
-FILE* openLDrawFile (str path, bool bSubDirectories) {
+FILE* openLDrawFile (str relpath, bool subdirs) {
-	str zTruePath = path;
 #ifndef WIN32
-	zTruePath.replace ("\\", "/");
+	relpath.replace ("\\", "/");
 #endif // WIN32
-	FILE* fp = fopen (path.chars (), "r");
+	printf ("Trying %s\n", relpath.chars ());
-	str zFilePath;
+	FILE* fp = fopen (relpath.chars (), "r");
+	str fullPath;
 	if (fp != null)
 		return fp;
 	if (~io_ldpath.value) {
 		// Try with just the LDraw path first
-		zFilePath = fmt ("%s" DIRSLASH "%s",
+		fullPath = fmt ("%s" DIRSLASH "%s", io_ldpath.value.chars(), relpath.chars());
-			io_ldpath.value.chars(), zTruePath.chars());
+		printf ("Trying %s\n", fullPath.chars());
-		logf ("Trying %s\n", zFilePath.chars());
+		fp = fopen (fullPath, "r");
-		fp = fopen (zFilePath, "r");
 		if (fp != null)
 			return fp;
-		if (bSubDirectories) {
+		if (subdirs) {
 			char const* saSubdirectories[] = {
 				"parts",
 				"p",
 			};
 			for (char const* sSubdir : saSubdirectories) {
-				zFilePath = fmt ("%s" DIRSLASH "%s" DIRSLASH "%s",
+				fullPath = fmt ("%s" DIRSLASH "%s" DIRSLASH "%s",
-					io_ldpath.value.chars(), sSubdir, zTruePath.chars());
+					io_ldpath.value.chars(), sSubdir, relpath.chars());
-				printf ("try %s\n", zFilePath.chars());
+				printf ("Trying %s\n", fullPath.chars());
-				fp = fopen (zFilePath.chars (), "r");
+				fp = fopen (fullPath.chars (), "r");
 				if (fp)
 					return fp;
 			}
 		}
 }
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
-void addRecentFile (str zPath) {
+void addRecentFile (str path) {
-	long lPos = io_recentfiles.value.first (zPath);
+	size_t pos = io_recentfiles.value.first (path);
 	// If this file already is in the list, pop it out.
-	if (lPos != -1) {
+	if (pos != npos) {
-		if (~io_recentfiles.value == ~zPath)
+		if (~io_recentfiles.value == ~path)
 			return; // only recent file - do nothing
 		// Pop it out.
-		str zFront = io_recentfiles.value.substr (0, lPos);
+		str front = io_recentfiles.value.substr (0, pos);
-		str zBack = io_recentfiles.value.substr (lPos + ~zPath + 1, -1);
+		str back;
-		io_recentfiles.value = zFront + zBack;
+		if (pos + ~path + 1 < io_recentfiles.value.len ())
+			back = io_recentfiles.value.substr (pos + ~path + 1, -1);
+		else
+			back = "";
+		io_recentfiles.value = front + back;
 	}
 	// If there's too many recent files, drop one out.
 	while (io_recentfiles.value.count ('@') > 3)
 		io_recentfiles.value = io_recentfiles.value.substr (io_recentfiles.value.first ("@") + 1, -1);
 	// Add the file
 	if (~io_recentfiles.value > 0)
 		io_recentfiles.value += "@";
-	io_recentfiles += zPath;
+	io_recentfiles += path;
 	config::save ();
 	g_win->updateRecentFilesMenu ();
 }
 	return true;
 }
 #define CHECK_TOKEN_COUNT(N) \
 	if (tokens.size() != N) \
-		return new LDGibberish (zLine, "Bad amount of tokens");
+		return new LDGibberish (line, "Bad amount of tokens");
 #define CHECK_TOKEN_NUMBERS(MIN,MAX) \
 	for (ushort i = MIN; i <= MAX; ++i) \
 		if (!isNumber (tokens[i])) \
-			return new LDGibberish (zLine, fmt ("Token #%u was `%s`, expected a number", \
+			return new LDGibberish (line, fmt ("Token #%u was `%s`, expected a number", \
 				(i + 1), tokens[i].chars()));
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
 }
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
-LDObject* parseLine (str zLine) {
+LDObject* parseLine (str line) {
-	vector<str> tokens = zLine.split (" ", true);
+	vector<str> tokens = line.split (" ");
 	if (!tokens.size ()) {
 		// Line was empty, or only consisted of whitespace
 		return new LDEmpty;
 	}
 	if (~tokens[0] != 1)
-		return new LDGibberish (zLine, "Illogical line code");
+		return new LDGibberish (line, "Illogical line code");
 	const char c = tokens[0][0];
 	switch (c - '0') {
 	case 0:
 		{
 					CHECK_TOKEN_NUMBERS (4, 19)
 					LDRadial::Type eType = LDRadial::NumTypes;
 					for (int i = 0; i < LDRadial::NumTypes; ++i) {
-						if (str (LDRadial::radialTypeName ((LDRadial::Type) i)).toupper ().strip (' ') == tokens[3]) {
+						if (str (LDRadial::radialTypeName ((LDRadial::Type) i)).upper ().strip (' ') == tokens[3]) {
 							eType = (LDRadial::Type) i;
 							break;
 						}
 					}
 					if (eType == LDRadial::NumTypes)
-						return new LDGibberish (zLine, fmt ("Unknown radial type %s", tokens[3].chars ()));
+						return new LDGibberish (line, fmt ("Unknown radial type %s", tokens[3].chars ()));
 					LDRadial* obj = new LDRadial;
 					obj->radType = eType;			// 3
 					obj->color = atol (tokens[4]);		// 4
 			// Try open the file
 			OpenFile* pFile = loadSubfile (tokens[14]);
 			// If we cannot open the file, mark it an error
 			if (!pFile)
-				return new LDGibberish (zLine, "Could not open referred file");
+				return new LDGibberish (line, "Could not open referred file");
 			LDSubfile* obj = new LDSubfile;
 			obj->color = atol (tokens[1]);
 			obj->pos = parseVertex (tokens, 2); // 2 - 4
 			return obj;
 		}
 	default: // Strange line we couldn't parse
-		return new LDGibberish (zLine, "Unknown line code number");
+		return new LDGibberish (line, "Unknown line code number");
 	}
 }
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

Mercurial > ldforge / file comparison

comparison: src/file.cpp

src/file.cpp