View Change

4 comments:

• Commit Message:

  • Patch Set #1, Line 16: collapsable

    collapsible

  • Patch Set #1, Line 18: Also clarified

    Also, clarified

    https://writing.wisc.edu/handbook/grammarpunct/conjadv

• File src/lib/string.c:

  • Patch Set #1, Line 50:

    char *strchr(const char *s, int c)
    {
    	for (; *s; s++) {
    		if (*s == c)
    			return (char *)s;
    	}
    	return NULL;
    }
    
    char *strrchr(const char *s, int c)
    {
    	char *p = NULL;
    
    	for (; *s; s++)
    		if (*s == c)
    			p = (char *)s;
    
    	return p;
    }
    

    These two seem wrong. If `c` is 0, then pointer to the terminating zero must be returned, instead NULL would be returned.

  • Patch Set #1, Line 72:

    register char *ret = to;
    	register char data;
    

    `register` is an anachronism, better just drop it

To view, visit change 32901. To unsubscribe, or for help writing mail filters, visit settings.