It is easier to add debug prints for sdp state machine

On Thu, Jun 19, 2008 at 8:04 PM, Sean Hefty <[EMAIL PROTECTED]> wrote:

> >+static inline void sdp_set_state(struct sock *sk, int state)
> >+{
> >+      sk->sk_state = state;
> >+}
>
> Why have this abstraction?
>
> >-      sk->sk_state = TCP_CLOSE;
> >+      sdp_set_state(sk, TCP_CLOSE);
>
> IMO, the previous code is simpler to maintain.
>
> - Sean
>
> _______________________________________________
> general mailing list
> [email protected]
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
>
> To unsubscribe, please visit
> http://openib.org/mailman/listinfo/openib-general
>
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to