Re: Package add request

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 28.09.2017 00:22, Kevin Kofler wrote:
adel.ks@xxxxxxxxxxxxx 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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx

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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux