2011-05-24 Stephen White <senorblanco@chromium.org>
authorsenorblanco@chromium.org <senorblanco@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 May 2011 18:50:48 +0000 (18:50 +0000)
committersenorblanco@chromium.org <senorblanco@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 May 2011 18:50:48 +0000 (18:50 +0000)
        Reviewed by Kenneth Russell.

        Remove most style checks on Source/ThirdParty/glu, since it doesn't
        follow WebKit style.
        https://bugs.webkit.org/show_bug.cgi?id=61365

        * Scripts/webkitpy/style/checker.py:
2011-05-24  Stephen White  <senorblanco@chromium.org>

        Reviewed by Kenneth Russell.

        Unfork glu sources as much as possible.
        https://bugs.webkit.org/show_bug.cgi?id=61365

        Restore the glu sources closer to the original version on oss.sgi.com
        by removing the WebKit-specific include paths, restoring the include
        order, and renaming the glu* functions via #defines rather than in
        the source.

        * glu/README.webkit:
        Update the README to reflect the changes from trunk.
        * glu/glu.gyp:
        Add Source/Thirdparty/glu as an include path, so we can remove the
        hardcoded paths from the source files.
        * glu/gluos.h:
        Add #defines to rename the glu entry points to internal_*.
        * glu/internal_glu.h:
        * glu/libtess/dict.c:
        * glu/libtess/geom.c:
        * glu/libtess/geom.h:
        * glu/libtess/memalloc.c:
        * glu/libtess/mesh.c:
        * glu/libtess/mesh.h:
        * glu/libtess/normal.c:
        * glu/libtess/normal.h:
        * glu/libtess/priorityq-heap.c:
        * glu/libtess/priorityq-sort.h:
        * glu/libtess/priorityq.c:
        * glu/libtess/priorityq.h:
        * glu/libtess/render.c:
        * glu/libtess/render.h:
        * glu/libtess/sweep.c:
        * glu/libtess/sweep.h:
        * glu/libtess/tess.c:
        * glu/libtess/tess.h:
        * glu/libtess/tessmono.c:
        Remove WebKit-specific include paths, and internal_ prefixes from glu
        entry points.  Restore include order to original source order.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@87178 268f45cc-cd09-0410-ab3c-d52691b4dbfc

26 files changed:
Source/ThirdParty/ChangeLog
Source/ThirdParty/glu/README.webkit
Source/ThirdParty/glu/glu.gyp
Source/ThirdParty/glu/gluos.h
Source/ThirdParty/glu/internal_glu.h
Source/ThirdParty/glu/libtess/dict.c
Source/ThirdParty/glu/libtess/geom.c
Source/ThirdParty/glu/libtess/geom.h
Source/ThirdParty/glu/libtess/memalloc.c
Source/ThirdParty/glu/libtess/mesh.c
Source/ThirdParty/glu/libtess/mesh.h
Source/ThirdParty/glu/libtess/normal.c
Source/ThirdParty/glu/libtess/normal.h
Source/ThirdParty/glu/libtess/priorityq-heap.c
Source/ThirdParty/glu/libtess/priorityq-sort.h
Source/ThirdParty/glu/libtess/priorityq.c
Source/ThirdParty/glu/libtess/priorityq.h
Source/ThirdParty/glu/libtess/render.c
Source/ThirdParty/glu/libtess/render.h
Source/ThirdParty/glu/libtess/sweep.c
Source/ThirdParty/glu/libtess/sweep.h
Source/ThirdParty/glu/libtess/tess.c
Source/ThirdParty/glu/libtess/tess.h
Source/ThirdParty/glu/libtess/tessmono.c
Tools/ChangeLog
Tools/Scripts/webkitpy/style/checker.py

index b4ff0cc365a80e3b868eaa00f5c66b93088c3892..cd3f2564de71b9394609a2fe85ed8453219d3f1f 100644 (file)
@@ -1,3 +1,45 @@
+2011-05-24  Stephen White  <senorblanco@chromium.org>
+
+        Reviewed by Kenneth Russell.
+
+        Unfork glu sources as much as possible.
+        https://bugs.webkit.org/show_bug.cgi?id=61365
+
+        Restore the glu sources closer to the original version on oss.sgi.com
+        by removing the WebKit-specific include paths, restoring the include
+        order, and renaming the glu* functions via #defines rather than in
+        the source.
+
+        * glu/README.webkit:
+        Update the README to reflect the changes from trunk.
+        * glu/glu.gyp:
+        Add Source/Thirdparty/glu as an include path, so we can remove the
+        hardcoded paths from the source files.
+        * glu/gluos.h:
+        Add #defines to rename the glu entry points to internal_*.
+        * glu/internal_glu.h:
+        * glu/libtess/dict.c:
+        * glu/libtess/geom.c:
+        * glu/libtess/geom.h:
+        * glu/libtess/memalloc.c:
+        * glu/libtess/mesh.c:
+        * glu/libtess/mesh.h:
+        * glu/libtess/normal.c:
+        * glu/libtess/normal.h:
+        * glu/libtess/priorityq-heap.c:
+        * glu/libtess/priorityq-sort.h:
+        * glu/libtess/priorityq.c:
+        * glu/libtess/priorityq.h:
+        * glu/libtess/render.c:
+        * glu/libtess/render.h:
+        * glu/libtess/sweep.c:
+        * glu/libtess/sweep.h:
+        * glu/libtess/tess.c:
+        * glu/libtess/tess.h:
+        * glu/libtess/tessmono.c:
+        Remove WebKit-specific include paths, and internal_ prefixes from glu
+        entry points.  Restore include order to original source order.
+
 2011-05-19  Evan Martin  <evan@chromium.org>
 
         Reviewed by Tony Chang.
index 5941f4bc30ed8106feb01d5cecd2b7a83aee01a7..e7d4e21d13dd80edfad7c2305d2974c00e4a4891 100644 (file)
@@ -8,17 +8,13 @@ LICENSE.txt in this directory.
 The following changes were made in order to incorporate this code:
 
   - The addition of a simplified gluos.h to eliminate operating system
-    dependencies.
+    dependencies.  All entry points to the tesselator were prefixed
+    with internal_ via #defines, to avoid symbol collisions with any
+    host OS version of GLU.
 
   - The removal of inclusion of GL/glu.h and replacement with an
     include of internal_glu.h.
 
-  - Includes were modified to be scoped within ThirdParty/glu/ and
-    ThirdParty/glu/libtess/.
-
-  - The entry points to the tessellator were prefixed with internal_
-    to avoid symbol collisions with any host OS version of GLU.
-
   - In tess.c, the obsolete entry points gluBeginPolygon,
     gluNextContour and gluEndPolygon in tess.c were #if 0'd out.
     Default branches were added to the switch statements in GotoState.
index 51cbc840ec6529dfcb73de68f3c176fbfe8d430c..b1dac00ef19a339aa75d9af990d18ccbb1b8b19a 100644 (file)
@@ -3,7 +3,7 @@
     'target_name': 'libtess',
     'type': 'static_library',
     'include_dirs': [
-      '../..',
+      '.',
     ],
     'sources': [
       'gluos.h',
index 6380cb23771912265b371d8107d376a5d8384f3b..3efc8c7486287a08fb10b4762896903a08888f86 100644 (file)
@@ -42,6 +42,18 @@ typedef unsigned int GLenum;
 typedef float GLfloat;
 typedef void GLvoid;
 
+#define gluNewTess internal_gluNewTess
+#define gluDeleteTess internal_gluDeleteTess
+#define gluTessProperty internal_gluTessProperty
+#define gluGetTessProperty internal_gluGetTessProperty
+#define gluTessNormal internal_gluTessNormal
+#define gluTessCallback internal_gluTessCallback
+#define gluTessVertex internal_gluTessVertex
+#define gluTessBeginPolygon internal_gluTessBeginPolygon
+#define gluTessBeginContour internal_gluTessBeginContour
+#define gluTessEndContour internal_gluTessEndContour
+#define gluTessEndPolygon internal_gluTessEndPolygon
+
 #undef MIN
 #undef MAX
 
index 4c0e863e065e62ffd5990fe119c83a352a464052..02c9cc80835b8fb29e9cb75e0592af30bf6853fa 100644 (file)
@@ -36,7 +36,7 @@
 #ifndef INTERNAL_GLU_H_
 #define INTERNAL_GLU_H_
 
-#include "ThirdParty/glu/gluos.h"
+#include "gluos.h"
 
 /* Primitives */
 #define GL_LINE_LOOP                            0x0002
index bbae1f2605bc20cf4c15ab982d602cceaf5b84a6..e149fd549942534aae47fdb424e554a03bce14ee 100644 (file)
@@ -40,8 +40,8 @@
 */
 
 #include <stddef.h>
-#include "ThirdParty/glu/libtess/dict-list.h"
-#include "ThirdParty/glu/libtess/memalloc.h"
+#include "dict-list.h"
+#include "memalloc.h"
 
 /* really __gl_dictListNewDict */
 Dict *dictNewDict( void *frame,
index c404202311f1411bcf0a1fc7a7162c1cc9a96fc7..461d8b4a2015b002a7338f14bd69963272c217e2 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/geom.c#5 $
 */
 
+#include "gluos.h"
 #include <assert.h>
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/geom.h"
+#include "mesh.h"
+#include "geom.h"
 
 int __gl_vertLeq( GLUvertex *u, GLUvertex *v )
 {
index d6c8497c4c2f4c39104534b5b9414a3108c6459a..b4d0c6678224ee7aafd6d34fbcd255887e67f060 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __geom_h_
 #define __geom_h_
 
-#include "ThirdParty/glu/libtess/mesh.h"
+#include "mesh.h"
 
 #ifdef NO_BRANCH_CONDITIONS
 /* MIPS architecture has special instructions to evaluate boolean
index 530646f8df7b736745c15ff0a361c6a5c932a51d..bd5b4ac798ddfea82e6eb51b4394c6c9c043c37d 100644 (file)
@@ -39,8 +39,8 @@
 ** $Header: //depot/main/gfx/lib/glu/libtess/memalloc.c#5 $
 */
 
+#include "memalloc.h"
 #include "string.h"
-#include "ThirdParty/glu/libtess/memalloc.h"
 
 int __gl_memInit( size_t maxFast )
 {
index 096f898bff582a4b8774d64b23fbb9ca7d27c581..7305fab1d2fdf7f64e06af4c717241cbd1f52d8b 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/mesh.c#6 $
 */
 
-#include <assert.h>
+#include "gluos.h"
 #include <stddef.h>
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/memalloc.h"
-#include "ThirdParty/glu/libtess/mesh.h"
+#include <assert.h>
+#include "mesh.h"
+#include "memalloc.h"
 
 #define TRUE 1
 #define FALSE 0
index b83641184bc6ce25b361f80860278b882066e2da..a92de0e963aafb820d13cb2771b1866a3aa902df 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __mesh_h_
 #define __mesh_h_
 
-#include "ThirdParty/glu/internal_glu.h"
+#include <internal_glu.h>
 
 typedef struct GLUmesh GLUmesh; 
 
index bbe57fd773a510d900ef7f468f52b9bf5d132875..8fb22f1de7e275f61408407a62ec20ddbd4f18d5 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/normal.c#5 $
 */
 
-#include <assert.h>
+#include "gluos.h"
+#include "mesh.h"
+#include "tess.h"
+#include "normal.h"
 #include <math.h>
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/normal.h"
-#include "ThirdParty/glu/libtess/tess.h"
+#include <assert.h>
 
 #define TRUE 1
 #define FALSE 0
index f52bd0904e8b292afb9f6f64314b6498f22e8e31..5a687473b59447daebb8da6fb4a59e541b405355 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __normal_h_
 #define __normal_h_
 
-#include "ThirdParty/glu/libtess/tess.h"
+#include "tess.h"
 
 /* __gl_projectPolygon( tess ) determines the polygon normal
  * and project vertices onto the plane of the polygon.
index 43af72727abba1625568cee5d174ac772029e659..163bde6a8d84c4c11158bbb61a7bc750294aad5d 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/priorityq-heap.c#5 $
 */
 
+#include <stddef.h>
 #include <assert.h>
 #include <limits.h>
-#include <stddef.h>
-#include "ThirdParty/glu/libtess/memalloc.h"
-#include "ThirdParty/glu/libtess/priorityq-heap.h"
+#include "priorityq-heap.h"
+#include "memalloc.h"
 
 #define INIT_SIZE      32
 
index 2a5e71c16c639de6e5cfbd5e1cf7269c72baf003..12c2f0d4b2cb4f53f3c9779594fe15434c394e94 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __priorityq_sort_h_
 #define __priorityq_sort_h_
 
-#include "ThirdParty/glu/libtess/priorityq-heap.h"
+#include "priorityq-heap.h"
 
 #undef PQkey
 #undef PQhandle
index 93f4179ad7c8b171980cf01112a0e63491e583cc..4b9e1aeae034fd128e70faacd77479bbff2d1703 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/priorityq.c#5 $
 */
 
+#include "gluos.h"
+#include <stddef.h>
 #include <assert.h>
 #include <limits.h>            /* LONG_MAX */
-#include <stddef.h>
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/memalloc.h"
+#include "memalloc.h"
 
 /* Include all the code for the regular heap-based queue here. */
 
-#include "ThirdParty/glu/libtess/priorityq-heap.c"
+#include "priorityq-heap.c"
 
 /* Now redefine all the function names to map to their "Sort" versions. */
 
-#include "ThirdParty/glu/libtess/priorityq-sort.h"
+#include "priorityq-sort.h"
 
 /* really __gl_pqSortNewPriorityQ */
 PriorityQ *pqNewPriorityQ( int (*leq)(PQkey key1, PQkey key2) )
index 69af1886ff3d38b95f04c0c6480bf45f1a615f10..3ae6b0c2629c2a96044bffff4ac25ad322aab41c 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __priorityq_sort_h_
 #define __priorityq_sort_h_
 
-#include "ThirdParty/glu/libtess/priorityq-heap.h"
+#include "priorityq-heap.h"
 
 #undef PQkey
 #undef PQhandle
index e7534f55e4742f1c254a7498e8279e2a0b6b02d5..aff129842699ba20c70f1e6c82ccaf69a91057a7 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/render.c#5 $
 */
 
+#include "gluos.h"
 #include <assert.h>
 #include <stddef.h>
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/render.h"
-#include "ThirdParty/glu/libtess/tess.h"
+#include "mesh.h"
+#include "tess.h"
+#include "render.h"
 
 #define TRUE 1
 #define FALSE 0
index 5f3e06c5e717c709c6a48d485a5541efdfc55cce..598423ea011e3908f8ceffe37e0d7cb9ca9ee4eb 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __render_h_
 #define __render_h_
 
-#include "ThirdParty/glu/libtess/mesh.h"
+#include "mesh.h"
 
 /* __gl_renderMesh( tess, mesh ) takes a mesh and breaks it into triangle
  * fans, strips, and separate triangles.  A substantial effort is made
index 415f592c016fe7a45957fdb71eb6fdf82ec5a139..6fe8a478f54244a3a6dbd8b613653d2c7a0bec28 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/sweep.c#5 $
 */
 
+#include "gluos.h"
 #include <assert.h>
+#include <stddef.h>
 #include <setjmp.h>            /* longjmp */
 #include <limits.h>            /* LONG_MAX */
-#include <stddef.h>
 
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/dict.h"
-#include "ThirdParty/glu/libtess/geom.h"
-#include "ThirdParty/glu/libtess/memalloc.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/priorityq.h"
-#include "ThirdParty/glu/libtess/sweep.h"
-#include "ThirdParty/glu/libtess/tess.h"
+#include "mesh.h"
+#include "geom.h"
+#include "tess.h"
+#include "dict.h"
+#include "priorityq.h"
+#include "memalloc.h"
+#include "sweep.h"
 
 #define TRUE 1
 #define FALSE 0
index 931a59a50b83cbc99580b621996a975d5959ba79..75d3e64095f244989983f089aad203aeabb39650 100644 (file)
@@ -42,7 +42,7 @@
 #ifndef __sweep_h_
 #define __sweep_h_
 
-#include "ThirdParty/glu/libtess/mesh.h"
+#include "mesh.h"
 
 /* __gl_computeInterior( tess ) computes the planar arrangement specified
  * by the given contours, and further subdivides this arrangement
index 0a639533f26b83087142581294a96ec912e4fd8b..c59a1789663052cc78825ab390c0e182f689ac56 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/tess.c#7 $
 */
 
+#include "gluos.h"
+#include <stddef.h>
 #include <assert.h>
 #include <setjmp.h>
-#include <stddef.h>
-
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/memalloc.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/normal.h"
-#include "ThirdParty/glu/libtess/render.h"
-#include "ThirdParty/glu/libtess/sweep.h"
-#include "ThirdParty/glu/libtess/tess.h"
-#include "ThirdParty/glu/libtess/tessmono.h"
+#include "memalloc.h"
+#include "tess.h"
+#include "mesh.h"
+#include "normal.h"
+#include "sweep.h"
+#include "tessmono.h"
+#include "render.h"
 
 #define GLU_TESS_DEFAULT_TOLERANCE 0.0
 #define GLU_TESS_MESH          100112  /* void (*)(GLUmesh *mesh)          */
@@ -92,7 +91,7 @@ typedef struct { GLUhalfEdge e, eSym; } EdgePair;
 
 
 GLUtesselator * GLAPIENTRY
-internal_gluNewTess( void )
+gluNewTess( void )
 {
   GLUtesselator *tess;
 
@@ -164,11 +163,11 @@ static void GotoState( GLUtesselator *tess, enum TessState newState )
       switch( tess->state ) {
       case T_DORMANT:
        CALL_ERROR_OR_ERROR_DATA( GLU_TESS_MISSING_BEGIN_POLYGON );
-       internal_gluTessBeginPolygon( tess, NULL );
+       gluTessBeginPolygon( tess, NULL );
        break;
       case T_IN_POLYGON:
        CALL_ERROR_OR_ERROR_DATA( GLU_TESS_MISSING_BEGIN_CONTOUR );
-       internal_gluTessBeginContour( tess );
+       gluTessBeginContour( tess );
        break;
       default:
         assert(0);
@@ -178,7 +177,7 @@ static void GotoState( GLUtesselator *tess, enum TessState newState )
       switch( tess->state ) {
       case T_IN_CONTOUR:
        CALL_ERROR_OR_ERROR_DATA( GLU_TESS_MISSING_END_CONTOUR );
-       internal_gluTessEndContour( tess );
+       gluTessEndContour( tess );
        break;
       case T_IN_POLYGON:
        CALL_ERROR_OR_ERROR_DATA( GLU_TESS_MISSING_END_POLYGON );
@@ -195,7 +194,7 @@ static void GotoState( GLUtesselator *tess, enum TessState newState )
 
 
 void GLAPIENTRY
-internal_gluDeleteTess( GLUtesselator *tess )
+gluDeleteTess( GLUtesselator *tess )
 {
   RequireState( tess, T_DORMANT );
   memFree( tess );
@@ -203,7 +202,7 @@ internal_gluDeleteTess( GLUtesselator *tess )
 
 
 void GLAPIENTRY
-internal_gluTessProperty( GLUtesselator *tess, GLenum which, GLdouble value )
+gluTessProperty( GLUtesselator *tess, GLenum which, GLdouble value )
 {
   GLenum windingRule;
 
@@ -242,7 +241,7 @@ internal_gluTessProperty( GLUtesselator *tess, GLenum which, GLdouble value )
 
 /* Returns tessellator property */
 void GLAPIENTRY
-internal_gluGetTessProperty( GLUtesselator *tess, GLenum which, GLdouble *value )
+gluGetTessProperty( GLUtesselator *tess, GLenum which, GLdouble *value )
 {
    switch (which) {
    case GLU_TESS_TOLERANCE:
@@ -270,7 +269,7 @@ internal_gluGetTessProperty( GLUtesselator *tess, GLenum which, GLdouble *value
 } /* gluGetTessProperty() */
 
 void GLAPIENTRY
-internal_gluTessNormal( GLUtesselator *tess, GLdouble x, GLdouble y, GLdouble z )
+gluTessNormal( GLUtesselator *tess, GLdouble x, GLdouble y, GLdouble z )
 {
   tess->normal[0] = x;
   tess->normal[1] = y;
@@ -278,7 +277,7 @@ internal_gluTessNormal( GLUtesselator *tess, GLdouble x, GLdouble y, GLdouble z
 }
 
 void GLAPIENTRY
-internal_gluTessCallback( GLUtesselator *tess, GLenum which, void (GLAPIENTRY *fn)())
+gluTessCallback( GLUtesselator *tess, GLenum which, void (GLAPIENTRY *fn)())
 {
   switch( which ) {
   case GLU_TESS_BEGIN:
@@ -417,7 +416,7 @@ static int EmptyCache( GLUtesselator *tess )
 
 
 void GLAPIENTRY
-internal_gluTessVertex( GLUtesselator *tess, GLdouble coords[3], void *data )
+gluTessVertex( GLUtesselator *tess, GLdouble coords[3], void *data )
 {
   int i, tooLarge = FALSE;
   GLdouble x, clamped[3];
@@ -464,7 +463,7 @@ internal_gluTessVertex( GLUtesselator *tess, GLdouble coords[3], void *data )
 
 
 void GLAPIENTRY
-internal_gluTessBeginPolygon( GLUtesselator *tess, void *data )
+gluTessBeginPolygon( GLUtesselator *tess, void *data )
 {
   RequireState( tess, T_DORMANT );
 
@@ -478,7 +477,7 @@ internal_gluTessBeginPolygon( GLUtesselator *tess, void *data )
 
 
 void GLAPIENTRY
-internal_gluTessBeginContour( GLUtesselator *tess )
+gluTessBeginContour( GLUtesselator *tess )
 {
   RequireState( tess, T_IN_POLYGON );
 
@@ -495,14 +494,14 @@ internal_gluTessBeginContour( GLUtesselator *tess )
 
 
 void GLAPIENTRY
-internal_gluTessEndContour( GLUtesselator *tess )
+gluTessEndContour( GLUtesselator *tess )
 {
   RequireState( tess, T_IN_CONTOUR );
   tess->state = T_IN_POLYGON;
 }
 
 void GLAPIENTRY
-internal_gluTessEndPolygon( GLUtesselator *tess )
+gluTessEndPolygon( GLUtesselator *tess )
 {
   GLUmesh *mesh;
 
index f837b90baee327a8cb95cf4309776fe44a8655a3..053fa00986e522d809dd437e557bcefaa72e12aa 100644 (file)
 #ifndef __tess_h_
 #define __tess_h_
 
+#include <internal_glu.h>
 #include <setjmp.h>
-
-#include "ThirdParty/glu/internal_glu.h"
-#include "ThirdParty/glu/libtess/dict.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/priorityq.h"
+#include "mesh.h"
+#include "dict.h"
+#include "priorityq.h"
 
 /* The begin/end calls must be properly nested.  We keep track of
  * the current state to enforce the ordering.
index 68fbe6818e1637d98497ef19232ae255e07ff3f2..2e2cc7901fefbff9c2abd55d2e513572511fc97f 100644 (file)
 ** $Header: //depot/main/gfx/lib/glu/libtess/tessmono.c#5 $
 */
 
-#include <assert.h>
+#include "gluos.h"
 #include <stdlib.h>
-
-#include "ThirdParty/glu/gluos.h"
-#include "ThirdParty/glu/libtess/geom.h"
-#include "ThirdParty/glu/libtess/mesh.h"
-#include "ThirdParty/glu/libtess/tessmono.h"
+#include "geom.h"
+#include "mesh.h"
+#include "tessmono.h"
+#include <assert.h>
 
 #define AddWinding(eDst,eSrc)  (eDst->winding += eSrc->winding, \
                                 eDst->Sym->winding += eSrc->Sym->winding)
index 69c420a0b7c0f70e3df7928478de2ab204a263bc..e470a4c11e4d96843c241184a2815cf2b6474e2b 100644 (file)
@@ -1,3 +1,13 @@
+2011-05-24  Stephen White  <senorblanco@chromium.org>
+
+        Reviewed by Kenneth Russell.
+
+        Remove most style checks on Source/ThirdParty/glu, since it doesn't
+        follow WebKit style.
+        https://bugs.webkit.org/show_bug.cgi?id=61365
+
+        * Scripts/webkitpy/style/checker.py:
+
 2011-05-24  Adam Roben  <aroben@apple.com>
 
         Don't allow forcing builds through the build.webkit.org web interface
index 27ba294f5a7c4d8e749474d995374461740e729a..20973a584a8ad3a3a69cfa343fd5a9dfb9ff8a2a 100644 (file)
@@ -206,6 +206,13 @@ _PATH_RULES_SPECIFIER = [
      ["-readability/naming",
       "-build/header_guard",
       "-build/include_order"]),
+
+    ([# glu's libtess is third-party code, and doesn't follow WebKit style.
+      "Source/ThirdParty/glu"],
+     ["-readability",
+      "-whitespace",
+      "-build/header_guard",
+      "-build/include_order"]),
 ]