<table><tr><td style="">asemke added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13228">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13228#inline-68832">View Inline</a><span style="color: #4b4d51; font-weight: bold;">garvitkhatri</span> wrote in <span style="color: #4b4d51; font-weight: bold;">MatrixView.h:35</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Hey just to make sure I don't do this again in future, we add in .cpp when it is a requirement of implementation, and add this to .h file if this is the requirement of both implementation and definition?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">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 <a href="http://en.cppreference.com/w/cpp/language/class" class="remarkup-link" target="_blank" rel="noreferrer">http://en.cppreference.com/w/cpp/language/class</a></p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R262 LabPlot</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13228">https://phabricator.kde.org/D13228</a></div></div><br /><div><strong>To: </strong>garvitkhatri, asemke<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>