Fixed loop logic.

Continue iterating when encountering invalid nodes.

CMakeLists.txt

@@ -38,7 +38,6 @@ add_executable(DriverDefPaths
 	${DriverDefPaths_SOURCE_FILES}
 )
 target_include_directories(DriverDefPaths PRIVATE
-	${EXPAT_INCLUDE_DIRS}
 	${DriverDefPaths_SOURCE_DIR}/Include
 )
 target_link_libraries(DriverDefPaths PUBLIC expat)

Source/Main.c

@@ -107,21 +107,18 @@ int main(int argc, char** argv) {
 					xml_attribute* pPathAttribute = XmlNodeFindAttribute(pDriverPackageFileNode, "Path");
 					xml_attribute* pNameAttribute = XmlNodeFindAttribute(pDriverPackageFileNode, "Name");
 					if (!pPathAttribute || !pNameAttribute)
-						break;
+						continue;
 
 					char* sDriverPath = pPathAttribute->sContent;
 					char* sDriverName = pNameAttribute->sContent;
 					if (!sDriverPath || !sDriverName)
-						break;
+						continue;
 
 					// Remove "$(mspackageroot)".
 					// "$(mspackageroot)" is always at the start of the path
 					// because that's the only place it makes sense.
 					size_t MprLength = strlen_literal("$(mspackageroot)");
-					if (
-						(strlen(sDriverPath) >= MprLength) &&
-						(strncmp(sDriverPath, "$(mspackageroot)", MprLength) == 0)
-					)
+					if (strncmp(sDriverPath, "$(mspackageroot)", MprLength) == 0)
 						sDriverPath += MprLength;
 
 					// Remove trailing backslash.

Source/XmlNode.c

@@ -52,16 +52,6 @@ void XmlNodeFree(xml_node* pNode) {
 	free(pNode);
 }
 
-/*
-void XmlNodeSetContent(xml_node* pNode, const char* sContent) {
-	assert(pNode != NULL);
-
-	size_t ContentSize = (strlen(sContent) + 1) * sizeof(*sContent);
-	pNode->sContent = realloc_guarded(pNode->sContent, ContentSize);
-	memcpy(pNode->sContent, sContent, ContentSize);
-}
-*/
-
 void XmlNodeAddChildren(xml_node* pParentNode, xml_node* pChildrenNode) {
 	assert(pParentNode != NULL);
 	assert(pChildrenNode != NULL);