diff options
| author | Even Rouault <even.rouault@spatialys.com> | 2018-05-29 22:45:18 +0200 |
|---|---|---|
| committer | Even Rouault <even.rouault@spatialys.com> | 2018-05-30 11:48:28 +0200 |
| commit | f773897a3025438326c1131e1586d9ddae080c4f (patch) | |
| tree | 989e9619ac1f1ad140298bef5327df3f2d417d92 /src/PJ_pipeline.c | |
| parent | e692e1567fb6117bd3e1380a80e10b72b7af3710 (diff) | |
| download | PROJ-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.c | 3 |
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; |
