Changeset View
Changeset View
Standalone View
Standalone View
toplevel.cpp
Show First 20 Lines • Show All 256 Lines • ▼ Show 20 Line(s) | |||||
257 | 257 | | |||
258 | void Toplevel::setReadyForPainting() | 258 | void Toplevel::setReadyForPainting() | ||
259 | { | 259 | { | ||
260 | if (!ready_for_painting) { | 260 | if (!ready_for_painting) { | ||
261 | ready_for_painting = true; | 261 | ready_for_painting = true; | ||
262 | if (compositing()) { | 262 | if (compositing()) { | ||
263 | addRepaintFull(); | 263 | addRepaintFull(); | ||
264 | emit windowShown(this); | 264 | emit windowShown(this); | ||
265 | if (Client *cl = dynamic_cast<Client*>(this)) { | 265 | if (auto *cl = dynamic_cast<AbstractClient*>(this)) { | ||
zzag: While we're on this, would it be feasible to use qobject_cast instead? | |||||
graesslin: Is there any real advantage of qobject_cast? | |||||
Supposedly it's faster for QObjects. Though not really enough that it'll make a noticeable difference, especially here. davidedmundson: Supposedly it's faster for QObjects.
Though not really enough that it'll make a noticeable… | |||||
266 | if (cl->tabGroup() && cl->tabGroup()->current() == cl) | 266 | if (cl->tabGroup() && cl->tabGroup()->current() == cl) | ||
267 | cl->tabGroup()->setCurrent(cl, true); | 267 | cl->tabGroup()->setCurrent(cl, true); | ||
268 | } | 268 | } | ||
269 | } | 269 | } | ||
270 | } | 270 | } | ||
271 | } | 271 | } | ||
272 | 272 | | |||
273 | void Toplevel::deleteEffectWindow() | 273 | void Toplevel::deleteEffectWindow() | ||
▲ Show 20 Lines • Show All 297 Lines • Show Last 20 Lines |
While we're on this, would it be feasible to use qobject_cast instead?