svn commit: r185356 - head/sys/dev/ixgbe

Alexander Kabaev kabaev at gmail.com
Wed Nov 26 18:58:22 PST 2008


On Thu, 27 Nov 2008 02:19:44 +0000 (UTC)
Jack F Vogel <jfv at FreeBSD.org> wrote:

> Author: jfv
> Date: Thu Nov 27 02:19:44 2008
> New Revision: 185356
> URL: http://svn.freebsd.org/changeset/base/185356
> 
> Log:
>   Small nit I just noticed, a pre-decrement should be post.
> 
> Modified:
>   head/sys/dev/ixgbe/ixgbe.c
> 
> Modified: head/sys/dev/ixgbe/ixgbe.c
> ==============================================================================
> --- head/sys/dev/ixgbe/ixgbe.c	Thu Nov 27 02:18:43 2008
> (r185355) +++ head/sys/dev/ixgbe/ixgbe.c	Thu Nov 27 02:19:44
> 2008	(r185356) @@ -3244,7 +3244,7 @@ fail:
>  	 * the rings that completed, the failing case will have
>  	 * cleaned up for itself. 'j' failed, so its the terminus.
>  	 */
> -	for (int i = 0; i < j; ++i) {
> +	for (int i = 0; i < j; i++) {
>  		rxr = &adapter->rx_rings[i];
>  		for (int n = 0; n < adapter->num_rx_desc; n++) {
>  			struct ixgbe_rx_buf *rxbuf;

Is C99 construct here intentional? If so, when did we agree on using
only C99 compilers on our code base?
 
-- 
Alexander Kabaev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-head/attachments/20081127/c48d3737/signature.pgp


More information about the svn-src-head mailing list