On Fri, Feb 03, 2006 at 04:52:12PM -0500, Justin Pryzby wrote:
> On Fri, Feb 03, 2006 at 10:41:40PM +0100, Jan Niehusmann wrote:
> > + for (; j<nsize; j++)
> > + nlist[j] = 0;
> Why not:
> memset(nlist+j, 0, (nsize-j)*sizeof(*nlist));
Should work as well, but is harder to read (IMHO).
BTW, look at line 530:
jobs = (JOB **)xrealloc (jobs, (js.j_jobslots * sizeof (JOB *)));
for (j = i; j < js.j_jobslots; j++)
jobs[j] = (JOB *)NULL;
so perhaps, in analogy to the existing code, the best version would be
for(; j<nsize; j++)
nlist[j] = (JOB *)NULL;
Jan
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]