From: "Andrew Jones" <luminous-is AT home DOT com>
Newsgroups: comp.os.msdos.djgpp
References: <88v09h$64n$1 AT nets3 DOT rz DOT RWTH-Aachen DOT DE>
Subject: Re: rectification about switch ... such a big mistake
Lines: 28
X-Priority: 3
X-MSMail-Priority: Normal
X-Newsreader: Microsoft Outlook Express 5.00.2919.6600
X-MimeOLE: Produced By Microsoft MimeOLE V5.00.2919.6600
Message-ID: <YnVs4.72554$45.4010587@news2.rdc1.on.home.com>
Date: Wed, 23 Feb 2000 18:10:00 GMT
NNTP-Posting-Host: 24.42.120.18
X-Complaints-To: abuse AT home DOT net
X-Trace: news2.rdc1.on.home.com 951329400 24.42.120.18 (Wed, 23 Feb 2000 10:10:00 PST)
NNTP-Posting-Date: Wed, 23 Feb 2000 10:10:00 PST
Organization: @Home Network Canada
To: djgpp AT delorie DOT com
DJ-Gateway: from newsgroup comp.os.msdos.djgpp
Reply-To: djgpp AT delorie DOT com

[snip]

>     switch (charact)
>            {
>             case 'A':
>             case 'M':
>                  check=1;
>            };
>     } while ((test!=1)||(check==0));

[snip]

I don't really understand the routine's purpose, but I think you might like to
put a break or two in there somewhere.

switch(charact)
  {
  case 'A': break; /* do nothing ?? */
  case 'M': check = 1; break; /* do something ?? */
  default: break; /* not required, but I always do it for completeness sake */
  } /* semi-colon at end of switch is not required */

Basically, even if A is pressed, your switch is "falling through" to what's
supposed to happen for 'M'.

AndrewJ