On Mon, Aug 27, 2012 at 7:01 PM, Michael S. Tsirkin <mst@xxxxxxxxxx> wrote:
> On Mon, Aug 27, 2012 at 06:56:38PM +0000, Blue Swirl wrote:
>> > +static uint32_t slow_bar_readb(void *opaque, target_phys_addr_t addr)
>> > +{
>> > + AssignedDevRegion *d = opaque;
>> > + uint8_t *in = d->u.r_virtbase + addr;
>>
>> Don't perform arithmetic with void pointers.
>
> Why not?
> We require gcc and it's a documented extension there.
We don't require GCC, Clang can be used for some targets already.
Though it supports this non-standard extension too.
It's a bad idea to introduce dependencies where it's not necessary.
In this case it's not much effort to add the identifier for the struct
and in fact the only benefit ever is that the lazy coder saves a few
key presses.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
