Review Request 129764: add missing include math.h to fix the build

Burkhard Lück lueck at hube-lueck.de
Wed Jan 4 07:33:14 UTC 2017


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129764/
-----------------------------------------------------------

(Updated Jan. 4, 2017, 7:33 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Games and Alexander Semke.


Changes
-------

Submitted with commit 732f9a4c7f13792164073cc3198d2237b7df4420 by Burkhard Lück to branch master.


Repository: knights


Description
-------

does not build without patch -> error was: ‘ceil’ was not declared in this scope
alternatively QtMath/qCeil could be used, what is preferred?


Diffs
-----

  src/historywidget.cpp b78772f 

Diff: https://git.reviewboard.kde.org/r/129764/diff/


Testing
-------

builds


Thanks,

Burkhard Lück

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20170104/5535f269/attachment.html>


More information about the kde-games-devel mailing list