Ulrich Drepper <drepper@redhat.com> writes:

> Andreas Jaeger wrote:
>
>>    for (cnt = 0; cnt < n_convert; ++cnt, inptr += 4)
>> -    *((uint32_t *) outptr)++ = bswap_32 (*(const uint32_t *) inptr);
>> +    *outptr32++ = bswap_32 (*(const uint32_t *) inptr);
>> +  outptr = (unsigned char *) outptr32;
>>  
>>    *inptrp = inptr;
>>    *outptrp = outptr;
>
> Why the second added line?  Just do it in the assignment to *outptrp.

Ok, will do...
Andreas
-- 
 Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
  SuSE Linux AG, Maxfeldstr. 5, 90409 Nürnberg, Germany
   GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126