Re: [Chromium-dev] new tile specification mode

[Brian P. <br...@tu...>]

Karl Rasche wrote:
> To make software projector alignment easier, I've added a new mode of
> handling tiles.
> 
> Instead of defining tiles in mural space, the tiles are separated into
> groups of "displays" (ie a projector). The tiles are specified relative to
> the origin of the display, *not* the origin of the mural.
> 
> Then, each display is given a 3x3 transform to warp it from
> (-1,-1)->(1,1) into mural space (actually, you have to pass it the
> inverse too, as I'm lazy and its much easier to grab when computing the
> corrective projection than to bloat up chromium with inversion code).
> 
> Finally, the view area is adjusted so it fits within the mural.


I'm working on applying your patches to the patches-1-branch branch in CVS.

The patches don't seem to follow a consistant format.  For example, the
tilesortspu_bucket.c file starts with:

*** /users/rkarl/chromium/cr/spu/tilesort/tilesortspu_bucket.c  Mon Jul 15 
14:40:44 2002
--- /users/rkarl/chromium/cr_align_topatch/spu/tilesort/tilesortspu_bucket.c 
Fri Aug  2 16:24:25 2002

but tilesortspu.h starts with:

*** /tmp/tilesortspu.h.cvs      Mon Aug  5 09:35:00 2002
--- /tmp/tilesortspu.h.new      Mon Aug  5 09:37:55 2002

and tilesortspu_config.c has no file information at all.

This means applying the patches takes manual intervention and it takes
longer than it should.

In the future, could you just make one big recursive diff patch?

If you want to diff against what's in CVS, you'd do this:
	cd cr
	cvs diff > patchfile

If you want to diff between two source trees you can use:
	diff -R cr.orig cr.new > patchfile.

Thanks.

-Brian