aacid added inline comments.

INLINE COMMENTS

> ngraham wrote in delegate.cpp:46
> I'd love to, and I tried, but this stuff is pretty new to me and I kept 
> getting `error: cannot define member function` What's wrong with this?
> 
>   connect(comboBox, &QComboBox::activated,
>           [comboBox]() { emit commitData(comboBox); });

without having tried it, you probably still need the QOverload to say which 
activated version you want, so

connect(comboBox, QOverload<int>::of(&QComboBox::activated), this, [this, 
comboBox] { emit commitData(comboBox); });

> ngraham wrote in delegate.h:41
> `QComboBox::activated` passes an int argument along that as far as I can tell 
> (I could be wrong) needs to be handled by the function it's connected to

nah, the right hand side needs to be a subset of the left hand side (or be able 
to convert from one to another)

REPOSITORY
  R432 File Sharing (Samba) integration

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

To: ngraham, #frameworks, apol
Cc: bruns, aacid

Reply via email to