Make some small improvements

CHANGELOG.md

@@ -5,8 +5,8 @@
 - **Added support for DWARF 1 symbol tables.**
 - Improved support for bitfields.
 - Improved support for 128-bit types.
-- Added support for parsing .gnu.linkonce.* section names.
-- Added support for parsing -fdata-sections and -ffunction-sections section names.
+- Added support for parsing `.gnu.linkonce.*` section names.
+- Added support for parsing `-fdata-sections` and `-ffunction-sections` section names.
 - Mac builds compiled for x64 processors are now provided in addition to the arm64 builds.
 - demangle: A newline is now printed at the end of the output.
 - stdump: The `functions` and `globals` subcommands now output a header comment like the `types` command.

src/ccc/mdebug_analysis.cpp

@@ -313,7 +313,6 @@ Result<void> LocalSymbolTableAnalyser::create_function(const char* mangled_name,
 	Result<Function*> function = m_database.functions.create_symbol(mangled_name, m_context.group.source,
 		m_context.group.module_symbol, address, m_context.importer_flags, m_context.demangler);
 	CCC_RETURN_IF_ERROR(function);
-	CCC_ASSERT(*function);
 	m_current_function = *function;
 
 	m_functions.emplace_back(m_current_function->handle());

src/ccc/symbol_database.h

@@ -326,7 +326,7 @@ struct GlobalStorage
 struct RegisterStorage
 {
 	s32 dbx_register_number = -1;
-	bool is_by_reference;
+	bool is_by_reference = false;
 
 	RegisterStorage() {}
 	friend auto operator<=>(const RegisterStorage& lhs, const RegisterStorage& rhs) = default;

src/ccc/util.cpp

@@ -112,14 +112,15 @@ std::string normalise_path(const char* input, bool use_backslashes_as_path_separ
 	}
 
 	// Remove "." and ".." parts.
-	for (s32 i = 0; i < (s32) parts.size(); i++) {
+	for (size_t i = 0; i < parts.size();) {
 		if (parts[i] == ".") {
 			parts.erase(parts.begin() + i);
-			i--;
 		} else if (parts[i] == ".." && i > 0 && parts[i - 1] != "..") {
 			parts.erase(parts.begin() + i);
 			parts.erase(parts.begin() + i - 1);
-			i -= 2;
+			i--;
+		} else {
+			i++;
 		}
 	}