[PATCH 0/5] dix: Remove more backing store leftovers

Daniel Martin consume.noise at gmail.com
Fri Sep 21 00:45:29 PDT 2012


On Thu, Sep 20, 2012 at 06:27:43PM -0400, Adam Jackson wrote:
> On 8/25/12 7:43 PM, Daniel Martin wrote:
> >These patches remove some more backing store leftovers. Basicly dead
> >snippets since Nov 2010 (commit: c4c4676).
> 
> For the series:
> 
> Reviewed-by: Adam Jackson <ajax at redhat.com>
> 
> The necessary driver changes are trivial, I'll happily run through
> them once this is merged.

I've started that too. But, didn't found the time prepare the commits. For
that round house kick coccinelle is very handy. Here's the semantic patch
I've tried:

-----8<---remove_mi_backing_store.cocci---8<-----
@@
@@

(
- #include <mibstore.h>
|
- #include "mibstore.h"
)

@@
expression E;
@@

- miInitializeBackingStore(E);

----->8---remove_mi_backing_store.cocci--->8-----

It even removes the comment prior to the line being removed. Usually
that are comments like "All drivers implementing backing store need
this".

You might be faster with doing this locally than I'm with preparing
commits, send them to the list and you import them later.


greetings,
Daniel Martin


More information about the xorg-devel mailing list