diff options
| author | Poul-Henning Kamp <phk@FreeBSD.org> | 2003-05-31 16:36:42 +0000 |
|---|---|---|
| committer | Poul-Henning Kamp <phk@FreeBSD.org> | 2003-05-31 16:36:42 +0000 |
| commit | 7021f3f539f6d715d44fecb739bef366bebde31c (patch) | |
| tree | a832cd95eee0962ff4e48feab26926fb0084f921 /sys | |
| parent | 239e385f11bbefc165e467356b8aeecf57358407 (diff) | |
| download | src-7021f3f539f6d715d44fecb739bef366bebde31c.tar.gz src-7021f3f539f6d715d44fecb739bef366bebde31c.zip | |
Don't use return(foo(...)); in a function returning void.
Found by: FlexeLint
Notes
Notes:
svn path=/head/; revision=115454
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/cam/scsi/scsi_all.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/sys/cam/scsi/scsi_all.c b/sys/cam/scsi/scsi_all.c index fe581af5cda3..72601dae35be 100644 --- a/sys/cam/scsi/scsi_all.c +++ b/sys/cam/scsi/scsi_all.c @@ -2453,10 +2453,12 @@ scsi_mode_sense(struct ccb_scsiio *csio, u_int32_t retries, u_int8_t page, u_int8_t *param_buf, u_int32_t param_len, u_int8_t sense_len, u_int32_t timeout) { - return(scsi_mode_sense_len(csio, retries, cbfcnp, tag_action, dbd, - page_code, page, param_buf, param_len, 0, - sense_len, timeout)); + + scsi_mode_sense_len(csio, retries, cbfcnp, tag_action, dbd, + page_code, page, param_buf, param_len, 0, + sense_len, timeout); } + void scsi_mode_sense_len(struct ccb_scsiio *csio, u_int32_t retries, void (*cbfcnp)(struct cam_periph *, union ccb *), @@ -2518,9 +2520,9 @@ scsi_mode_select(struct ccb_scsiio *csio, u_int32_t retries, u_int8_t *param_buf, u_int32_t param_len, u_int8_t sense_len, u_int32_t timeout) { - return(scsi_mode_select_len(csio, retries, cbfcnp, tag_action, - scsi_page_fmt, save_pages, param_buf, - param_len, 0, sense_len, timeout)); + scsi_mode_select_len(csio, retries, cbfcnp, tag_action, + scsi_page_fmt, save_pages, param_buf, + param_len, 0, sense_len, timeout); } void |