9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
* [9fans] Re: nedmail.c
@ 2003-08-05  0:46 boyd
  2003-08-05  1:09 ` David Presotto
  0 siblings, 1 reply; 2+ messages in thread
From: boyd @ 2003-08-05  0:46 UTC (permalink / raw)
  To: 9fans

looks good:

brahma% diff /n/sources/plan9/sys/src/cmd/upas/ned/nedmail.c src/upas/ned
1445d1444
< 	int i;
1494c1493,1499
< 	if(deld)
---
> 	switch(deld){
> 	case 0:
> 		break;
> 	case 1:
> 		Bprint(&out, "!1 message deleted\n");
> 		break;
> 	default:
1496,1500c1501,1502
<
< 	// renumber
< 	i = 1;
< 	for(m = top.child; m != nil; m = m->next)
< 		m->id = natural ? m->fileno : i++;
---
> 		break;
> 	}


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [9fans] Re: nedmail.c
  2003-08-05  0:46 [9fans] Re: nedmail.c boyd
@ 2003-08-05  1:09 ` David Presotto
  0 siblings, 0 replies; 2+ messages in thread
From: David Presotto @ 2003-08-05  1:09 UTC (permalink / raw)
  To: 9fans

[-- Attachment #1: Type: text/plain, Size: 164 bytes --]

Ya, your version was redundant for the switch(deld) case.  The
other difference was for rsc who wanted renumbering in the 'y'
case to make it consistent with 'i'.

[-- Attachment #2: Type: message/rfc822, Size: 2030 bytes --]

From: boyd@sdgm.net
To: 9fans@cse.psu.edu
Subject: [9fans] Re: nedmail.c
Date: Mon, 4 Aug 2003 20:46:08 -0400
Message-ID: <0acb4a8457c0dfaa533230e13afd0d74@sdgm.net>

looks good:

brahma% diff /n/sources/plan9/sys/src/cmd/upas/ned/nedmail.c src/upas/ned
1445d1444
< 	int i;
1494c1493,1499
< 	if(deld)
---
> 	switch(deld){
> 	case 0:
> 		break;
> 	case 1:
> 		Bprint(&out, "!1 message deleted\n");
> 		break;
> 	default:
1496,1500c1501,1502
<
< 	// renumber
< 	i = 1;
< 	for(m = top.child; m != nil; m = m->next)
< 		m->id = natural ? m->fileno : i++;
---
> 		break;
> 	}

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2003-08-05  1:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-08-05  0:46 [9fans] Re: nedmail.c boyd
2003-08-05  1:09 ` David Presotto

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).