make eager-region-commit false by default on windows

src/options.c

@@ -42,19 +42,24 @@ static mi_option_desc_t options[_mi_option_last] =
   { MI_DEBUG, UNINIT, "show_errors" },
   { 0, UNINIT, "verbose" },
 
+  #if MI_SECURE
+  { MI_SECURE, INITIALIZED, "secure" }, // in a secure build the environment setting is ignored
+  #else
+  { 0, UNINIT, "secure" },
+  #endif
+
   // the following options are experimental and not all combinations make sense.
+  { 1, UNINIT, "eager_commit" },        // note: if eager_region_commit is on, this should be on too.
+  #ifdef _WIN32
+  { 0, UNINIT, "eager_region_commit" }, // don't commit too eagerly on windows (just for looks...)
+  #else
+  { 1, UNINIT, "eager_region_commit" },
+  #endif
+  { 0, UNINIT, "large_os_pages" },      // use large OS pages, use only with eager commit to prevent fragmentation of VMA's
   { 0, UNINIT, "page_reset" },
   { 0, UNINIT, "cache_reset" },
-  { 1, UNINIT, "eager_commit" },
-  { 1, UNINIT, "eager_region_commit" }, // eager_commit should be on when eager_region_commit is on
-  { 0, UNINIT, "large_os_pages" },      // use large OS pages, use only with eager commit to prevent fragmentation of VMA's
   { 0, UNINIT, "reset_decommits" },
-  { 0, UNINIT, "reset_discards" },
+  { 0, UNINIT, "reset_discards" }
-  #if MI_SECURE
-  { MI_SECURE, INITIALIZED, "secure" } // in a secure build the environment setting is ignored
-  #else
-  { 0, UNINIT, "secure" }
-  #endif
 };
 
 static void mi_option_init(mi_option_desc_t* desc);