On Wed, 15 Oct 2008 14:39:25 +0200
Doron Shoham <dorons@xxxxxxxxxxxx> wrote:
> > Please fix the following checkpatch errors and resend an updated
> > patch:
> >
> > fujita@arbre:~/git/tgt$ ./scripts/checkpatch.pl ~/Mail/iscsi/stgt/3410
> > WARNING: multiple assignments should be avoided
> > #113: FILE: usr/tgtadm.c:390:
> > + total = cid = hostno = sid = lun = rc = 0;
> >
> > ERROR: do not use assignment in if condition
> > #170: FILE: usr/tgtadm.c:514:
> > + if ((rc = verify_mode_params(argc, argv, "LmotT"))) {
> >
> > WARNING: line over 80 characters
> > #176: FILE: usr/tgtadm.c:520:
> > + eprintf("creating a new target needs the name\n");
> >
>
> do we follow the above coding style of 80 characters per line?
Yeah, we do in most generally though there are some exceptions. Leave
parts in which you think that breaking 80 character rule is better.
--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
