jonab@sdcrdcf.UUCP (Jonathan Biggar) (11/17/84)
System: rn version 4.1 Bug #: 4 Priority: LOW Subject: 2.10.2 disables checkexpired() optimization. Index: rcln.c Prereq: 4.1 Description: Due to abs1st[ngnum] being set earlier (because its value can come from the active file in news 2.10.2), checkexpired() is never being called. This has two effects, neither of which are fatal. First, marking cross-referenced articles will take a bit longer because the average length of line in the .newsrc will be greater. This also affects the load time of the .newsrc, and if it continues over a period of time can cause rn to produce .newsrc lines that are too long for readnews to handle (though rn will). Second, expired articles will be reported as unread until the newsgroup is entered and the articles are found to be missing. Neophyte news users may be discouraged by messages that say there are 3198 unread articles in net.flame, when in fact there are only 57. Repeat-By: Get to a newsgroup in which you have expired but unread articles (you can manufacture such articles by editing your .newsrc). Compare the number of unread articles mentioned on the newsgroup selection level with the number of unread left to read on the first line of the first article in the newsgroup. The latter number should be 1 less than the former number. If not, you have the problem. Fix: From rn, say "| patch -d DIR", where DIR is your rn source directory. Outside of rn, say "cd DIR; patch <thisarticle". If you don't have the patch program, apply the following by hand, or get patch. *** /tmp/,RCSt1002003 Wed Sep 26 09:32:29 1984 --- /tmp/,RCSt2002003 Wed Sep 26 09:32:35 1984 *************** *** 1,4 ! /* $Header: rcln.c,v 4.1 84/09/24 12:04:36 lwall Exp $ * * $Log: rcln.c,v $ * Revision 4.1 84/09/24 12:04:36 lwall --- 1,4 ----- ! /* $Header: rcln.c,v 4.1.1.2 84/09/25 17:20:55 lwall Exp $ * * $Log: rcln.c,v $ * Revision 4.1.1.2 84/09/25 17:20:55 lwall *************** *** 1,6 /* $Header: rcln.c,v 4.1 84/09/24 12:04:36 lwall Exp $ * * $Log: rcln.c,v $ * Revision 4.1 84/09/24 12:04:36 lwall * Real baseline. * --- 1,13 ----- /* $Header: rcln.c,v 4.1.1.2 84/09/25 17:20:55 lwall Exp $ * * $Log: rcln.c,v $ + * Revision 4.1.1.2 84/09/25 17:20:55 lwall + * Getting abs1st from active file before calling checkexpired caused + * checkexpired to not get called. + * + * Revision 4.1.1.1 84/09/25 13:25:34 lwall + * Branch for sdcrdcf changes. + * * Revision 4.1 84/09/24 12:04:36 lwall * Real baseline. * *************** *** 322,327 char tmpbuf[64], *mybuf = tmpbuf; char *nums; int length; ART_NUM ngsize = getngsize(ngnum); ART_NUM unread = ngsize; ART_NUM newmax; --- 329,337 ----- char tmpbuf[64], *mybuf = tmpbuf; char *nums; int length; + #ifdef CACHEFIRST + bool virgin_ng = (!abs1st[ngnum]); + #endif ART_NUM ngsize = getngsize(ngnum); ART_NUM unread = ngsize; ART_NUM newmax; *************** *** 336,342 return; } #ifdef CACHEFIRST ! if (!abs1st[ngnum]) #else if (!toread[ngnum]) #endif --- 346,352 ----- return; } #ifdef CACHEFIRST ! if (virgin_ng) #else if (!toread[ngnum]) #endif