D15625: Sublime: Fix crash caused when all tool view items are small

[René J.V. Bertin]

rjvbb accepted this revision.
rjvbb added a comment.
This revision is now accepted and ready to land.


  Looks like you nailed it this time.
  
  However, maximumWidth|Height can still be 0 when rect.width()|height() == buttonSpacing * (_items.size() - 1) (and given the values I've seen that doesn't seem outlandish). And that's not even a requirement; if all items have 0 width or height you will still end up without any small items.
  
  In other words, maybe you shouldn't justify the fallback with potential "latent miscalculations" but simply with the observation that you cannot guarantee that there will be at least 1 large item.

REPOSITORY
  R32 KDevelop

BRANCH
  segfault-fix

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

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