diff options
| author | Michael Brown | 2015-07-27 16:15:12 +0200 |
|---|---|---|
| committer | Michael Brown | 2015-07-27 16:21:17 +0200 |
| commit | 99d351605a965eb18f86aa8c68cb9aab9d4b5895 (patch) | |
| tree | d22b25bcea72f8a1622de94b3d03d220f1d330c3 /src/core | |
| parent | [downloader] Provide direct access to the underlying data transfer buffer (diff) | |
| download | ipxe-99d351605a965eb18f86aa8c68cb9aab9d4b5895.tar.gz ipxe-99d351605a965eb18f86aa8c68cb9aab9d4b5895.tar.xz ipxe-99d351605a965eb18f86aa8c68cb9aab9d4b5895.zip | |
[build] Fix compiler warnings on some gcc versions
xfer_buffer() uses intf_get_dest_op() to obtain the destination
interface for xfer_deliver(), in order to check that this is the same
interface which provides xfer_buffer(). The return value from
intf_get_dest_op() (which contains the actual method implementing
xfer_deliver()) is not used.
On some gcc versions, this triggers a "value computed is not used"
warning, since the explicit type cast included within the
intf_get_dest_op() macro is treated as a "value computed".
Fix by explicitly casting the result of intf_get_dest_op() to void.
Reported-by: Matthew Helton <mwhelton@gmail.com>
Reported-by: James A. Peltier <jpeltier@sfu.ca>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
Diffstat (limited to 'src/core')
| -rw-r--r-- | src/core/xferbuf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/xferbuf.c b/src/core/xferbuf.c index afc72ae7..24011855 100644 --- a/src/core/xferbuf.c +++ b/src/core/xferbuf.c @@ -309,7 +309,7 @@ struct xfer_buffer * xfer_buffer ( struct interface *intf ) { /* Check that this operation is provided by the same interface * which handles xfer_deliver(). */ - intf_get_dest_op ( intf, xfer_deliver, &xfer_deliver_dest ); + ( void ) intf_get_dest_op ( intf, xfer_deliver, &xfer_deliver_dest ); if ( op && ( dest == xfer_deliver_dest ) ) { xferbuf = op ( object ); |