]> granicus.if.org Git - graphviz/commitdiff
Increase stack size for dot MSBUILD project file (avoid crash on larger graphs) smattr/7fb8dc28-80f2-4874-bed4-f670f9c83c1b
authorgreg <gregory.shklover@gmail.com>
Fri, 7 Oct 2022 00:34:47 +0000 (17:34 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 7 Oct 2022 00:37:24 +0000 (17:37 -0700)
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>
CHANGELOG.md
cmd/dot/dot.vcxproj

index bc9c1223eb36d46acd49663cd370e4efb2b00cd7..83a962b56c4ebfb6cb8eb5e8a6e156caa35a3b84 100644 (file)
@@ -6,6 +6,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased (6.0.2)]
 
+### Changed
+
+- Reserved stack size on Windows for the `dot.exe` binary has been increased
+  from the default 1MB to ~3.8MB.
+
 ### Fixed
 
 - Using `aspect` with a custom value in the `packmode` attribute is no longer
index a2b0ca57c460081fdb7ef7709311c8fd80ed0cfa..bbfbe4d2ef626da061102ba1da65d47bd60a3d15 100644 (file)
@@ -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