<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 7, 2017 at 5:50 PM, Cillian O'Donnell <span dir="ltr"><<a href="mailto:cpodonnell8@gmail.com" target="_blank">cpodonnell8@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>IGNORE THIS ONE. <br><br></div>It's a duplicate, that still has warnings. I couldn't figure out how to get rid of it. I'm new to git.<br></div>
<br></blockquote><div><br></div><div>This might be one where PRItime_t needs to be defined in rtems/inttypes.h and that</div><div>used. I think that's the type returned.</div><div><br></div><div>Also you should be working on a git branch and be able to do something like</div><div>"git rebase -i master" and follow the instructions to delete a single patch.</div><div><br></div><div>Before sending again:</div><div><br></div><div>+ Check all against my few comments</div><div>+ fix the short message so it is something like</div><div><span style="font-size:12.8px">"rtems/src/</span><wbr style="font-size:12.8px"><span style="font-size:12.8px">ratemonreportstatistics.c: Use inttypes.h constants to remove N warnings"</span><br></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">That type of message makes it easier to spot what changed in the git web view.</span></div><div><span style="font-size:12.8px">Use a it of judgement to shorten the filename. Often "BSP/XXX/YYY.c: message"</span></div><div><span style="font-size:12.8px">is enough to point someone to the file.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">The patches generally look good. Mastering printf() is a challenge. It is complicated</span></div><div><span style="font-size:12.8px">and doing the format strings so they are portably warning free is tricky.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Heading home now. I should already be there. :)</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">--joel</span></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">______________________________<wbr>_________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br></div></div>