[PATCH] Simplify XineramifyXv.

Adam Jackson ajax at nwnk.net
Tue Apr 27 08:56:52 PDT 2010


On Mon, 2010-04-26 at 15:37 -0700, Jamey Sharp wrote:
> The structure of the adaptor-matching algorithm was obscured by trying
> to use "continue" and "break" as the only control-flow primitives. It's
> a lot more clear if you add "return" to that set.
> 
> In some ways this patch is easier to review with `git diff -w`.

I'm not sure I like what the algorithm actually comes up with, but I
certainly like your version of it better.

Reviewed-by: Adam Jackson <ajax at redhat.com>

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100427/b168b146/attachment.pgp>


More information about the xorg-devel mailing list