bshah added inline comments.

INLINE COMMENTS

> broulik wrote in notification.h:128
> Maybe add a new `rawHints() const`?
> 
> Not a fan of ref return

I avoided using "raw" word in the function name, because when we process hints 
we don't actually modify it. We just process rest of notifications data, so it 
is still same thing.

I will drop ref return however.

> broulik wrote in server_p.cpp:232
> Can you use generated XML interface for this, maybe?

It would introduce weird dependency loop so I'd rather not.

REPOSITORY
  R120 Plasma Workspace

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

To: bshah, #plasma, broulik, davidedmundson
Cc: nicolasfella, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to