zzag added inline comments.

INLINE COMMENTS

> outputconfiguration.cpp:184
> +            uint16_t *s = reinterpret_cast<uint16_t*>(wl_array_add(dest, 
> sizeof(uint16_t)));
> +            *s = (uint16_t)c;
> +        }

Maybe `static_cast`?

I haven't used wl_array but can't you allocate big enough contiguous chunk of 
memory and call memcpy, e.g.

  wl_array wlRed;
  
  wl_array_init(&wlRed);
  auto* redDest = wl_array_add(&wlRed, sizeof(uint16_t) * red.count());
  memcpy(redDest, red.data(), sizeof(uint16_t) * red.count());

REPOSITORY
  R127 KWayland

REVISION DETAIL
  https://phabricator.kde.org/D12388

To: romangg, #frameworks
Cc: zzag, cfeck, michaelh, bruns

Reply via email to