[ofw] [Patch][IPoIB_NDIS6_CM] Spacing, code style and tabulation patch

Smith, Stan stan.smith at intel.com
Tue Sep 21 10:54:15 PDT 2010


Please see below.

________________________________
From: ofw-bounces at lists.openfabrics.org [mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of Alex Naslednikov
Sent: Tuesday, September 21, 2010 5:30 AM
To: ofw at lists.openfabrics.org
Subject: [ofw] [Patch][IPoIB_NDIS6_CM] Spacing, code style and tabulation patch

This patch replaces spaces by tabs and add brackets to "if" case
Index: B:/users/xalex/WoF-trunk/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp
===================================================================
--- B:/users/xalex/WoF-trunk/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp        (revision 2934)
+++ B:/users/xalex/WoF-trunk/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp     (working copy)
@@ -1980,7 +1980,8 @@
 __recv_cb_internal(
                IN                           const     ib_cq_handle_t                                                                h_cq,
                IN                                                           void                                                                                       *cq_context,
-    IN              uint32_t*                    p_recv_cnt );
+             IN                                                           uint32_t                                                                               *p_recv_cnt
+             );


 static void
@@ -5797,8 +5798,9 @@
                cl_obj_unlock( &p_port->obj );

                if (p_port->send_mgr.pending_list.count <= 0)
+             {
                                return;
-
+             };

Braces around a single statement are not needed; some coding standards disallow this.
Additionally the ';' on the trailing brace is not needed.
Otherwise, OK.

                p_item =  cl_qlist_remove_head( &p_port->send_mgr.pending_list );
                while (p_item != cl_qlist_end(&p_port->send_mgr.pending_list))
                {
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20100921/f54ce17b/attachment.html>


More information about the ofw mailing list