Hi Luuk, Inline: On 1/11/23 10:45, Luuk Hendriks wrote:
Hi Paolo, all, Thanks for the update. I have a question about something that was actually already part of the previous version: Sec 3.2: In a non-indexed enterprise-specific TLV, the Length includes the PEN, which I guess helps in making it backward compatible. For indexed TLVs, the Length does not include the Index (Sec3 in bmp-tlv-13), but what about indexed enterprise-specific TLVs? Does the length include the length of the PEN, but not the Index?
Exactly!
If indeed the PEN is included and the Index is not, would it make sense to swap the Index and PEN fields? Feels more consistent to interpret/process to me, but perhaps I'm missing any downsides of such a reordering.
That makes a lot of sense, indeed. Will update this in the next revision of the document. Thank you!
Paolo _______________________________________________ GROW mailing list [email protected] https://www.ietf.org/mailman/listinfo/grow
