Re: [Xen-devel] [PATCH for-4.5 1/2] xen/vsprintf: Introduce %*ph extended format specifier for hex buffers

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 26.09.14 at 13:49, <tim@xxxxxxx> wrote:
> At 11:10 +0100 on 26 Sep (1411726206), Andrew Cooper wrote:
>> +        for ( i = 0; i < nr_bytes; ++i )
>> +        {
>> +            /* Each byte: 2 chars, 0-padded, base 16, no hex prefix. */
>> +            str = number(str, end, hex_buffer[i], 16, 2, -1, ZEROPAD);
>> +            if ( str < end )
> 
> (str < end && i + 1 < nr_bytes) to avoid a trailing space?

Not exactly, as the increment of str must then also be suppressed.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel