diff options
| author | jthorn <jthorn@0f49ee68-0e4f-0410-9b9c-b2c123ded7ef> | 2005-08-24 10:04:04 +0000 |
|---|---|---|
| committer | jthorn <jthorn@0f49ee68-0e4f-0410-9b9c-b2c123ded7ef> | 2005-08-24 10:04:04 +0000 |
| commit | 9afee299246e85290d32ba81d37905b8549f84bf (patch) | |
| tree | e62d8d4d200c037ced50fd16ea5cc57e83030621 | |
| parent | 0b54751a550481eb5db8eb06e85704170cf6cf87 (diff) | |
fix a bug found by Thomas Radke:
> AEILocalInterp/src/InterpLocalUniform.c:2058 where the return statement is
> marked as an "/*** ERROR RETURN ***/" but is actually also reached in the
> non-error case. I believe there are some curly brackets missing, this return
> statement needs to be enclosed in the preceding if-block along with the
> CCTK_VWarn() call.
git-svn-id: http://svn.aei.mpg.de/numrel/AEIThorns/AEILocalInterp/trunk@29 0f49ee68-0e4f-0410-9b9c-b2c123ded7ef
| -rw-r--r-- | src/InterpLocalUniform.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/InterpLocalUniform.c b/src/InterpLocalUniform.c index 860e085..9e7653e 100644 --- a/src/InterpLocalUniform.c +++ b/src/InterpLocalUniform.c @@ -2047,7 +2047,8 @@ if (p_error_info->found_per_point_status) -p_error_info->error_count, "error_point_status"); if (status < 0) - then CCTK_VWarn(ERROR_MSG_SEVERITY_LEVEL, + then { + CCTK_VWarn(ERROR_MSG_SEVERITY_LEVEL, __LINE__, __FILE__, CCTK_THORNSTRING, "\n" " CCTK_InterpLocalUniform():\n" @@ -2056,6 +2057,7 @@ if (p_error_info->found_per_point_status) , status); return status; /*** ERROR RETURN ***/ + } } return 0; /*** NORMAL RETURN ***/ |