Peter Stuge wrote: > > @@ -671,9 +674,9 @@ int abis_nm_rcvmsg(struct msgb *msg) > > default: > > LOGP(DNM, LOGL_ERROR, "unknown ABIS OML message discriminator > > 0x%x\n", > > oh->mdisc); > > - return -EINVAL; > > + rc = -EINVAL; > > } > > It looks like this is at the end of a switch () statement, so maybe > it is a good idea to include a goto statement for clarity, even if > that is not absolutely neccessary given how the code looks right now.
A break; would also be fine I think. Anything to make the flow explicit. //Peter
