On 09/07/2013 09:40 PM, Michio Honda wrote:
Hi,
Sorry for that I didn't respond to that warning.
You are right, laddr == NULL && addrcnt == 1 is the indicator of the function called by
asconf_mgmt().
Since your patch is actually redundant, I would suggest putting comment on the
line of "if ((laddr == NULL) && (addrcnt == 1)) {", and/or on the checking in your patch.
I think as is is just fine. One can read through the Git log and then see
the rationale behind a change (which is in most cases even more worth than
a comment). If this function should ever be called from somewhere else for
whatever reason, then this comment would already be obsolete.
--
To unsubscribe from this list: send the line "unsubscribe linux-sctp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
