D23618: Use nullptr everywhere
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag added a comment. In D23618#524957 <https://phabricator.kde.org/D23618#524957>, @davidedmundson wrote: > > I nack this till after the upcoming Plasma 5.17 branch-off in three weeks. Then it can go in imo. Other opinions? > > I assume vlad used a script, at which point you should "just" need to apply the same script to your commits before doing a rebase. Yeah, I ran clang-tidy with modernize-use-nullptr check. However, flags need extra care because clang-tidy likes to change 0 to nullptr, e.g. NET::States(0) -> NET::States(nullptr), etc. REPOSITORY R108 KWin BRANCH use-nullptr-everywhere REVISION DETAIL https://phabricator.kde.org/D23618 To: zzag, #kwin, davidedmundson Cc: romangg, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart