Fix wrong comments in struct ip_mreq_source

Fix wrong comments in struct ip_mreq_source

[Phil Blundell]

The member names in this struct are fairly self-explanatory, but were
contradicted by the comments.  This patch swaps the comments around so
that they match.

I verified that the order of the elements does match the corresponding
uapi header from the Linux kernel.

OK to commit?  (I wasn't quite sure whether this was obvious enough to
count as a "trivial fix".)

p.

2017-05-03  Phil Blundell  <pb@pbcl.net>

        * inet/netinet/in.h (struct ip_mreq_source): Correct misplaced
        comments.

diff --git a/inet/netinet/in.h b/inet/netinet/in.h
index 365bc6878e..e064188721 100644
--- a/inet/netinet/in.h
+++ b/inet/netinet/in.h
@@ -275,10 +275,10 @@ struct ip_mreq_source
     /* IP multicast address of group.  */
     struct in_addr imr_multiaddr;
 
-    /* IP address of source.  */
+    /* IP address of interface.  */
     struct in_addr imr_interface;
 
-    /* IP address of interface.  */
+    /* IP address of source.  */
     struct in_addr imr_sourceaddr;
   };
 #endif

Re: Fix wrong comments in struct ip_mreq_source

[Carlos O'Donell]

On 05/03/2017 03:42 PM, Phil Blundell wrote:
> The member names in this struct are fairly self-explanatory, but were
> contradicted by the comments.  This patch swaps the comments around so
> that they match.
> 
> I verified that the order of the elements does match the corresponding
> uapi header from the Linux kernel.
> 
> OK to commit?  (I wasn't quite sure whether this was obvious enough to
> count as a "trivial fix".)
> 
> p.
> 
> 2017-05-03  Phil Blundell  <pb@pbcl.net>
> 
>         * inet/netinet/in.h (struct ip_mreq_source): Correct misplaced
>         comments.

OK. This counts as a trivial fix. Even if you'd gotten it wrong the comment
change could just be another commit away from being right.

-- 
Cheers,
Carlos.