On 28.09.2017 00:22, Kevin Kofler wrote:
adel...@zegrapher.com wrote:
and Boost for only one small function

Since I was curious, I looked for that function and found it in
polynomial.cpp. But why do you include the headers in polynomial.h? The
usage of Boost in Polynomial::translateX is an implementation detail and so
the headers should be included only in the implementation file
polynomial.cpp.

Of course, this is not a big issue, it only increases the compile time by
some fraction of a second, but I wanted to give you the hint anyway.

        Kevin Kofler
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org

Yea well, since I used the convention of including libs only in the .h I didn't do so. But yes you're right! I will update that.
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org

Reply via email to