无法理解 C 源代码，它不能在 GCC 中编译，而是在 Visual C++ 中编译。

Question

在 GCC 中我收到以下错误：

aes.c：在函数“copy_block”中：
aes.c:278: 错误：需要左值作为增量操作数
aes.c:278：错误：需要左值作为增量操作数

这是一段代码：

static void copy_block( void * d, void *s, uint_8t nn )
{
    while( nn-- )
        *((uint_8t*)d)++ = *((uint_8t*)s)++;
}

我试图将其更改为可编译版本，但不幸的是，对于我作为 Java 程序员来说，不清楚这里到底发生了什么。

也许有人知道我如何更改它在 GCC 中可编译的源代码，或者有人知道这里详细发生了什么。对我来说，左手值的取消引用似乎很奇怪，但不知怎的，它似乎在 Visual C++ 中完美工作。

这是一个小的遗留程序，我必须将其移植到 Linux 机器上。

提前感谢您的帮助。

Answer 1

试试这个：

#include <string.h>
static void copy_block( void * d, void *s, uint_8t nn ) {
  memcpy(d, s, (size_t)nn);
}

那里所做的事情并不好。

Answer 2

这应该可以做到：

static void copy_block(void *d, void *s, uint_8t nn)
{
    uint_8t *ud = (uint_8t *)d;
    uint_8t *us = (uint_8t *)s;

    while (nn--)
    {
        *ud = *us;
        ++ud;
        ++us;
    }
}

它也更具可读性。

Answer 3

这是一些看起来非常毛茸茸的代码。看起来 *(pointer)++ 子表达式的优先级规则存在问题。 （Visual C++ 接受它介于奇迹和错误之间）它似乎只是一个（可怕的）memcpy 重新实现，所以我建议您使用标准版本：

memcpy(d, s, nn);

我建议在 Visual C++ 中进行快速特征测试，以确保两个版本的代码实际上执行相同的操作（正如它们看起来的那样）。您可能在这里依赖于编译器中的一些奇怪的边缘情况。

Answer 4

问题是强制转换返回右值，但您试图对强制转换返回的指针执行自动增量。试试这个：

uint8_t *da = d, *sa = s;
while(nn--) *da++ = *sa++;

另外两个注意事项：

• nn 可能应该比 uint8_t 大很多，并且可能应该是 size_t。
• 经过这一更改，该函数被命名为 memcpy 并在您的标准库中找到（可能更有效）（我相信在 string.h 中）。

Answer 5

您可能搞砸了括号位置。
您想要增加指针，而不是取消引用的值。

static void copy_block( void * d, void *s, uint_8t nn )
{
    while( nn-- )
        *((uint_8t*)d++) = *((uint_8t*)s++);
}

作为奖励提示，使用 memcpy ...更快！

Answer 6

它是 为什么这个转换的结果不是一个重复的左值？

基本上强制转换不是左值。但它可以与某些编译器一起使用。不仅是旧的 GCC，最近的 Tasking 也“正确”编译并且代码可以工作。

Answer 7

应该是：

static void copy_block( void * d, void *s, uint_8t nn )
{
    while( nn-- )
        (*((uint_8t*)d))++ = (*((uint_8t*)s))++;
}