Fix and speed-up one of the getMapping() functions

1 files changed, 26 insertions, 10 deletions

diff --git a/src/coordinateoperation.cpp b/src/coordinateoperation.cpp
index 4ada57fb..3fa5ea9c 100644
--- a/src/coordinateoperation.cpp
+++ b/src/coordinateoperation.cpp
@@ -264,21 +264,37 @@ getMappingsFromPROJName(const std::string &projName) {
 
 // ---------------------------------------------------------------------------
 
-const ParamMapping *getMapping(const MethodMapping *mapping,
-                               const OperationParameterValue *param) {
-    const auto &param_name = param->parameter()->name();
-    const std::string &name = *(param_name->description());
-    const std::string &code = param_name->code();
-    const int epsg_code = !code.empty() ? ::atoi(code.c_str()) : 0;
+static const ParamMapping *getMapping(const MethodMapping *mapping,
+                                      const OperationParameterNNPtr &param) {
+    // First try with id
+    const int epsg_code = param->getEPSGCode();
+    if (epsg_code) {
+        for (int i = 0; mapping->params[i] != nullptr; ++i) {
+            const auto *paramMapping = mapping->params[i];
+            if (paramMapping->epsg_code == epsg_code) {
+                return paramMapping;
+            }
+        }
+    }
+
+    // then equivalent name
+    const std::string &name = param->nameStr();
     for (int i = 0; mapping->params[i] != nullptr; ++i) {
         const auto *paramMapping = mapping->params[i];
         if (metadata::Identifier::isEquivalentName(paramMapping->wkt2_name,
-                                                   name.c_str()) ||
-            (epsg_code != 0 && paramMapping->epsg_code == epsg_code) ||
-            areEquivalentParameters(paramMapping->wkt2_name, name)) {
+                                                   name.c_str())) {
+            return paramMapping;
+        }
+    }
+
+    // and finally different name, but equivalent parameter
+    for (int i = 0; mapping->params[i] != nullptr; ++i) {
+        const auto *paramMapping = mapping->params[i];
+        if (areEquivalentParameters(paramMapping->wkt2_name, name)) {
             return paramMapping;
         }
     }
+
     return nullptr;
 }
 
@@ -1002,7 +1018,7 @@ void OperationParameterValue::_exportToWKT(
 void OperationParameterValue::_exportToWKT(io::WKTFormatter *formatter,
                                            const MethodMapping *mapping) const {
     const ParamMapping *paramMapping =
-        mapping ? getMapping(mapping, this) : nullptr;
+        mapping ? getMapping(mapping, d->parameter) : nullptr;
     if (paramMapping && paramMapping->wkt1_name == nullptr) {
         return;
     }