2014-07-22 19:10 GMT+02:00 Bruno Chareyre <[email protected]>:
> Ah, now I see that most YADE_PLUGIN are back in MatchMaker.cpp. I was
> not expecting that.
> Not really where I would look for them. Though I don't see an obvious
> solution.
> Why is GenericSphereContact in DomainLimiter.cpp instead?
> Maybe a common.cpp?

Sorry, it was temporal solution, I will move all of those plugins
into a common.cpp.

> Overall, I would not try too hard to eliminate the cpp's.
> Implementation in hpp is not good practice, and this is not where we
> gain compilation time (instead, it is likely to be slower if the same
> thing is compiled many times).
> Merging is what makes things faster.

Agree, but I did not move a lot of code into .hpp. Mostly just
default constructors and destructors. Yes, there were some
functions, which are now in .hpp, se we can move them into a
common.cpp.

Anton

_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yade-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to