cast to pointer from integer of different size [-Wint-to-pointer-cast]

image->data + y * image->bytes_per_line + x * 4

Спасибо.

Имелось ввиду

memcpy((void *) & image->data[y * image->bytes_per_line + x * 4],
           &color, 4);

?

Да уж... 4 байта через memcpy это круто.

gcc оптимизирует, т.к. размер константный

gcc оптимизирует, т.к. размер константный

Да понятно, но сам факт. Говнокод редкостный.

А если запустить на архитектуре, не умеющей запись невыровненных слов? Предложишь более универсальный способ, чем memcpy?

почему не написать например так:

void setcolor(XImage * image, uint32_t color, unsigned x, unsigned y)
{
    (uint32_t)image->data[y * image->bytes_per_line + x * 4] = color;
}

Потому что нельзя. Надо объяснять почему, или если подумать, то до самого дойдёт?

А если запустить на платформе, где биг-ендиан (или странный ендиан) и 64 бита, вообще будет нулями забивать. Надо колор порезать на компоненты битовыми масками и сдвигами, а потом присвоить в нужные чар-смещения. Решения с кастами — эталонный g-code.

Можно

void setcolor(XImage * image, uint32_t color, unsigned x, unsigned y)
{
     * ( (uint32_t*) & image->data[y * image->bytes_per_line + x * 4] ) = color;
}

Ну или

void setcolor(XImage * image, uint32_t color, unsigned x, unsigned y)
{
      ( (uint32_t*) & image->data[y * image->bytes_per_line + x * 4] ) [0] = color;
}

Если ну очень хочется

упс...
тут правильно, да
cast to pointer from integer of different size [-Wint-to-pointer-cast] (комментарий)