Attachment 312590 Details for Bug 435159 – new savelist removal

[patch] new savelist removal

ksh-20080202-nosavelist.patch (text/plain), 2.03 KB, created by James M. Leddy on 2008-07-24 19:44:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: James M. Leddy

Created: 2008-07-24 19:44:06 UTC

Size: 2.03 KB

patch

obsolete

>--- ksh-20080202/src/cmd/ksh93/sh/jobs.c	2008-07-22 16:26:41.000000000 -0400
>+++ ksh-20080202/src/cmd/ksh93/sh/jobs.c.nosavelist	2008-07-22 16:26:13.000000000 -0400
>@@ -43,8 +43,6 @@
> #   define WIFCONTINUED(wstat)	(0)
> #endif
> 
>-#define	NJOB_SAVELIST	4
>-
> /*
>  * temporary hack to get W* macros to work
>  */
>@@ -61,34 +59,12 @@ struct jobsave
> 	unsigned short	exitval;
> };
> 
>-static struct jobsave *job_savelist;
>-static int njob_savelist;
>-
>-static void init_savelist(void)
>-{
>-	register struct jobsave *jp;
>-	while(njob_savelist < NJOB_SAVELIST)
>-	{
>-		jp = newof(0,struct jobsave,1,0);
>-		jp->next = job_savelist;
>-		job_savelist = jp;
>-		njob_savelist++;
>-	}
>-}
>-
> /*
>  * return next on link list of jobsave free list
>  */
> static struct jobsave *jobsave_create(pid_t pid)
> {
>-	register struct jobsave *jp = job_savelist;
>-	if(jp)
>-	{
>-		njob_savelist--;
>-		job_savelist = jp->next;
>-	}
>-	else
>-		jp = newof(0,struct jobsave,1,0);
>+	register struct jobsave *jp = newof(0,struct jobsave,1,0);
> 	if(jp)
> 		jp->pid = pid;
> 	return(jp);
>@@ -421,8 +397,6 @@ void job_init(int lflag)
> #   if defined(SIGCLD) && (SIGCLD!=SIGCHLD)
> 	signal(SIGCLD,job_waitsafe);
> #   endif
>-	if(njob_savelist < NJOB_SAVELIST)
>-		init_savelist();
> 	if(!sh_isoption(SH_INTERACTIVE))
> 		return;
> 	/* use new line discipline when available */
>@@ -1052,8 +1026,6 @@ void	job_clear(void)
> 		free((void*)jp);
> 	}
> 	bck.list = 0;
>-	if(njob_savelist < NJOB_SAVELIST)
>-		init_savelist();
> 	job.pwlist = NIL(struct process*);
> 	job.numpost=0;
> 	job.waitall = 0;
>@@ -1076,8 +1048,6 @@ int job_post(pid_t pid, pid_t join)
> 	register struct process *pw;
> 	register History_t *hp = sh.hist_ptr;
> 	sh.jobenv = sh.curenv;
>-	if(njob_savelist < NJOB_SAVELIST)
>-		init_savelist();
> 	if(job.toclear)
> 	{
> 		job_clear();
>@@ -1676,14 +1646,7 @@ static int job_chksave(register pid_t pi
> 		else
> 			bck.list = jp->next;
> 		bck.count--;
>-		if(njob_savelist < NJOB_SAVELIST)
>-		{
>-			njob_savelist++;
>-			jp->next = job_savelist;
>-			job_savelist = jp;
>-		}
>-		else
>-			free((void*)jp);
>+		free((void*)jp);
> 	}
> 	return(r);
> }

Actions: View | Diff

Attachments on bug 435159: 310235 | 311037 | 311902 | 312590 | 323354