<div dir="ltr">We all love you Sean :-)<div><br></div><div style>I didn't mean to give you or Kyle a hard time at all. At least not seriously hard time :-)</div><div style><br></div><div style>-berk</div></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Thu, Mar 21, 2013 at 11:10 AM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Thu, 21 Mar 2013 09:54:40 -0400, Berk Geveci said:<br>
<br>
>I just took a look at the dashboard. Yikes! It looks like some of the<br>
>changes that were attempting to fix errors/warnings caused wide spread<br>
>warnings/errors :-)<br>
<br>
</div>Yup.  Sorry about that!!  But in my defence:<br>
  - the gerrit builds were totally green<br>
  - the AIX compiler is wrong<br>
  - it passed peer review<br>
  - many nightly buildbots seem to have stronger warning<br>
    settings vs the gerrit CDash@Home machines<br>
<br>
I didn't personally catch it because I was working on fixing instances of one particular warning, so it was the only warning I had turned on. :(<br>
<br>
To prevent this in the future:<br>
  - I'll be even more careful.<br>
  - I'll be making a CDash@Home machine with torturous settings<br>
  - we need a Windows gerrit buildbot!<br>
  - we should increase the warnings on the gerrit buildbots<br>
<div class="im"><br>
<br>
On Thu, 21 Mar 2013 10:24:51 -0400, Kyle Lutz said:<br>
<br>
>And Sean, in the future, could you mark me as a reviewer for these<br>
>types of patches.<br>
<br>
</div>Will do.  It's hard to know who to add as reviewers.  I usually use git blame, or add a random Dave. :)<br>
<br>
Cheers,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
</font></span></blockquote></div><br></div>