Action #834
Feature #833: Add functions to public Gammalib API
Decide on naming convention for functions and how to expose them in C++ (namespace or prefix)
Status: | Closed | Start date: | 04/12/2013 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assigned To: | - | % Done: | 100% | |
Category: | - | |||
Target version: | 00-08-00 | |||
Duration: |
Recurrence
No recurrence.
History
#1 Updated by Knödlseder Jürgen over 11 years ago
- Target version set to 00-08-00
#2 Updated by Knödlseder Jürgen over 11 years ago
gamma
seems not to be a possible namespace:
../../include/GTools.hpp:37: error: 'namespace gamma { }' redeclared as different kind of symbol /usr/include/architecture/i386/math.h:602: error: previous declaration of 'double gamma(double)'
See for example: http://gcc.1065356.n5.nabble.com/Bug-libstdc-49762-New-symbols-not-kept-in-std-namespace-td301112.html
I guess we cannot use gamma
#3 Updated by Knödlseder Jürgen over 11 years ago
Here a couple of alternatives to gamma
:
gammalib gamlib glib gamlb gam gal glbI guess I prefer
gammalib
at the end as it’s explicit, probably rather unique. Drawback: it’s rather long.#4 Updated by Deil Christoph over 11 years ago
+1 for explicit gammalib
#5 Updated by Knödlseder Jürgen over 11 years ago
Here an interesting discussion about C++ namespaces: http://stackoverflow.com/questions/603378/c-namespace-usage-and-naming-rules
#6 Updated by Knödlseder Jürgen over 11 years ago
- % Done changed from 0 to 20
Christoph Deil wrote:
+1 for explicit
gammalib
Okay, so let’s go with this.
I’ll start implementing the gammalib
namespace for the GTools
constants first.
#7 Updated by Knödlseder Jürgen over 11 years ago
- Status changed from New to Closed
- % Done changed from 20 to 100
- Remaining (hours) set to 0.0
Use gammalib
namespace.