D7597: added possibility to specify local path

Kevin Funk noreply at phabricator.kde.org
Wed Aug 30 14:27:36 UTC 2017


kfunk added inline comments.

INLINE COMMENTS

> schnere wrote in uploadprofileitem.h:23
> I've just adapted what was already there.
> 
> It seems the developer of the plugin has used the different Qt::UserRole as keys to store multiple configuration  values in a single QStrandardItem?
> So it might be not a perfect solution, but removing the "keys" would cause a lot of work?

I meant just removing the enum values, not the enums themselves. Hard-coding the values in case you just want normal numbering is not necessary.

Example:
`enum { A = 1, B, C };` => B will be 2, C will be 3 automatically.

REPOSITORY
  R69 KDevelop Upload

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

To: schnere, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170830/b1ce54f2/attachment.html>


More information about the KDevelop-devel mailing list