<< Previous Message Main Index Next Message >>
<< Previous Message in Thread This Month Next Message in Thread >>
Date   : Sat, 02 Jul 1983 20:31:11-EDT
From   : goldfarb.ucf-cs@rand-relay
Subject: umodem fix

There is a slight problem with the latest version of umodem (umodem32.c).
A modification to set a logfile default was done in this version and it
will fail to work on some systems because the modifier used a bitwise
instead of a logical 'NOT' to flip a toggle.  The fix is probably not
worth changing the revision number, but I did so  here to avoid confusion
(and besides, I use RCS).  Following is a diff between 3.2 and 3.3 (disregard
the first line, which is RCS info).

1c1
< static char *Rcsid = "$Header: /usr1/grad/goldfarb/src/umodem/RCS/umodem.c,v
3.2 83/07/02 19:26:08 goldfarb Exp $";
---
> static char *Rcsid = "$Header: /usr1/grad/goldfarb/src/umodem/RCS/umodem.c,v
3.3 83/07/02 20:23:23 goldfarb Exp $";
76c76,84
<  *                no logging.
---
>  *                logging.
>  *           -- Version 3.3 Mod by Ben Goldfarb, 07/02/83
>  *                . Corrected problem with above implementation of "LOGDEFAULT".
>  *                A bitwise, instead of a logical, negation operator was
>  *                  used to complement LOGFLAG when the '-l' command line flag
>  *                  was specified.  This can cause LOGFLAG to be true when it
>  *                  should be false.  While I don't consider a one character 
>  *                  replacement to be worthy of a minor revision number, this
>  *                  will prevent confusion.
216c224
< #if LOGDEFAULT
---
> #if LOGDEFAULT == 1
241c249
<         LOGFLAG = LOGDEFAULT;
---
>         LOGFLAG = LOGDEFAULT; /* open (don't open) log file */
259c267
<                 case 'l' : LOGFLAG = ~LOGDEFAULT;  /* turn off log report */
---
>                 case 'l' : LOGFLAG = !LOGDEFAULT;  /* turn off log report */
<< Previous Message Main Index Next Message >>
<< Previous Message in Thread This Month Next Message in Thread >>