On 06/28/2010 11:47 AM, David Howells wrote:
Justin P. Mattock<justinmattock@xxxxxxxxx> wrote:
would just removing ret be good or will things go out of whack because of no
ret
Don't you then get warnings for not capturing the result?
David
not sure if it would give warnings for not capturing the result.
then doing this would be the way to go.
if (ret) {
printk(KERN_WARNING "dev%d: Failed to create physical_node sysfs link:
%d\n");
}
but like you had said unnecessary braces, and only one statement
I'll look at this some more too see if I come up with anything.
Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
