D28445: Reduce the amount of objects that are compiled several times

Aleix Pol Gonzalez <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
apol added a comment.


  In D28445#638781 <https://phabricator.kde.org/D28445#638781>, @zzag wrote:
  
  > Do you have stats that show how much time it takes to compile kwin w/ and w/o this change?
  
  
  The problem isn't build time per se, which also is, but that every time you save a file you'll get changes in few hundred objects.
  You can try it easily by modifying cursors.cpp or screens.cpp on a ready build directory and triggering a build. There will be few hundred build steps to be done (on my laptop which isn't that bad it's easily 4 or 5 minutes to wait).

INLINE COMMENTS

> anthonyfieroni wrote in CMakeLists.txt:376
> It should be installed or static.

Why, no?

> zzag wrote in utils.cpp:80-85
> Why were these two moved?

Because they need to be out of the `#ifndef KCMRULES`.

REPOSITORY
  R108 KWin

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

To: apol, #kwin
Cc: zzag, anthonyfieroni, iasensio, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.