🚀 go-pugleaf

Honey Danber uucp has the following code fragment to execute when closing
down the connection:

from cico.c
		.
		.
		.
	(void) alarm(2 * MAXMSGTIME);	/* give slow guys some thrash time */
	omsg('O', "OOOOO", Ofn);
	CDEBUG(4, "send OO %d,", ret);
	if (!setjmp(Sjbuf)) {
		for (;;) {
			omsg('O', "OOOOO", Ofn);
			ret = imsg(msg, Ifn);
			if (ret != 0)
				break;
			if (msg[0] == 'O')
				break;
		}
	}
	(void) alarm(0);
		.
		.
		.

Why do two consecutive omg('O',"OOOOO", Ofn) calls before calling imsg() ?

It has caused problems on a (non-TTY) channel with scarce buffering resources.
I've fixed the behavior of the channel but wonder if there is a reason for
the double omsg(), or whether putting omsg at the bottom of the for(;;) loop
would work as well.  Are there compatibility problems with other verions if
such a change is made?

	Mike Wescott
	ncrcae!wescott