Subject: Dead and Poor code in the kernel.
To: None <amiga-dev@NetBSD.ORG>
From: Timothy Newsham <newsham@aloha.net>
List: amiga-dev
Date: 01/03/1995 21:12:07
Hi,

  I was doing some reading and came across a few things in
autoconf.c:

int
mbprint(auxp, pnp)
        void *auxp;
        char *pnp;
{
        if (pnp)
                printf("%s at %s", (char *)auxp, pnp);
        return(UNCONF);
}

Doesn't seem to be called from anywhere.

int
matchname(fp, sp)
        char *fp, *sp;
{
        int len;

        len = strlen(fp);
        if (strlen(sp) != len)
                return(0);
        if (bcmp(fp, sp, len) == 0)
                return(1);
        return(0);
}

Seems to be nothing more than a strcmp() returning true or false.
Am I wrong?  Why was this implemented like this?


It looks like some cleanup is in order.

                         Tim N.