<div dir="ltr"><div class="gmail_quote">On Mon, Apr 11, 2016 at 5:35 AM, Trevor Perrin <span dir="ltr"><<a href="mailto:trevp@trevp.net" target="_blank">trevp@trevp.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid"><span>On Sat, Apr 9, 2016 at 5:25 PM, Rhys Weatherley<br>
<<a href="mailto:rhys.weatherley@gmail.com">rhys.weatherley@gmail.com</a>> wrote:<br>
> A few minor comments as I've been working on Noise-C.<br>
<br>
</span>Hi Rhys,<br>
<br>
Nice start on the C lib, good name.<br>
<br>
Definitely send any feedback or unclear parts you find in the spec,<br>
feedback from implementers is gold at this point.<br></blockquote><div><br></div></div><div class="gmail_extra">I got a little confused with the description of handshake patterns vs message patterns in 5.3. Handshake State.  It got clearer on the third reading:</div><div class="gmail_extra"><br></div><div class="gmail_extra">- A handshake pattern (e.g. XX) maps to a sequence of message patterns, one for each handshake packet that is exchanged.</div><div class="gmail_extra">- Each message pattern is a sequence of tokens indicating what to do in the current handshake packet.</div><div class="gmail_extra"><br></div><div class="gmail_extra">The information is there but my eyes missed it on the first two readings.  The two different meanings of "pattern" got confused in my brain and I kept thinking I was looking at the other one.  Not sure how to clarify.  "Message template" instead of "message pattern" perhaps?  Patterns -> templates -> tokens.  Different words at each level.</div><div class="gmail_extra"><br></div><div class="gmail_extra">In section 9.2 in the description of Noise Pipes there is a change over from using "..." to "-----" as a separator in the pattern definitions.  Are they the same thing, or something different?  The diagrams aren't clear as to where the fallback happens.  The text indicates that the responder rejects the first IK message and then sends something else with a different packet type.  It may help if there were two diagrams for the whole sequence from the start, Noise_IK, and Noise_IK_to_XXfallback.  They will both have the same prefix, but with a comment saying "if failure happens here, switch to the other pattern and continue".</div><div class="gmail_extra"><br></div><div class="gmail_extra">Cheers,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Rhys.</div><div class="gmail_extra"><br></div></div>