Re: [PATCH 4/9]: Resolve dependencies of features on choice of CCID

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



|  I don't quite understand why you're defining on and off when there is
|  built in stuff for the kernel for true/false. You then pass the
|  address of the on/off - not the value. Maybe you might like to explain
|  the rationale behind this as I'm probably missing something.
No you are not - thanks so much for pointing this out! These constants have
been a constant eyesore and I had wanted to get rid of them, but lacked an
idea. 

The pointer was used to have a sentinel in the table, since the caller can not
know how large the table is. But a look at RFC 4340, 6.4 showed that `0' is 
never a valid feature number, so that is used as an end-of-table condition now.

Thanks a lot.
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux