On 06/28/2011 03:26 PM, Jim Ramsay wrote:
>
> + // a "new" session means to login a multiple of the
> + // currently-detected session.
We do not use c++ style comments.
> + if (op == OP_NEW) {
> + op = OP_NOOP;
> + do_login = 1;
> + rec->session.multiple = 1;
> + }
> +
> /* drop down to node ops */
> rc = exec_node_op(op, do_login, do_logout, do_show,
> do_rescan, do_stats, info_level,
> @@ -2162,6 +2170,11 @@ free_info:
> free(info);
> goto out;
> } else {
> + if (op == OP_NEW) {
> + log_error("session mode: Operation 'new' only
> allowed with specific session IDs");
We want to try and keep it to 80 cols.
Something went buggy with the formatting of this patch too. For some
reason where ever there is a "=" I see "=3D".
And the lines seem to be getting broken up. The line above got a newline
added to it so it looks like 2 lines instead of one.
The other patches did not look like this.
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To post to this group, send email to open-iscsi@googlegroups.com.
To unsubscribe from this group, send email to
open-iscsi+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/open-iscsi?hl=en.
