ognarb added a comment.

  In D16721#355367 <https://phabricator.kde.org/D16721#355367>, @anthonyfieroni 
wrote:
  
  > > I'm adding Krita as reviewer, because Karbon is unmaintained and I hope 
someone at Krita as some experience with the toolbox
  >
  > I'm the maintainer of Karbon, for now.
  
  
  Good to know :D
  
  > Please revert foreach -> Q_FOREACH changes, it should be done in separate 
patch. Clean dead / commented code, white-space changes, don't remove QObject 
macro, unless you have a good reason for that. Override is good feature but in 
separate patch for old code, for new one it's good.
  
  I reverted all the 'cosmetic' change. I just have a problem with  the 
Q_OBJECT macro in KoToolBoxLayout_p.h. If I try to add the Q_OBJECT macro I 
have then a linking error. I tried to search when this Q_OBJECT were added and 
I found this commit: R8:703df9162bab8db8a168cca6e4dfec27b8347095 
<https://phabricator.kde.org/R8:703df9162bab8db8a168cca6e4dfec27b8347095>
  
  > Back-porting patches from Krita are welcome but in accepted condition to 
other part of the project.

INLINE COMMENTS

> anthonyfieroni wrote in KoToolBoxDocker.cpp:39
> Why empty or white-space label?

No idea was already like this in the original krita file.

REPOSITORY
  R8 Calligra

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

To: ognarb, #krita, #calligra:_3.0
Cc: anthonyfieroni, Calligra-Devel-list, dcaliste, cochise, vandenoever

Reply via email to