D13228: Qt5.11 fix

Garvit Khatri noreply at phabricator.kde.org
Fri Jun 1 09:48:21 UTC 2018


garvitkhatri added inline comments.

INLINE COMMENTS

> asemke wrote in MatrixView.h:35
> We don't add include in the header if we don't need the full information about the class and it is sufficient to work with "incomplete types". This is the case if we only have pointers or references to objects declared in the header file you try to include. In such cases it is enough to have "forward declaration" in the header file and to include in the cpp file - this helps to reduce the dependencies between the source files and to speed up the compilation times. You can find more information on this if you search for "forward declaration", e.g. here http://en.cppreference.com/w/cpp/language/class

I have read this, will keep on investigating in future. I have made the relevant changes do let me know if they look good. I will push them to master once approved.

REPOSITORY
  R262 LabPlot

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

To: garvitkhatri, asemke
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180601/fd3a600a/attachment.html>


More information about the kde-edu mailing list