On Tue, Jul 3, 2012 at 12:41 AM, Keith Packard <kei...@keithp.com> wrote:
>
> Dave Airlie <airl...@gmail.com> writes:
>
>> +void
>> +AttachOutputGPU(ScreenPtr pScreen, ScreenPtr new)
>> +{
>> + assert(new->isGPU);
>> + xorg_list_add(&new->output_head, &pScreen->output_slave_list);
>> + new->current_master = pScreen;
>> +}
>
> I'd love an assert that new isn't already on a list somehow?
I'm not really sure you can tell that, since list members don't have
to make sense
by default,
I could add
assert(!new->current_master);
which should be true.
Dave.
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
