From f7f8750f739cc4cf56da608c1b18e34cc8ab5e6d Mon Sep 17 00:00:00 2001 From: Vladislav Bolkhovitin Date: Fri, 13 Jun 2008 17:31:29 +0000 Subject: [PATCH] Patch from Bart Van Assche : The patch below fixes the following class of checkpatch warnings: - WARNING: braces {} are not necessary for single statement blocks How this patch has been verified: - Checked that make -s clean && make -s scst iscsi-scst still works. - Verified that checkpatch does no longer complain about the placement of braces. The patch below only changes comments, so it should be safe to include it in the 1.0.0 release. Signed-off-by: Bart Van Assche git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@415 d57e44dd-8a1f-0410-8b47-8ef2f437770f --- iscsi-scst/kernel/iscsi.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/iscsi-scst/kernel/iscsi.c b/iscsi-scst/kernel/iscsi.c index cee8b27a2..ecd771bf2 100644 --- a/iscsi-scst/kernel/iscsi.c +++ b/iscsi-scst/kernel/iscsi.c @@ -668,7 +668,7 @@ static struct iscsi_cmnd *create_status_rsp(struct iscsi_cmnd *req, int status, sg = rsp->sg = scst_alloc(PAGE_SIZE, GFP_KERNEL|__GFP_NOFAIL, &rsp->sg_cnt); if (sg == NULL) { - /* ToDo */ + ;/* ToDo */; } rsp->own_sg = 1; sense = (struct iscsi_sense_data *)page_address(sg_page(&sg[0])); @@ -731,7 +731,7 @@ static void iscsi_cmnd_reject(struct iscsi_cmnd *req, int reason) sg = rsp->sg = scst_alloc(PAGE_SIZE, GFP_KERNEL|__GFP_NOFAIL, &rsp->sg_cnt); if (sg == NULL) { - /* ToDo */ + ;/* ToDo */; } rsp->own_sg = 1; addr = page_address(sg_page(&sg[0])); @@ -941,7 +941,7 @@ static void cmnd_prepare_get_rejected_cmd_data(struct iscsi_cmnd *cmnd) sg = cmnd->sg = scst_alloc(PAGE_SIZE, GFP_KERNEL|__GFP_NOFAIL, &cmnd->sg_cnt); if (sg == NULL) { - /* ToDo */ + ;/* ToDo */; } cmnd->own_sg = 1; cmnd->bufflen = PAGE_SIZE; @@ -1231,10 +1231,10 @@ static int noop_out_start(struct iscsi_cmnd *cmnd) cmnd->sg = sg = scst_alloc(size, GFP_KERNEL|__GFP_NOFAIL, &cmnd->sg_cnt); if (sg == NULL) { - /* ToDo */ + ;/* ToDo */; } if (cmnd->sg_cnt > ISCSI_CONN_IOV_MAX) { - /* ToDo */ + ;/* ToDo */; } cmnd->own_sg = 1; cmnd->bufflen = size;