<div dir="ltr"><img src="http://t.sidekickopen06.com/e1t/o/5/f18dQhb0S7ks8dDMPbW2n0x6l2B9gXrN7sKj6v4dZ2TMfD98PRrZF6W8rBqX01pctGFVXshwp1k1H6H0?si=4870762816077824&pi=4942c1b3-a361-498b-c6e9-3ec3d0c073d0" style="display:none!important" height="1" width="1"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 20, 2016 at 12:00 AM, Scott Kitterman <span dir="ltr"><<a href="mailto:sklist@kitterman.com" target="_blank">sklist@kitterman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div class=""><div class="h5"><br>
</div></div>It didn't compile for me.  Please see the attached patch.</blockquote><div><br></div><div>Supremely tiny nit here. While looking at Scott's patch for opendmarc.c that fixed the missing quote mark in the DMARC_POLICY_SPF_ORIGIN_HELO, I noted that the two modes addressed in there were either:</div><div><br></div><div>mode = "helo:"</div><div><br></div><div>and</div><div><br></div><div>mode = "mailfrom"</div><div><br></div><div>The helo has a colon, the mailfrom doesn't (that wasn't introduced by Scott's patch, he was fixing the quote).</div><div><br></div><div>Apparently it works as-is, but is there a reason why the colon use isn't consistent?</div><div><br></div><div>SteveJ</div></div></div></div>