[PATCH 04/11] Changes inttype used and adds casts, fixes 6 format warnings

Gedare Bloom gedare at rtems.org
Fri Apr 7 23:50:08 UTC 2017


On Fri, Apr 7, 2017 at 7:05 PM, Joel Sherrill <joel at rtems.org> wrote:
>
>
> On Fri, Apr 7, 2017 at 5:50 PM, Cillian O'Donnell <cpodonnell8 at gmail.com>
> wrote:
>>
>> IGNORE THIS ONE.
>>
>> It's a duplicate, that still has warnings. I couldn't figure out how to
>> get rid of it. I'm new to git.
>>
>
> This might be one where PRItime_t needs to be defined in rtems/inttypes.h
> and that
> used. I think that's the type returned.
>
> Also you should be working on a git branch and be able to do something like
> "git rebase -i master" and follow the instructions to delete a single patch.
>
> Before sending again:
>
> + Check all against my few comments
> + fix the short message so it is something like
> "rtems/src/ratemonreportstatistics.c: Use inttypes.h constants to remove N
> warnings"
>
> That type of message makes it easier to spot what changed in the git web
> view.
> Use a it of judgement to shorten the filename. Often "BSP/XXX/YYY.c:
> message"
> is enough to point someone to the file.
>
Also, note you can use git format-patch -v2 to indicate the second
version of a patch. This is useful to help identify patches that have
been previously reviewed and commented on.

> The patches generally look good. Mastering printf() is a challenge. It is
> complicated
> and doing the format strings so they are portably warning free is tricky.
>
> Heading home now. I should already be there. :)
>
> --joel
>
>>
>> _______________________________________________
>> devel mailing list
>> devel at rtems.org
>> http://lists.rtems.org/mailman/listinfo/devel
>
>
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel



More information about the devel mailing list