src/datawizard: fix indent and add default switch case

src/datawizard/filters.c

@@ -1040,11 +1040,10 @@ void _starpu_data_partition_access_submit(starpu_data_handle_t target, int write
 	_starpu_data_partition_access_look_up(target, NULL, write);
 }
 
-void
-starpu_filter_nparts_compute_chunk_size_and_offset(unsigned n, unsigned nparts,
-					     size_t elemsize, unsigned id,
-					     unsigned blocksize, unsigned *chunk_size,
-					     size_t *offset)
+void starpu_filter_nparts_compute_chunk_size_and_offset(unsigned n, unsigned nparts,
+							size_t elemsize, unsigned id,
+							unsigned blocksize, unsigned *chunk_size,
+							size_t *offset)
 {
 	*chunk_size = n/nparts;
 	unsigned remainder = n % nparts;

src/datawizard/interfaces/bcsr_filters.c

@@ -93,7 +93,7 @@ void starpu_bcsr_filter_canonical_block(void *father_interface, void *child_inte
 
 unsigned starpu_bcsr_filter_canonical_block_get_nchildren(STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f, starpu_data_handle_t handle)
 {
-  return (unsigned)starpu_bcsr_get_nnz(handle);
+	return (unsigned)starpu_bcsr_get_nnz(handle);
 }
 
 struct starpu_data_interface_ops *starpu_bcsr_filter_canonical_block_child_ops(STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f, STARPU_ATTRIBUTE_UNUSED unsigned child)

src/datawizard/interfaces/block_filters.c

@@ -19,7 +19,7 @@
 #include <datawizard/filters.h>
 
 static void _starpu_block_filter_block(int dim, void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-			       unsigned id, unsigned nparts, uintptr_t shadow_size)
+				       unsigned id, unsigned nparts, uintptr_t shadow_size)
 {
 	struct starpu_block_interface *block_father = (struct starpu_block_interface *) father_interface;
         struct starpu_block_interface *block_child = (struct starpu_block_interface *) child_interface;
@@ -60,6 +60,8 @@ static void _starpu_block_filter_block(int dim, void *father_interface, void *ch
 			nn = nz;
 			blocksize = block_father->ldz;
 			break;
+		default:
+			STARPU_ASSERT_MSG(0, "Unknown value for dim");
 	}
 
 	size_t elemsize = block_father->elemsize;

src/datawizard/interfaces/matrix_filters.c

@@ -52,8 +52,10 @@ static void _starpu_matrix_filter_block(int dim, void *father_interface, void *c
 			nn = ny;
 			blocksize = matrix_father->ld;
 			break;
+		default:
+			STARPU_ASSERT_MSG(0, "Unknown value for dim");
 	}
-	
+
 	size_t elemsize = matrix_father->elemsize;
 
 	STARPU_ASSERT_MSG(nchunks <= nn, "cannot split %u elements in %u parts", nn, nchunks);
@@ -80,6 +82,8 @@ static void _starpu_matrix_filter_block(int dim, void *father_interface, void *c
 			matrix_child->nx = nx;
 			matrix_child->ny = child_nn;
 			break;
+		default:
+			STARPU_ASSERT_MSG(0, "Unknown value for dim");
 	}
 
 	matrix_child->elemsize = elemsize;

src/datawizard/interfaces/tensor_filters.c

@@ -21,168 +21,172 @@
 static void _starpu_tensor_filter_block(int dim, void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
                    unsigned id, unsigned nparts, uintptr_t shadow_size)
 {
-    struct starpu_tensor_interface *tensor_father = (struct starpu_tensor_interface *) father_interface;
-    struct starpu_tensor_interface *tensor_child = (struct starpu_tensor_interface *) child_interface;
-
-    unsigned blocksize;
-    /* the element will be split, in case horizontal, it's nx, in case vertical, it's ny, in case depth, it's nz, in case time, it's nt*/
-    uint32_t nn;
-    uint32_t nx;
-    uint32_t ny;
-    uint32_t nz;
-    uint32_t nt;
-
-    switch(dim)
-    {
-        /* horizontal*/
-        case 1:
-            /* actual number of elements */
-            nx = tensor_father->nx - 2 * shadow_size;
-            ny = tensor_father->ny;
-            nz = tensor_father->nz;
-            nt = tensor_father->nt;
-            nn = nx;
-            blocksize = 1;
-            break;
-        /* vertical*/
-        case 2:
-            nx = tensor_father->nx;
-            /* actual number of elements */
-            ny = tensor_father->ny - 2 * shadow_size;
-            nz = tensor_father->nz;
-            nt = tensor_father->nt;
-            nn = ny;
-            blocksize = tensor_father->ldy;
-            break;
-        /* depth*/
-        case 3:
-            nx = tensor_father->nx;
-            ny = tensor_father->ny;
-            /* actual number of elements */
-            nz = tensor_father->nz - 2 * shadow_size;
-            nt = tensor_father->nt;
-            nn = nz;
-            blocksize = tensor_father->ldz;
-            break;
-        /* time*/
-        case 4:
-            nx = tensor_father->nx;
-            ny = tensor_father->ny;
-            nz = tensor_father->nz;
-            /* actual number of elements */
-            nt = tensor_father->nt - 2 * shadow_size;
-            nn = nt;
-            blocksize = tensor_father->ldt;
-            break;
-    }
-
-    size_t elemsize = tensor_father->elemsize;
-
-    STARPU_ASSERT_MSG(nparts <= nn, "cannot split %u elements in %u parts", nn, nparts);
-
-    uint32_t child_nn;
-    size_t offset;
-    starpu_filter_nparts_compute_chunk_size_and_offset(nn, nparts, elemsize, id, blocksize, &child_nn, &offset);
-
-    child_nn += 2 * shadow_size;
-
-    STARPU_ASSERT_MSG(tensor_father->id == STARPU_TENSOR_INTERFACE_ID, "%s can only be applied on a tensor data", __func__);
-    tensor_child->id = tensor_father->id;
-
-    switch(dim)
-    {
-        case 1:
-            tensor_child->nx = child_nn;
-            tensor_child->ny = ny;
-            tensor_child->nz = nz;
-            tensor_child->nt = nt;
-            break;
-        case 2:
-            tensor_child->nx = nx;
-            tensor_child->ny = child_nn;
-            tensor_child->nz = nz;
-            tensor_child->nt = nt;
-            break;
-        case 3:
-            tensor_child->nx = nx;
-            tensor_child->ny = ny;
-            tensor_child->nz = child_nn;
-            tensor_child->nt = nt;
-            break;
-        case 4:
-            tensor_child->nx = nx;
-            tensor_child->ny = ny;
-            tensor_child->nz = nz;
-            tensor_child->nt = child_nn;
-            break;
-    }
-
-    tensor_child->elemsize = elemsize;
-
-    if (tensor_father->dev_handle)
-    {
-        if (tensor_father->ptr)
-            tensor_child->ptr = tensor_father->ptr + offset;
-        tensor_child->ldy = tensor_father->ldy;
-        tensor_child->ldz = tensor_father->ldz;
-        tensor_child->ldt = tensor_father->ldt;
-        tensor_child->dev_handle = tensor_father->dev_handle;
-        tensor_child->offset = tensor_father->offset + offset;
-    }
+	struct starpu_tensor_interface *tensor_father = (struct starpu_tensor_interface *) father_interface;
+	struct starpu_tensor_interface *tensor_child = (struct starpu_tensor_interface *) child_interface;
+
+	unsigned blocksize;
+	/* the element will be split, in case horizontal, it's nx, in case vertical, it's ny, in case depth, it's nz, in case time, it's nt*/
+	uint32_t nn;
+	uint32_t nx;
+	uint32_t ny;
+	uint32_t nz;
+	uint32_t nt;
+
+	switch(dim)
+	{
+		/* horizontal*/
+		case 1:
+			/* actual number of elements */
+			nx = tensor_father->nx - 2 * shadow_size;
+			ny = tensor_father->ny;
+			nz = tensor_father->nz;
+			nt = tensor_father->nt;
+			nn = nx;
+			blocksize = 1;
+			break;
+			/* vertical*/
+	        case 2:
+			nx = tensor_father->nx;
+			/* actual number of elements */
+			ny = tensor_father->ny - 2 * shadow_size;
+			nz = tensor_father->nz;
+			nt = tensor_father->nt;
+			nn = ny;
+			blocksize = tensor_father->ldy;
+			break;
+		/* depth*/
+	        case 3:
+			nx = tensor_father->nx;
+			ny = tensor_father->ny;
+			/* actual number of elements */
+			nz = tensor_father->nz - 2 * shadow_size;
+			nt = tensor_father->nt;
+			nn = nz;
+			blocksize = tensor_father->ldz;
+			break;
+			/* time*/
+	        case 4:
+			nx = tensor_father->nx;
+			ny = tensor_father->ny;
+			nz = tensor_father->nz;
+			/* actual number of elements */
+			nt = tensor_father->nt - 2 * shadow_size;
+			nn = nt;
+			blocksize = tensor_father->ldt;
+			break;
+		default:
+			STARPU_ASSERT_MSG(0, "Unknown value for dim");
+	}
+
+	size_t elemsize = tensor_father->elemsize;
+
+	STARPU_ASSERT_MSG(nparts <= nn, "cannot split %u elements in %u parts", nn, nparts);
+
+	uint32_t child_nn;
+	size_t offset;
+	starpu_filter_nparts_compute_chunk_size_and_offset(nn, nparts, elemsize, id, blocksize, &child_nn, &offset);
+
+	child_nn += 2 * shadow_size;
+
+	STARPU_ASSERT_MSG(tensor_father->id == STARPU_TENSOR_INTERFACE_ID, "%s can only be applied on a tensor data", __func__);
+	tensor_child->id = tensor_father->id;
+
+	switch(dim)
+	{
+        	case 1:
+			tensor_child->nx = child_nn;
+			tensor_child->ny = ny;
+			tensor_child->nz = nz;
+			tensor_child->nt = nt;
+			break;
+        	case 2:
+			tensor_child->nx = nx;
+			tensor_child->ny = child_nn;
+			tensor_child->nz = nz;
+			tensor_child->nt = nt;
+			break;
+        	case 3:
+			tensor_child->nx = nx;
+			tensor_child->ny = ny;
+			tensor_child->nz = child_nn;
+			tensor_child->nt = nt;
+			break;
+        	case 4:
+			tensor_child->nx = nx;
+			tensor_child->ny = ny;
+			tensor_child->nz = nz;
+			tensor_child->nt = child_nn;
+			break;
+		default:
+			STARPU_ASSERT_MSG(0, "Unknown value for dim");
+	}
+
+	tensor_child->elemsize = elemsize;
+
+	if (tensor_father->dev_handle)
+	{
+		if (tensor_father->ptr)
+			tensor_child->ptr = tensor_father->ptr + offset;
+		tensor_child->ldy = tensor_father->ldy;
+		tensor_child->ldz = tensor_father->ldz;
+		tensor_child->ldt = tensor_father->ldt;
+		tensor_child->dev_handle = tensor_father->dev_handle;
+		tensor_child->offset = tensor_father->offset + offset;
+	}
 }
 
 void starpu_tensor_filter_block(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                   unsigned id, unsigned nparts)
+				unsigned id, unsigned nparts)
 {
-    _starpu_tensor_filter_block(1, father_interface, child_interface, f, id, nparts, 0);
+	_starpu_tensor_filter_block(1, father_interface, child_interface, f, id, nparts, 0);
 }
 
 void starpu_tensor_filter_block_shadow(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                      unsigned id, unsigned nparts)
+				       unsigned id, unsigned nparts)
 {
-    uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
+	uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
 
-    _starpu_tensor_filter_block(1, father_interface, child_interface, f, id, nparts, shadow_size);
+	_starpu_tensor_filter_block(1, father_interface, child_interface, f, id, nparts, shadow_size);
 }
 
 void starpu_tensor_filter_vertical_block(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
                     unsigned id, unsigned nparts)
 {
-    _starpu_tensor_filter_block(2, father_interface, child_interface, f, id, nparts, 0);
+	_starpu_tensor_filter_block(2, father_interface, child_interface, f, id, nparts, 0);
 }
 
 void starpu_tensor_filter_vertical_block_shadow(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                      unsigned id, unsigned nparts)
+						unsigned id, unsigned nparts)
 {
-    uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
+	uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
 
-    _starpu_tensor_filter_block(2, father_interface, child_interface, f, id, nparts, shadow_size);
+	_starpu_tensor_filter_block(2, father_interface, child_interface, f, id, nparts, shadow_size);
 }
 
 void starpu_tensor_filter_depth_block(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                    unsigned id, unsigned nparts)
+				      unsigned id, unsigned nparts)
 {
-    _starpu_tensor_filter_block(3, father_interface, child_interface, f, id, nparts, 0);
+	_starpu_tensor_filter_block(3, father_interface, child_interface, f, id, nparts, 0);
 }
 
 void starpu_tensor_filter_depth_block_shadow(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                      unsigned id, unsigned nparts)
+					     unsigned id, unsigned nparts)
 {
-    uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
+	uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
 
-    _starpu_tensor_filter_block(3, father_interface, child_interface, f, id, nparts, shadow_size);
+	_starpu_tensor_filter_block(3, father_interface, child_interface, f, id, nparts, shadow_size);
 }
 
 void starpu_tensor_filter_time_block(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                    unsigned id, unsigned nparts)
+				     unsigned id, unsigned nparts)
 {
-    _starpu_tensor_filter_block(4, father_interface, child_interface, f, id, nparts, 0);
+	_starpu_tensor_filter_block(4, father_interface, child_interface, f, id, nparts, 0);
 }
 
 void starpu_tensor_filter_time_block_shadow(void *father_interface, void *child_interface, STARPU_ATTRIBUTE_UNUSED struct starpu_data_filter *f,
-                      unsigned id, unsigned nparts)
+					    unsigned id, unsigned nparts)
 {
-    uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
+	uintptr_t shadow_size = (uintptr_t) f->filter_arg_ptr;
 
-    _starpu_tensor_filter_block(4, father_interface, child_interface, f, id, nparts, shadow_size);
+	_starpu_tensor_filter_block(4, father_interface, child_interface, f, id, nparts, shadow_size);
 }