[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070114112412.GA32719@rhun.haifa.ibm.com>
Date: Sun, 14 Jan 2007 13:24:12 +0200
From: Muli Ben-Yehuda <muli@...ibm.com>
To: Jan Engelhardt <jengelh@...ux01.gwdg.de>
Cc: Jeremy Fitzhardinge <jeremy@...p.org>,
Andrew Morton <akpm@...l.org>, linux-kernel@...r.kernel.org,
virtualization@...ts.osdl.org, xen-devel@...ts.xensource.com,
Ian Pratt <ian.pratt@...source.com>,
Christian Limpach <Christian.Limpach@...cam.ac.uk>,
Chris Wright <chrisw@...s-sol.org>
Subject: Re: [patch 20/20] XEN-paravirt: Add Xen virtual block device driver.
On Sun, Jan 14, 2007 at 12:05:13PM +0100, Jan Engelhardt wrote:
> >+ (void)xenbus_switch_state(info->xbdev, XenbusStateConnected);
>
> Cast remove, if xenbus_switch_state does not have __must_check.
> Also elsewhere.
Hmm, why? this way you know that the programmer isn't checking the
return value *on purpose*. More information is good.
Cheers,
Muli
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists