diff options
| author | Russ Cox <rsc@swtch.com> | 2021-01-05 23:38:09 -0500 |
|---|---|---|
| committer | Russ Cox <rsc@swtch.com> | 2021-01-05 23:38:09 -0500 |
| commit | 1c845e0bd5ff897dc5e90f2c24db4ecd81a8f60c (patch) | |
| tree | 5b4c0a3ba8784251a08dc46ed3a8f23e6b48cc60 /src/cmd/samterm | |
| parent | 0a513e65607223d11ba94003256b13ef5779e7e8 (diff) | |
| download | plan9port-1c845e0bd5ff897dc5e90f2c24db4ecd81a8f60c.tar.gz plan9port-1c845e0bd5ff897dc5e90f2c24db4ecd81a8f60c.tar.bz2 plan9port-1c845e0bd5ff897dc5e90f2c24db4ecd81a8f60c.zip | |
acme, sam, samterm: remove weird switch usage
For whatever reason all three of these programs
contain switches like:
switch(x) {
case 1:
if(cond)
case 2:
f();
}
Like Duff's device, this is legal C but more obscure
than it really needs to be.
This commit assumes those are intended as written
and simply writes them more clearly. I did consider
that maybe they are mistakes, but in the case of sam/regexp.c,
my rewrite in this commit matches the acme/regx.c that
has been in plan9port since I added acme in 2003.
(I didn't bother to dig up the old Plan 9 releases.)
Assuming acme/regx.c has been correct for the past
two decades, this commit should be correct too.
Diffstat (limited to 'src/cmd/samterm')
| -rw-r--r-- | src/cmd/samterm/flayer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cmd/samterm/flayer.c b/src/cmd/samterm/flayer.c index e9fde31c..a8e70d0c 100644 --- a/src/cmd/samterm/flayer.c +++ b/src/cmd/samterm/flayer.c @@ -169,8 +169,8 @@ newvisibilities(int redraw) break; case V(Some, Some): - if(l->f.b==0 && redraw) case V(None, Some): + if(ov == None || (l->f.b==0 && redraw)) flprepare(l); if(l->f.b && redraw){ flrefresh(l, l->entire, 0); |