memset定义的问题以及dietlibc库中memcpy实现是不是有问题？

wtan

memset是这样定义的：
void* memset(void *, int, size_t count)

为什么不定义成这样呢？
void* memset(void *, char, size_t count)
这样不是更贴切一些吗？


在dietlibc这个库中memset是这样实现的：
void* memset(void * dst, int s, size_t count) {
    register char * a = dst;
    count++;        /* this actually creates smaller code than using count-- */
    while (--count)
        *a++ = s;
    return dst;
}

register char * a = dst;
上面这一句居然没有进行强制转换。
问一下，这样行不行，dietlibc代码的质量究竟怎样？

群雄逐鹿中原

代码质量不怎样，memset 唯一要求就是快

wtan

像memcpy采用了比较高效率的拷贝方法，memmove、memset之类的只是采用常规的方法，不知为什么要以它们区别对待，要是有说明文档就好了。

jtkk

C语言中char会自动提升为int，也许前人为了避免此种隐式类型提升，才这样做的吧！猜的！

wtan

#define UNALIGNED(x,y) (((unsigned long)x & (sizeof (unsigned long)-1)) ^ ((unsigned long)y & (sizeof (unsigned long)-1)))

#define STRALIGN(x) (((unsigned long)x&3)?4-((unsigned long)x&3):0)

void *
memcpy (void *dst, const void *src, size_t n)
{
    void           *res = dst;
    unsigned char  *c1, *c2;
#ifdef WANT_SMALL_STRING_ROUTINES
    c1 = (unsigned char *) dst;
    c2 = (unsigned char *) src;
    while (n--) *c1++ = *c2++;
    return (res);
#else
    int             tmp;
    unsigned long  *lx1 = NULL;
    const unsigned long *lx2 = NULL;

    if (!UNALIGNED(dst, src) && n > sizeof(unsigned long)) {

        if ((tmp = STRALIGN(dst))) {
            c1 = (unsigned char *) dst;
            c2 = (unsigned char *) src;
            while (tmp-- && n--)
                *c1++ = *c2++;
            if (n == (size_t) - 1)
                return (res);
        }

        lx1 = (unsigned long *) dst;
        lx2 = (unsigned long *) src;

        for (; n >= sizeof(unsigned long); n -= sizeof(unsigned long))
            *lx1++ = *lx2++;
    }

    if (n) {
        c1 = (unsigned char *) (lx1?lx1:dst);
        c2 = (unsigned char *) (lx1?lx2:src);
        while (n--)
            *c1++ = *c2++;
    }

    return (res);
#endif
}


上面是dietlibc库中memcpy函数的原代码，如果两个地址都是对齐的，或者都没有对齐并且相差的数量是不一样的，程序是没有问题的，但是如果两个地址没有对齐并且相差的数量一样，比如一个地址是0x0012ff41，一个地址是0x0012ff61，这个时候就会得到错误的结果，请各位分析一下，是不是这样？或者是我没看明白？

wtan

没有人感兴趣啊，

wtan

C语言中char会自动提升为int，也许前人为了避免此种隐式类型提升，才这样做的吧！猜的！
jtkk 发表于 2010-02-10 13:01

    有道理，反正是要占用这么多空间的。

langue

C 语言的 character literal 是 int 类型。你看 getchar() 的返回值也是 int。

wtan

C 语言的 character literal 是 int 类型。你看 getchar() 的返回值也是 int。
langue 发表于 2010-02-10 16:36

   是啊,好多函数都是这样。

starwing83

其实还是因为C标准规定了C的字面字符常量是个int的缘故……C++就不是这样……