Hi John, Your suggestion works for me and I would say is the easiest fix.
Jim -----Original Message----- From: John Scudder <[email protected]> Sent: Wednesday, May 24, 2023 1:26 PM To: James Guichard <[email protected]> Cc: The IESG <[email protected]>; [email protected]; lsr <[email protected]>; Acee Lindem (acee) <[email protected]>; Peter Psenak <[email protected]>; Ketan Talaulikar <[email protected]>; Lizhenbin <[email protected]> Subject: Re: Jim Guichard's No Objection on draft-ietf-lsr-ospfv3-srv6-extensions-11: (with COMMENT) I have an elaboration on one of Jim’s points: > On May 24, 2023, at 12:23 PM, Jim Guichard via Datatracker <[email protected]> > wrote: > > - Section 7.1 SRv6 Locator TLV: > > - The text 'Locator continued..' in Figure 5 might be confusing as perhaps > it is just me but when I initially read it, I thought that multiple > Locators could be carried in the TLV. This is not the case of > course. It would be easier on the eyes if the entire 'Locator' field of > Figure 5 were just a single block of 128-bits. Same comment for Figures > 6, 7, and 8. I guess an alternative strategy to the perfectly good one Jim proposes, would be to clean up the use of the ellipsis (…) and replace the final “continued” with “concluded” as in: OLD: +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Locator (128 bits) ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Locator continued ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Locator continued ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Locator continued ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ NEW: +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Locator (128 bits) ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | ... Locator continued ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | ... Locator continued ... | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | ... Locator concluded | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ Thanks, Jim, for catching this. —John _______________________________________________ Lsr mailing list [email protected] https://www.ietf.org/mailman/listinfo/lsr
