[Gtkradiant] r394 - in GtkRadiant/branches/Rambetter-math-fix-experiments: libs libs/mathlib tools/quake3/common tools/quake3/q3map2

svn-noreply at zerowing.idsoftware.com svn-noreply at zerowing.idsoftware.com
Fri Dec 31 01:05:26 CST 2010


Author: rambetter
Date: Fri Dec 31 01:05:26 2010
New Revision: 394

Modified:
   GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib.h
   GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib/mathlib.c
   GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c
   GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.h
   GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c
Log:
Making more progress on	the EXPERIMENTAL_HIGH_PRECISION_MATH_Q3MAP2_FIXES code.

- Renaming CopyWindingAccuToNormal() to	CopyWindingAccuToRegular() in order to
avoid confusing	"normal" (as in	"regular") with	"normal" (as in	the normal
to the plane).

- In mathlib.h/mathlib.c, breaking out all the *Accu() #define functions
into actual functions in mathlib.c.  I think this is much safer	for ensuring
that the correct types are passed in.  Otherwise math might take place
in the 32 bit world without any	warning.

- Adding VectorCopyAccuToRegular() and VectorCopyRegularToAccu() to mathlib.

- Improving comments on	VEC_SMALLEST_EPSILON_AROUND_ONE	and
VEC_ACCU_SMALLEST_EPSILON_AROUND_ONE.

- Had to "adjust" some code in polylib.c due to	the change in functions
in mathlib (moving from	#define	to actual functions).

In case	you're curious as to the correctness of	this code at this point:
I was able to compile Icy Fantasy -bsp stage in	the same amount	of time	that
it took	in the original	q3map2 (23 seconds to be exact), and there are no
more disappearing triangles.  I'm almost positive the regression tests are
fixed at this point, but there is no hurry in testing them because I still
have to	comb through quite a bit of code to ensure that	things are being
done correctly with respect to math accuracy.


Modified: GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib.h
Url: http://zerowing.idsoftware.com/viewcvs/radiant/GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib.h?view=diff&rev=394&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib.h&r1=393&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib.h&r2=394
==============================================================================

Modified: GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib/mathlib.c
Url: http://zerowing.idsoftware.com/viewcvs/radiant/GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib/mathlib.c?view=diff&rev=394&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib/mathlib.c&r1=393&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/libs/mathlib/mathlib.c&r2=394
==============================================================================

Modified: GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c
Url: http://zerowing.idsoftware.com/viewcvs/radiant/GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c?view=diff&rev=394&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c&r1=393&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c&r2=394
==============================================================================

Modified: GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.h
Url: http://zerowing.idsoftware.com/viewcvs/radiant/GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.h?view=diff&rev=394&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.h&r1=393&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.h&r2=394
==============================================================================

Modified: GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c
Url: http://zerowing.idsoftware.com/viewcvs/radiant/GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c?view=diff&rev=394&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c&r1=393&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c&r2=394
==============================================================================



More information about the Gtkradiant mailing list