aboutsummaryrefslogtreecommitdiff
path: root/src/PJ_pipeline.c
diff options
context:
space:
mode:
authorEven Rouault <even.rouault@spatialys.com>2018-05-29 22:45:18 +0200
committerEven Rouault <even.rouault@spatialys.com>2018-05-30 11:48:28 +0200
commitf773897a3025438326c1131e1586d9ddae080c4f (patch)
tree989e9619ac1f1ad140298bef5327df3f2d417d92 /src/PJ_pipeline.c
parente692e1567fb6117bd3e1380a80e10b72b7af3710 (diff)
downloadPROJ-f773897a3025438326c1131e1586d9ddae080c4f.tar.gz
PROJ-f773897a3025438326c1131e1586d9ddae080c4f.zip
Fix warnings found by clang with new warning flags to be added in later commit
Fixes consist in: - no use of comma operator for multi statement purpose - avoid confusing comma in for loops first and third clauses - avoid implicit long to int casts by storing to long, or explicit bound checking before cast
Diffstat (limited to 'src/PJ_pipeline.c')
-rw-r--r--src/PJ_pipeline.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/PJ_pipeline.c b/src/PJ_pipeline.c
index d623423b..618d4688 100644
--- a/src/PJ_pipeline.c
+++ b/src/PJ_pipeline.c
@@ -400,7 +400,8 @@ PJ *OPERATION(pipeline,0) {
set_ellipsoid(P);
/* Now loop over all steps, building a new set of arguments for each init */
- for (i_current_step = i_first_step, i = 0; i < nsteps; i++) {
+ i_current_step = i_first_step;
+ for (i = 0; i < nsteps; i++) {
int j;
int current_argc = 0;
int err;