On Tue, 2015-06-16 at 13:33 -0700, Jacob E Keller wrote:
> On Tue, 2015-06-16 at 17:19 +0300, Sergei Shtylyov wrote:
> > On 6/16/2015 4:47 PM, Jeff Kirsher wrote:
> >
> > > From: Jacob Keller <jacob.e.kel...@intel.com>
> >
> > > The value will never be negative, and we use the %i print format,
> > >
> > > use
> >
> > "%i" is the same as "%d", AFAIR. You need "%u" for the unsigned
> >
> > variables.
> >
> > > unsigned int for the loop counter. Issue found using cppcheck.
> >
> > > Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com>
> > > Tested-by: Krishneil Singh <krishneil.k.si...@intel.com>
> > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com>
> >
> > WBR, Sergei
> >
>
> Oops, you are right.
>
> Regards,
> Jake
The actual code does use %u, but I made a typo in the description. I am
sending an rc2.
Regards,
JakeN�����r��y����b�X��ǧv�^�){.n�+���z�^�)����w* jg��� �����ݢj/���z�ޖ��2�ޙ����&�)ߡ�a���� �G���h��j:+v���w��٥
