Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

erfam.h is installed #11

Closed
sergiopasra opened this issue Sep 10, 2013 · 4 comments
Closed

erfam.h is installed #11

sergiopasra opened this issue Sep 10, 2013 · 4 comments

Comments

@sergiopasra
Copy link
Contributor

With the current src/Makefile.am, erfam.h is installed alongside erfa.h when the users does make install

Is this correct or erfam.h is an internal header, used for compilation only?

@timj
Copy link
Contributor

timj commented Sep 10, 2013

It should be installed. People rely on it for macros such as DPI and DAS2R.

@sergiopasra
Copy link
Contributor Author

If the macros are public interface, I would expect the names to have a prefix, like ERFA_DPI. The prefix serves as a namespace to avoid colissions. We have very short names here: DC, DAU, etc.

For comparation, math.h macros are prefixed with M_

#define M_PI 3.14159265358979323846
#define M_PIl 3.141592653589793238462643383279502884L

gsl.h macros are prefixed with GSL (or _GSL if they are private macros)

@timj
Copy link
Contributor

timj commented Sep 11, 2013

Indeed, but that is how SLALIB did it and that is how SOFA does it. It's got a long history. I'm not necessarily against an ERFA__ prefix though (if it doesn't complicate the importing script too much).

@sergiopasra
Copy link
Contributor Author

Well, as the prefix macro thing is a different topic, I'm closing this, my original question has been answered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants