davidedmundson added a comment.
Looks pretty good. +1
INLINE COMMENTS
> outputdevice.cpp:323
> + uint16_t *curvePtr = reinterpret_cast<uint16_t *>(curve->data);
> + for (size_t i = 0; i < curve->size / sizeof(uint16_t); i++) {
> + destination->append(curvePtr[i]);
You can do this all in once
destination->resize(curve->size);
memcpy(...)
REPOSITORY
R127 KWayland
REVISION DETAIL
https://phabricator.kde.org/D12388
To: romangg, #frameworks
Cc: davidedmundson, zzag, cfeck, michaelh, bruns
