diff options
| author | Even Rouault <even.rouault@spatialys.com> | 2021-09-05 15:41:40 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-05 15:41:40 +0200 |
| commit | 71b372e6f08b2f40fbd043c80b56bdb8d2c0b5a0 (patch) | |
| tree | eaae225629ae6101db6094b046f319df4f6aba7a /src/transformations/helmert.cpp | |
| parent | 09367628bfe698d6a73a1b928bcf611ea675103d (diff) | |
| parent | b91af0075a7e8a189e2cd443a823a0798e0b9ed9 (diff) | |
| download | PROJ-71b372e6f08b2f40fbd043c80b56bdb8d2c0b5a0.tar.gz PROJ-71b372e6f08b2f40fbd043c80b56bdb8d2c0b5a0.zip | |
Merge pull request #2841 from rouault/cppcheck_fixes
Cppcheck fixes
Diffstat (limited to 'src/transformations/helmert.cpp')
| -rw-r--r-- | src/transformations/helmert.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/transformations/helmert.cpp b/src/transformations/helmert.cpp index f9bb45e0..253dc2f6 100644 --- a/src/transformations/helmert.cpp +++ b/src/transformations/helmert.cpp @@ -369,7 +369,8 @@ static PJ_XYZ helmert_forward_3d (PJ_LPZ lpz, PJ *P) { point.lpz = lpz; if (Q->fourparam) { - point.xy = helmert_forward(point.lp, P); + const auto xy = helmert_forward(point.lp, P); + point.xy = xy; return point.xyz; } @@ -409,7 +410,8 @@ static PJ_LPZ helmert_reverse_3d (PJ_XYZ xyz, PJ *P) { point.xyz = xyz; if (Q->fourparam) { - point.lp = helmert_reverse(point.xy, P); + const auto lp = helmert_reverse(point.xy, P); + point.lp = lp; return point.lpz; } @@ -448,7 +450,8 @@ static PJ_COORD helmert_forward_4d (PJ_COORD point, PJ *P) { build_rot_matrix(P); } - point.xyz = helmert_forward_3d (point.lpz, P); + const auto xyz = helmert_forward_3d (point.lpz, P); + point.xyz = xyz; return point; } @@ -466,7 +469,8 @@ static PJ_COORD helmert_reverse_4d (PJ_COORD point, PJ *P) { build_rot_matrix(P); } - point.lpz = helmert_reverse_3d (point.xyz, P); + const auto lpz = helmert_reverse_3d (point.xyz, P); + point.lpz = lpz; return point; } |
