D14929: Default to QT_NO_CAST_FROM_ASCII/QT_NO_CAST_TO_ASCII/QT_NO_CAST_FROM_BYTEARRAY

Morten Volden noreply at phabricator.kde.org
Sun Aug 26 12:45:38 BST 2018


volden added a comment.


  In D14929#315654 <https://phabricator.kde.org/D14929#315654>, @volden wrote:
  
  > In D14929#314836 <https://phabricator.kde.org/D14929#314836>, @kossebau wrote:
  >
  > > As discussed on IRC with Kevin, have now worked on fixing this right away, and this can go in now without all the temporary remove_definitions.
  > >  @volden Given you have mentioned you have a series of patches for the perforce plugin, I left that plugin out for now, so you do not run into conflicts. We would then complete the explicit-cast work here later :)
  >
  >
  > @kossebau I actually played around a little with the cast stuff myself. I now have it in a compiling/tests passing condition. Be happy to make a review request of that if you like.
  
  
  For the perforce plugin only that is, I should probably add. :-)

REPOSITORY
  R32 KDevelop

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

To: kossebau, #kdevelop, kfunk
Cc: volden, kfunk, kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180826/1b5f4993/attachment.html>


More information about the KDevelop-devel mailing list