<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 7, 2017 at 9:58 AM, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</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"><span class="gmail-">On Fri, Apr 7, 2017 at 10:05 AM, Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
><br>
><br>
> On Fri, Apr 7, 2017 at 4:15 AM, Cillian O'Donnell <<a href="mailto:cpodonnell8@gmail.com">cpodonnell8@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Great! That looks like a cleaner, more standardized solution. I was just<br>
>> hacking away until the warnings stopped. So can I go ahead and #include<br>
>> <inttypes.h> or should I wait until <rtems/inttypes.h> is defined before<br>
>> doing any more? Does this work for printk() too?<br>
><br>
><br>
> First the easy answer. printk() supports a subset of printf() formats. So<br>
> as long as it is a supported printf format, then yes. But I just looked for<br>
> printk() documentation and I don't see any. The code is in<br>
> cpukit/libcsupport/src/<wbr>vprintk.c for a double check. It is supposed to<br>
> support everything required by any test.<br>
><br>
> If a file only uses C99 standard PRI constants, then use <inttypes.h><br>
><br>
> If a file needs the extra ones being collected into <rtems/inttypes.h>,<br>
> then it should include that.<br>
><br>
> No need to introduce an unneeded dependency.<br>
><br>
> Funny, how this simple set of warnings has rippled. It also makes me<br>
> want to file a ticket for printk() documentation </span></blockquote><div><br></div><div>#2985</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"><span class="gmail-">and adding code style<br>
> rules about <inttypes.h>.<br>
><br>
</span>Please do.<br></blockquote><div><br></div><div>Done. <a href="https://devel.rtems.org/wiki/Developer/Coding/Conventions">https://devel.rtems.org/wiki/Developer/Coding/Conventions</a></div><div><br></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">
<br>
Also we should consider converting our code style into a doc instead<br>
of wiki page.<br></blockquote><div><br></div><div>Beyond consider. I think it must happen. I think this is necessary artifact</div><div>for qualification reviews.</div><div><br></div><div><a href="https://devel.rtems.org/ticket/2986">https://devel.rtems.org/ticket/2986</a><br></div><div><br></div><div>Please review my tickets and coding style change. :)</div><div><br></div><div>--joel</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
> --joel<br>
><br>
><br>
>><br>
>><br>
>> Thanks Joel.<br>
>><br>
>> ______________________________<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>
><br>
><br>
><br>
> ______________________________<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>
</div></div></blockquote></div><br></div></div>