Take advantage of fact that free and realloc can handle null pointers

to simplify code.
Fixes CactusBase/1509.
Also fixes bug in reallocation of width_alldirs[].


git-svn-id: http://svn.cactuscode.org/arrangements/CactusBase/Boundary/trunk@234 6a38eb6e-646e-4a02-a296-d141613ad6c4

1 files changed, 2 insertions, 6 deletions

diff --git a/src/RobinBoundary.c b/src/RobinBoundary.c
index c60f020..7c0caa5 100644
--- a/src/RobinBoundary.c
+++ b/src/RobinBoundary.c
@@ -163,13 +163,9 @@ int BndRobin(const cGH *GH, int num_vars, int *vars, int *faces, int *widths,
     /* Determine boundary width on all faces */
     /* allocate memory for buffer */
     gdim = CCTK_GroupDimI(gi);
-    if (!width_alldirs)
+    if (gdim > max_gdim)
     {
-      width_alldirs = (int *) malloc(2*gdim*sizeof(int));
-      max_gdim = gdim;
-    } else if (gdim > max_gdim)
-    {
-      width_alldirs = realloc(width_alldirs, gdim*sizeof(int));
+      width_alldirs = realloc(width_alldirs, 2*gdim*sizeof(int));
       max_gdim = gdim;
     }