[groff] 10/74: src/preproc/html/pre-html.cpp: Annotate issue.

[G. Branden Robinson]

gbranden pushed a commit to branch branden-2023-04-28
in repository groff.

commit b29466689b6f06093d3b9443e17b5718d1ec4da4
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
AuthorDate: Sun Apr 16 01:03:30 2023 -0500

    src/preproc/html/pre-html.cpp: Annotate issue.
    
    Throw more light on this troublesome code.
---
 src/preproc/html/pre-html.cpp | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/preproc/html/pre-html.cpp b/src/preproc/html/pre-html.cpp
index fecfb0539..6bfab94b0 100644
--- a/src/preproc/html/pre-html.cpp
+++ b/src/preproc/html/pre-html.cpp
@@ -70,6 +70,14 @@
 
 // Most MS-DOS and Win32 environments will be missing the 'fork'
 // capability (some, like Cygwin, have it, but it is better avoided).
+//
+// Bruno Haible adds in 2023:
+//   However, because on Cygwin, neither ... __MSDOS__ [nor] _WIN32 is
+//   defined, the code [here] *will* use fork() on Cygwin.  Which works
+//   fine, but is known to be slow.  However, I don't know whether the
+//   native Windows code (_WIN32) will work on Cygwin: many native
+//   Windows APIs don't work right from within Cygwin, because the
+//   worlds inside and outside a Cygwin process are quite different.
 
 # define MAY_FORK_CHILD_PROCESS 0