Confusing or confused code

Michael Thayer Michael.Thayer at Sun.COM
Thu Mar 11 03:59:29 PST 2010


I suspect that this code:

until line 1313 is supposed to be freeing the old list at
scrp->clockRanges, duplicating the user supplied list and storing it
there.  If that is right, then it looks to me though like it will not
work unless scrp->clockRanges is initially NULL and the user only
supplies a list with a single element.  Did I miss something important


Sun Microsystems GmbH        Michael Thayer
Werkstrasse 24               VirtualBox engineer
71384 Weinstadt, Germany     mailto:michael.thayer at

Sitz der Gesellschaft:
Sun Microsystems GmbH, Sonnenallee 1, 85551 Kirchheim-Heimstetten
Amtsgericht Muenchen: HRB 161028
Geschaeftsfuehrer: Thomas Schroeder, Wolfgang Engels
Vorsitzender des Aufsichtsrates: Martin Haering

More information about the xorg-devel mailing list