cpp conversion: fix One-Definition-Rule violations

Defining struct pj_opaque with different definitions is a violation
of the C++ One-Definition-Rule. When using link-time optimizations, this
could break badly.
The solution adopted here is to wrap those structures into a C++
anonymous namespace so they are considered different

1 files changed, 2 insertions, 0 deletions

diff --git a/src/PJ_chamb.cpp b/src/PJ_chamb.cpp
index 8d8f05ee..0da3899b 100644
--- a/src/PJ_chamb.cpp
+++ b/src/PJ_chamb.cpp
@@ -7,6 +7,7 @@
 #include "projects.h"
 
 typedef struct { double r, Az; } VECT;
+namespace { // anonymous namespace
 struct pj_opaque {
     struct { /* control point data */
         double phi, lam;
@@ -18,6 +19,7 @@ struct pj_opaque {
     XY p;
     double beta_0, beta_1, beta_2;
 };
+} // anonymous namespace
 
 PROJ_HEAD(chamb, "Chamberlin Trimetric") "\n\tMisc Sph, no inv"
 "\n\tlat_1= lon_1= lat_2= lon_2= lat_3= lon_3=";