diff options
| author | Aaron Puchert <aaron.puchert@sap.com> | 2017-11-06 17:52:29 +0100 |
|---|---|---|
| committer | Aaron Puchert <aaron.puchert@sap.com> | 2017-11-06 18:04:54 +0100 |
| commit | c0c95e33abf2f724e5827cbf41ede41e2939e188 (patch) | |
| tree | e14c3a6ce8322d3185670ce51d7a376152c25678 /src/PJ_laea.c | |
| parent | e7e86d14f86eb66365d6aa725c5e9b496b3ba7f7 (diff) | |
| download | PROJ-c0c95e33abf2f724e5827cbf41ede41e2939e188.tar.gz PROJ-c0c95e33abf2f724e5827cbf41ede41e2939e188.zip | |
Use enumerations where appropriate
Enumerations have the following advantages over #define:
- they clearly connect a variable and the allowed constants,
- the meaning of code is not obfuscated by integer values,
- they are visible to the compiler, which can warn about (possibly)
incorrect usage.
There should be no functional change.
Diffstat (limited to 'src/PJ_laea.c')
| -rw-r--r-- | src/PJ_laea.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/PJ_laea.c b/src/PJ_laea.c index 270b8dc8..0c5d5db9 100644 --- a/src/PJ_laea.c +++ b/src/PJ_laea.c @@ -5,6 +5,13 @@ PROJ_HEAD(laea, "Lambert Azimuthal Equal Area") "\n\tAzi, Sph&Ell"; +enum Mode { + N_POLE = 0, + S_POLE = 1, + EQUIT = 2, + OBLIQ = 3 +}; + struct pj_opaque { double sinb1; double cosb1; @@ -15,16 +22,12 @@ struct pj_opaque { double dd; double rq; double *apa; - int mode; + enum Mode mode; }; #define EPS10 1.e-10 #define NITER 20 #define CONV 1.e-10 -#define N_POLE 0 -#define S_POLE 1 -#define EQUIT 2 -#define OBLIQ 3 static XY e_forward (LP lp, PJ *P) { /* Ellipsoidal, forward */ XY xy = {0.0,0.0}; |
