More deprecated stuff removed

Author: RPGHacker

src/asar/assembleblock.cpp

@@ -1956,13 +1956,12 @@ void assembleblock(const char * block)
 	{
 		const char* current_file = get_current_file_name();
 		string name;
-#ifdef _WIN32
+		// RPG Hacker: Should this also throw on absolute paths?
+		// E.g., on something starting with C:/ or whatever.
 		if (strchr(par, '\\'))
 		{
-			//todo throw error
-			asar_throw_warning(0, warning_id_feature_deprecated, "windows specific paths", "convert paths to crossplatform style");
+			asar_throw_error(0, error_type_block, error_id_platform_paths);
 		}
-#endif
 		name=safedequote(par);
 		assemblefile(name);
 	}
@@ -2002,13 +2001,12 @@ void assembleblock(const char * block)
 		}
 		const char* current_file = get_current_file_name();
 		string name;
-#ifdef _WIN32
+		// RPG Hacker: Should this also throw on absolute paths?
+		// E.g., on something starting with C:/ or whatever.
 		if (strchr(par, '\\'))
 		{
-			//todo throw error
-			asar_throw_warning(0, warning_id_feature_deprecated, "windows specific paths", "convert paths to crossplatform style");
+			asar_throw_error(0, error_type_block, error_id_platform_paths);
 		}
-#endif
 		name = safedequote(par);
 		char * data;//I couldn't find a way to get this into an autoptr
 		if (!readfile(name, current_file, &data, &len)) asar_throw_error(0, error_type_block, vfile_error_to_error_id(asar_get_last_io_error()), name.data());

src/asar/errors.cpp

@@ -298,6 +298,8 @@ static asar_error_mapping asar_errors[] =
 
 	{ ERR(macro_param_outside_macro), "Reference to macro parameter outside of macro" },
 	{ ERR(invalid_depth_resolve), "Invalid %s resolution depth: Trying to backwards-resolve a %s using %i '^', but current scope only supports up to %i '^'." },
+	
+	{ ERR(platform_paths), "Platform-specific paths aren'supported. Please use platform-independent paths (use / instead of \\)." },
 };
 // RPG Hacker: Sanity check. This makes sure that the element count of asar_error
 // matches with the number of constants in asar_error_id. This is important, because

src/asar/errors.h

@@ -274,6 +274,8 @@ enum asar_error_id : int
 	error_id_macro_param_outside_macro,
 	error_id_invalid_depth_resolve,
 
+	error_id_platform_paths,
+
 	error_id_end,
 	error_id_count = error_id_end - error_id_start - 1,
 };

tests/invalid-paths.asm

@@ -0,0 +1,4 @@
+;`errEplatform_paths
+;`errEplatform_paths
+incsrc "include\data.asm"
+incbin "data\filename with spaces.bin"