#7 Does not compile for Plasma 5.21 Beta (5.20.90)

[Jan Keith Darunday]

Operating System: Arch Linux
KDE Plasma Version: 5.20.90
KDE Frameworks Version: 5.78.0
Qt Version: 5.15.2
Kernel Version: 5.10.9-arch1-1

It looks like the prePaintWindow function parameter changed its data type from int to std::chrono::milliseconds. I've attached a more complete log as well.

/home/skeithc/.cache/paru/clone/kwin-effect-shapecorners-git/src/shapecorners/shapecorners.cpp:208:48: error: cannot convert ‘int’ to ‘std::chrono::milliseconds’ {aka ‘std::chrono::duration<long int, std::ratio<1, 1000> >’}
  208 |         KWin::effects->prePaintWindow(w, data, time);
      |                                                ^~~~
      |                                                |
      |                                                int
In file included from /home/skeithc/.cache/paru/clone/kwin-effect-shapecorners-git/src/shapecorners/shapecorners.h:23,
                 from /home/skeithc/.cache/paru/clone/kwin-effect-shapecorners-git/src/shapecorners/dbus.h:5,
                 from /home/skeithc/.cache/paru/clone/kwin-effect-shapecorners-git/src/shapecorners/shapecorners.cpp:20:
/usr/include/kwineffects.h:838:102: note:   initializing argument 3 of ‘virtual void KWin::EffectsHandler::prePaintWindow(KWin::EffectWindow*, KWin::WindowPrePaintData&, std::chrono::milliseconds)’
  838 |     virtual void prePaintWindow(EffectWindow* w, WindowPrePaintData& data, std::chrono::milliseconds presentTime) = 0;
      |                                                                            ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
/home/skeithc/.cache/paru/clone/kwin-effect-shapecorners-git/src/shapecorners/shapecorners.cpp:228:44: error: cannot convert ‘int’ to ‘std::chrono::milliseconds’ {aka ‘std::chrono::duration<long int, std::ratio<1, 1000> >’}
  228 |     KWin::effects->prePaintWindow(w, data, time);
      |                                            ^~~~
      |                                            |
      |                                            int

[Jan Keith Darunday]

This seems to be the related change in KDE's github repository containing text on why this change was applied:
https://github.com/KDE/kwin/commit/9f2cb0ae1b6fa6a79158006714907abfac12929a#diff-2f42c80f465471eafc1f0febd9f104a0ef0eef017bb55f1a0c890cebbd4ae78b

Looks like it's related to the new support for per-screen rendering (I think this is the support for multiple framerates per screen)
https://kde.org/announcements/plasma/5/5.20.90/

In addition, we have also added support for mixed-refresh-rate display setups on Wayland, e.g. you can have one screen refreshing at 144Hz and another at 60Hz. Preliminary support for multiple GPUs was also added on Wayland.

[Robert]

Hi, thanks for reporting, I'll fix it once it hits stable.

[Robert]

Fixed