From: greg <gregory.shklover@gmail.com>
Date: Fri, 7 Oct 2022 00:34:47 +0000 (-0700)
Subject: Increase stack size for dot MSBUILD project file (avoid crash on larger graphs)
X-Git-Tag: 7.0.0~7^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e664c0c8a6291e220ec62414c1879fb974c77c85;p=graphviz

Increase stack size for dot MSBUILD project file (avoid crash on larger graphs)

Comment from Matthew Fernandez: It is believed this resolves some instances of
stack overflow in `dfs_enter_outedge` experienced with larger graphs on Windows.

Co-authored-by: Matthew Fernandez <matthew.fernandez@gmail.com>
---

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 484da7f31..204ef18f2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -20,6 +20,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   compiler does not support.
 - The `-ffast-math` compiler flag is no longer enabled by the Autotools build
   system.
+- Reserved stack size on Windows for the `dot.exe` binary has been increased
+  from the default 1MB to ~3.8MB.
 
 ### Fixed
 
diff --git a/cmd/dot/dot.vcxproj b/cmd/dot/dot.vcxproj
index a2b0ca57c..bbfbe4d2e 100644
--- a/cmd/dot/dot.vcxproj
+++ b/cmd/dot/dot.vcxproj
@@ -68,6 +68,7 @@
       <DataExecutionPrevention />
       <TargetMachine>MachineX86</TargetMachine>
       <AdditionalDependencies>cgraph.lib;gvc.lib;%(AdditionalDependencies)</AdditionalDependencies>
+      <StackReserveSize>4000000</StackReserveSize>
     </Link>
     <PostBuildEvent>
       <Command>$(OutDir)dot -c
@@ -98,6 +99,7 @@ copy /B /Y $(OutDir)dot.exe $(OutDir)twopi.exe</Command>
       <DataExecutionPrevention />
       <TargetMachine>MachineX86</TargetMachine>
       <AdditionalDependencies>cgraph.lib;gvc.lib;%(AdditionalDependencies)</AdditionalDependencies>
+      <StackReserveSize>4000000</StackReserveSize>
     </Link>
     <PostBuildEvent>
       <Command>$(OutDir)dot -c