<div dir="ltr"><img src="http://t.sidekickopen06.com/e1t/o/5/f18dQhb0S7ks8dDMPbW2n0x6l2B9gXrN7sKj6v4dZ2TMfD98PRrZF6W8rBqX01pctGFVXshwp1k1H6H0?si=4870762816077824&pi=4cf3b9d7-0a09-4bdc-d2bf-573f74dda0dc" style="display:none!important" height="1" width="1"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 20, 2016 at 4:01 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wednesday, July 20, 2016 03:28:19 PM Steve Jenkins wrote:<br>
> On Wed, Jul 20, 2016 at 12:00 AM, Scott Kitterman <<a href="mailto:sklist@kitterman.com">sklist@kitterman.com</a>><br>
><br>
> wrote:<br>
> > It didn't compile for me.  Please see the attached patch.<br>
><br>
> Supremely tiny nit here. While looking at Scott's patch for opendmarc.c<br>
> that fixed the missing quote mark in the DMARC_POLICY_SPF_ORIGIN_HELO, I<br>
> noted that the two modes addressed in there were either:<br>
><br>
> mode = "helo:"<br>
><br>
> and<br>
><br>
> mode = "mailfrom"<br>
><br>
> The helo has a colon, the mailfrom doesn't (that wasn't introduced by<br>
> Scott's patch, he was fixing the quote).<br>
><br>
> Apparently it works as-is, but is there a reason why the colon use isn't<br>
> consistent?<br>
<br>
</span>Good question.  I didn't look beyond making it build.</blockquote><div><br></div><div>Sometimes my OCD comes in handy. LOL.</div><div><br></div><div>SteveJ </div></div></div></div>