Moving template implementation to .tpp #65

New Issue

muellerr · 2020-05-04T16:28:34+02:00

muellerr commented

2020-05-04 16:28:34 +02:00

When writing template classes, function have to be implemented directly in the header.
However, it is still possible to separate the interface (declarations) from the implementation by using a neat trick. I made use of this to make the interface to separate the local pool declarations from the implementation.

Trick taken from: https://stackoverflow.com/questions/495021/why-can-templates-only-be-implemented-in-the-header-file

If using eclipse, this is useful: https://blog.roland-speith.de/?p=1066

IMPORTANT: *.tpp is chosen for a reason, its not a source file. naming it to *.cpp might lead to the build system doing things it should not do

This is not a regular source file. The file is simply included at the end of the header and is still a part of the header. It's just a nice trick to separate interface from implementation.

When writing template classes, function have to be implemented directly in the header. However, it is still possible to separate the interface (declarations) from the implementation by using a neat trick. I made use of this to make the interface to separate the local pool declarations from the implementation. Trick taken from: https://stackoverflow.com/questions/495021/why-can-templates-only-be-implemented-in-the-header-file <br> If using eclipse, this is useful: https://blog.roland-speith.de/?p=1066 IMPORTANT: *.tpp is chosen for a reason, its not a source file. naming it to *.cpp might lead to the build system doing things it should not do This is not a regular source file. The file is simply included at the end of the header and is still a part of the header. It's just a nice trick to separate interface from implementation.

muellerr added the

feature

bug

labels 2020-05-04 16:30:13 +02:00

~~muellerr referenced this issue 2020-05-04 17:11:48 +02:00~~

Storagemanager Refactoring #66

gaisser commented

2020-05-05 20:05:00 +02:00

I like the idea of separating those in two files but I don't see this as a bug.

gaisser referenced a pull request that will close this issue

2020-05-05 20:08:55 +02:00

Storagemanager Refactoring #66