[PATCH] mi: removed unused PointerNonInterestBox functions.

Oliver McFadden oliver.mcfadden at nokia.com
Thu May 13 07:24:06 PDT 2010


On Thu, 2010-05-13 at 13:23 +0200, ext Keith Packard wrote:
> On Thu, 13 May 2010 11:45:02 +0300, Oliver McFadden <oliver.mcfadden at nokia.com> wrote:
> 
> 
> > -    PointerNonInterestBoxProcPtr PointerNonInterestBox;
> > +    PointerNonInterestBoxProcPtr PointerNonInterestBox; /** unused */
> 
> I'd just go ahead and delete this line. Otherwise,

Wouldn't this break the ScreenRec ABI? I noticed this structure also has
a few other variables marked as unused. I assume for this exact reason?

I would rather get rid of the unused function pointers, too. Would I
need to bump some version defines somewhere? I am not sure how this
should be handled.

> 
> Reviewed-by: Keith Packard <keithp at keithp.com>




More information about the xorg-devel mailing list