Re: [gnugo-devel] closing processes in matcher_check

[Evan Berggren Daniel]

On Sun, 18 Aug 2002 address@hidden wrote:

>
> > This patch makes matcher_check properly close the pipes it opens to the
> > gtp programs.  This prevents zombie processes being left around when
> > playing multiple games.  It also applies the same fix to twogtp-a.  twogtp
> > probably has the same problem, but I haven't looked at it.
>
> The patch wasn't included in your message.


sigh.  not paying attention.

Index: matcher_check
===================================================================
RCS file: /cvsroot/gnugo/gnugo/interface/gtp_examples/matcher_check,v
retrieving revision 1.5
diff -u -r1.5 matcher_check
--- matcher_check       8 Aug 2002 20:57:49 -0000       1.5
+++ matcher_check       18 Aug 2002 20:34:32 -0000
@@ -199,6 +199,8 @@
        $game_list{$sgffile} = $result;
     }
     $games-- if $games > 0;
+    close $prog_in;
+    close $prog_out;
     print "games remaining: $games\n";
 }

Index: twogtp-a
===================================================================
RCS file: /cvsroot/gnugo/gnugo/interface/gtp_examples/twogtp-a,v
retrieving revision 1.8
diff -u -r1.8 twogtp-a
--- twogtp-a    8 Mar 2002 03:47:03 -0000       1.8
+++ twogtp-a    18 Aug 2002 20:34:32 -0000
@@ -238,6 +238,10 @@
        $game_list{$sgffile} = $resultw . "|" . $resultb
     }
     $games-- if $games > 0;
+    close $black_in;
+    close $black_out;
+    close $white_in;
+    close $white_out;
     print "games remaining: $games\n";
 }