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]

Reply via email to