Patch from Bart Van Assche <bart.vanassche@gmail.com>:

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 <bart.vanassche@gmail.com>


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@415 d57e44dd-8a1f-0410-8b47-8ef2f437770f
This commit is contained in:
Vladislav Bolkhovitin
2008-06-13 17:31:29 +00:00
parent 8aa85fae74
commit f7f8750f73

View File

@@ -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;