diff options
| author | Kristian Evers <kristianevers@gmail.com> | 2018-03-15 08:29:49 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-03-15 08:29:49 +0100 |
| commit | 16618bf5763aa235e01755d4865b455ccb14c928 (patch) | |
| tree | 41c5920fd36155375648dff8fd96d197308091f3 /src/pj_param.c | |
| parent | 30090445886a8f6842028b5ab20601f4c7336fe2 (diff) | |
| parent | 3cc7b4c62382650f9bf00365b25f8fa7f6eed603 (diff) | |
| download | PROJ-16618bf5763aa235e01755d4865b455ccb14c928.tar.gz PROJ-16618bf5763aa235e01755d4865b455ccb14c928.zip | |
Merge pull request #860 from rouault/cppcheck
Fix various issues spotted by cppcheck and clang static analyzer
Diffstat (limited to 'src/pj_param.c')
| -rw-r--r-- | src/pj_param.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/pj_param.c b/src/pj_param.c index 133f3ea6..b52ac940 100644 --- a/src/pj_param.c +++ b/src/pj_param.c @@ -131,17 +131,20 @@ PROJVALUE pj_param (projCtx ctx, paralist *pl, const char *opt) { /* Not found */ if (0==pl) { + /* Return value after the switch, so that the return path is */ + /* taken in all cases */ switch (type) { case 'b': case 'i': value.i = 0; - return value; + break; case 'd': case 'r': value.f = 0.; - return value; + break; case 's': value.s = 0; - return value; + break; } + return value; } /* Found parameter - now find its value */ |