Re: [tcpm] draft-gont-tcpm-tcp-seq-validation

[Fernando Gont <[e-mail]>]

Hi, Rui,

Thanks so much for the review! Please find my comments inline...

On 09/11/2013 02:08 PM, Rui Paulo wrote:
> * Section 3.4 says " (Failed) Simultaneous Connection Termination" in
> the TCP time sequence ASCII

mm.. yes.. anything I'm missing?



> * Section 4.1.  TCP sequence number validation -> text is repeated

Yes. I guess the question here is what style we should use to do the update.

In other documents I authored in the past, I was asked to quote the text
to be replaced, and then the text it should be replaced with -- that's
why you see "duplicate" text.

Two possible improvements are:

* Indent the quoted text (although the indentation will mean that the
quoted text won't look the same as in RFC793, because the length of each
line would increase as a result of indentation, and hence they'd have to
be rewrapped), or,

* Try to quote only the specific sentences and tables to be replaced --
although this would make the text/spec harder to read.

Thoughts?



> * Section 7 doesn't look like an acknowledgement.

Yep. This will be fixed in the next rev.

Thanks!

Cheers,
-- 
Fernando Gont
e-mail: [e-mail] || [e-mail]
PGP Fingerprint: 7809 84F5 322E 45C7 F1C9 3945 96EE A9EF D076 FFF1