D14685: [effects/sheet] Modernize code

Anthony Fieroni <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
anthonyfieroni added inline comments.

INLINE COMMENTS

> sheet.h:73-81
> +inline int SheetEffect::requestedEffectChainPosition() const
>  {
> -public:
> -    WindowInfo();
> -    ~WindowInfo();
> -    bool deleted;
> -    bool added;
> -    bool closed;
> -    QTimeLine *timeLine;
> -    int parentY;
> -};
> +    return 60;
> +}
> +
> +inline int SheetEffect::duration() const
> +{

I think, it's not mandatory, but since you change code style, does it better for function implementation to go in cpp files only.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: anthonyfieroni, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, 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.