Update formatting and comments. Updated todo

Author: Neko-Box-Coder

Src/runcpp2/CompilingLinking.cpp

@@ -312,7 +312,8 @@ namespace
                         i,
                         &profile,
                         &currentOutputTypeInfo,
-                        substitutionMap,
+                        substitutionMap,    //NOTE: substitutionMap is used by the next source file, 
+                                            //      therefore need to copy it.
                         &buildDir,
                         compileAsExecutable,
                         &scriptInfo,
@@ -437,7 +438,7 @@ namespace
 
                         return true;
                     }
-                )
+                ) //std::async
             ); //actions.emplace_back
             finished.emplace_back(false);
 
@@ -491,8 +492,8 @@ namespace
 
                 actions.clear();
                 finished.clear();
-            }
-        }
+            } //if(actions.size() >= maxThreads || i == sourceFiles.size() - 1)
+        } //for(int i = 0; i < sourceFiles.size(); ++i)
 
         ssLOG_OUTPUT_ALL_CACHE_GROUPED();
         return !failedAny;

Src/runcpp2/DependenciesHelper.cpp

@@ -234,8 +234,16 @@ namespace
 
                 dependencies.at(i) 
                             ->AbsoluteIncludePaths
-                            .push_back(ghc::filesystem::absolute(   dependenciesCopiesPaths.at(i) + "/" + 
-                                                                    dependencies.at(i)->IncludePaths.at(j)).string());
+                            .push_back
+                            (
+                                ghc::filesystem::absolute
+                                (
+                                    dependenciesCopiesPaths.at(i) + 
+                                    "/" + 
+                                    dependencies.at(i)->IncludePaths.at(j)
+                                )
+                                .string()
+                            );
 
                 ssLOG_DEBUG("Include path added: " << dependencies.at(i)->AbsoluteIncludePaths.back());
             }

Src/runcpp2/runcpp2.cpp

@@ -502,7 +502,7 @@ runcpp2::StartPipeline( const std::string& scriptPath,
         if(result != PipelineResult::SUCCESS)
             return result;
 
-        //Check if script info has changed if provided
+        //Check if script info has changed if provided and run setup if needed
         bool recompileNeeded = false;
         bool relinkNeeded = false;
         std::vector<std::string> changedDependencies;

mkdocs/docs/TODO.md

@@ -17,6 +17,7 @@
 - Allow runcpp2 to be library for scriptable pipeline
 - Add ability to reference local YAML for user config
 - Add version for default user config and prompt for update
+- Add platform map for PreferredProfile for user config
 - Add more default profiles
 
 ### v0.4.0
@@ -36,6 +37,44 @@
 ## Planned But Low Priority
 
 - Smoother CMake support by reading cmake target properties (https://stackoverflow.com/a/56738858/23479578)
+<!--
+if(NOT CMAKE_PROPERTY_LIST)
+    execute_process(COMMAND cmake --help-property-list OUTPUT_VARIABLE CMAKE_PROPERTY_LIST)
+    
+    # Convert command output into a CMake list
+    string(REGEX REPLACE ";" "\\\\;" CMAKE_PROPERTY_LIST "${CMAKE_PROPERTY_LIST}")
+    string(REGEX REPLACE "\n" ";" CMAKE_PROPERTY_LIST "${CMAKE_PROPERTY_LIST}")
+    list(REMOVE_DUPLICATES CMAKE_PROPERTY_LIST)
+endif()
+    
+function(print_properties)
+    message("CMAKE_PROPERTY_LIST = ${CMAKE_PROPERTY_LIST}")
+endfunction()
+    
+function(print_target_properties target)
+    if(NOT TARGET ${target})
+      message(STATUS "There is no target named '${target}'")
+      return()
+    endif()
+
+    foreach(property ${CMAKE_PROPERTY_LIST})
+        string(REPLACE "<CONFIG>" "${CMAKE_BUILD_TYPE}" property ${property})
+
+        # Fix https://stackoverflow.com/questions/32197663/how-can-i-remove-the-the-location-property-may-not-be-read-from-target-error-i
+        if(property STREQUAL "LOCATION" OR property MATCHES "^LOCATION_" OR property MATCHES "_LOCATION$")
+            continue()
+        endif()
+
+        get_property(was_set TARGET ${target} PROPERTY ${property} SET)
+        if(was_set)
+            get_target_property(value ${target} ${property})
+            message("${target} ${property} = ${value}")
+        endif()
+    endforeach()
+endfunction()
+
+print_target_properties(matplot)
+-->
 - Add the ability to specify different profiles(?)/defines for different source files
 - Handle escape characters at the end
     - To avoid situation like this: