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

svn-noreply at zerowing.idsoftware.com svn-noreply at zerowing.idsoftware.com
Tue Jan 4 22:10:48 CST 2011


Author: rambetter
Date: Tue Jan  4 22:10:48 2011
New Revision: 409

Modified:
   GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c
   GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c
Log:
Tie up some tid bit loose ends related to q3map2 math improvements:

- Improved comments on SnapWeldVectorAccu() and FixWindingAccu().
In particular, FixWindingAccu() has a new "protocol".  The return value
now has a different meaning.  qtrue means the winding was altered, qfalse
means it wasn't.  The caller should check for degenerate windings (2 points
or less in winding).  Also, input winding to FixWindingAccu() cannot be NULL,
otherwise error is raised.  This is just cleaner in my opinion.

- In code that calls FixWindingAccu(), extra logic for handling NULL (which
now isn't allowed) and degenerate output winding (2 points or less).

- In ChopWindingInPlaceAccu() in polylib.c, factoring code a little bit to
reduce verbosity.


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=409&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c&r1=408&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/common/polylib.c&r2=409
==============================================================================

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=409&p1=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c&r1=408&p2=GtkRadiant/branches/Rambetter-math-fix-experiments/tools/quake3/q3map2/brush.c&r2=409
==============================================================================



More information about the Gtkradiant mailing list