Bug 44707 – Add thirdparty directory and incorporate GLU tessellator

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Kenneth Russell]

For new rendering algorithms being incorporated into WebKit, the polygon tessellator from the OpenGL utility library is needed. Some small modifications were needed from the original sources in SGI's OpenGL Sample Implementation (http://oss.sgi.com/projects/ogl-sample/). It is desired to be able to build these sources on any platform that wishes to utilize these new algorithms. For this reason a 3rdparty directory is being added to WebCore and the GLU tessellator sources within.

[Kenneth Russell]

Created attachment 65601 [details]
Patch

From the ChangeLog:

This directory is intended to contain copies of third-party libraries used by WebCore, in particular those which may require some modification in order to incorporate.

No tests at this time; these sources are being added in preparation for incorporating other code which uses them, at which point the code will be exercised and testable.

[WebKit Review Bot]

Attachment 65601 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
Last 3072 characters of output:
tyq-sort.h:114:  Extra space between void and pqDeletePriorityQ  [whitespace/declaration] [3]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:114:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:114:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:116:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:116:  Extra space between int and pqInit  [whitespace/declaration] [3]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:116:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:116:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:117:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:117:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:117:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:118:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:118:  Extra space between PQkey and pqExtractMin  [whitespace/declaration] [3]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:118:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:118:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:119:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:119:  Extra space between void and pqDelete  [whitespace/declaration] [3]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:119:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:119:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:121:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:121:  Extra space between PQkey and pqMinimum  [whitespace/declaration] [3]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:121:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:121:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:122:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:122:  Extra space between int and pqIsEmpty  [whitespace/declaration] [3]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:122:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/3rdparty/glu/libtess/priorityq-sort.h:122:  Extra space before )  [whitespace/parens] [2]
WebCore/3rdparty/glu/gluos.h:32:  #ifndef header guard has wrong style, please use: gluos_h  [build/header_guard] [5]
WebCore/3rdparty/glu/gluos.h:45:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 4489 in 34 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Kenneth Russell]

Created attachment 65616 [details]
Revised patch

Renamed 3rdparty directory to thirdparty on suggestion from hyatt.

[WebKit Review Bot]

Attachment 65616 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
Last 3072 characters of output:
y/glu/libtess/priorityq.h:114:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:114:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/libtess/priorityq.h:116:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/thirdparty/glu/libtess/priorityq.h:116:  Extra space between int and pqInit  [whitespace/declaration] [3]
WebCore/thirdparty/glu/libtess/priorityq.h:116:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:116:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/libtess/priorityq.h:117:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/thirdparty/glu/libtess/priorityq.h:117:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:117:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/libtess/priorityq.h:118:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/thirdparty/glu/libtess/priorityq.h:118:  Extra space between PQkey and pqExtractMin  [whitespace/declaration] [3]
WebCore/thirdparty/glu/libtess/priorityq.h:118:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:118:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/libtess/priorityq.h:119:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/thirdparty/glu/libtess/priorityq.h:119:  Extra space between void and pqDelete  [whitespace/declaration] [3]
WebCore/thirdparty/glu/libtess/priorityq.h:119:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:119:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/libtess/priorityq.h:121:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/thirdparty/glu/libtess/priorityq.h:121:  Extra space between PQkey and pqMinimum  [whitespace/declaration] [3]
WebCore/thirdparty/glu/libtess/priorityq.h:121:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:121:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/libtess/priorityq.h:122:  Tab found; better to use spaces  [whitespace/tab] [1]
WebCore/thirdparty/glu/libtess/priorityq.h:122:  Extra space between int and pqIsEmpty  [whitespace/declaration] [3]
WebCore/thirdparty/glu/libtess/priorityq.h:122:  Extra space after ( in function call  [whitespace/parens] [4]
WebCore/thirdparty/glu/libtess/priorityq.h:122:  Extra space before )  [whitespace/parens] [2]
WebCore/thirdparty/glu/internal_glu.h:36:  #ifndef header guard has wrong style, please use: internal_glu_h  [build/header_guard] [5]
WebCore/thirdparty/glu/internal_glu.h:113:  internal_gluNewTess is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WebCore/thirdparty/glu/internal_glu.h:141:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 4490 in 34 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Adam Barth]

> If any of these errors are false positives, please file a bug against check-webkit-style.

We really need to teach the stylebot to ignore third-party code.

[Darin Fisher (:fishd, Google)]

Comment on attachment 65616 [details]
Revised patch

View in context: https://bugs.webkit.org/attachment.cgi?id=65616&action=prettypatch

> WebCore/ChangeLog:30
> +        (dictNewDict):
nit: for new files that are added, it is nice to strip out mention of
the functions included in the new file since they aren't that helpful
when reading a ChangeLog entry like this.  you really just want to
know the set of files being added and the function names just add
noise.  (this was the advice that darin adler gave to me once upon
a time.)

otherwise, R=me

[Kenneth Russell]

(In reply to comment #6)
> (From update of attachment 65616 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=65616&action=prettypatch
> 
> > WebCore/ChangeLog:30
> > +        (dictNewDict):
> nit: for new files that are added, it is nice to strip out mention of
> the functions included in the new file since they aren't that helpful
> when reading a ChangeLog entry like this.  you really just want to
> know the set of files being added and the function names just add
> noise.  (this was the advice that darin adler gave to me once upon
> a time.)

Will do this upon landing.

> otherwise, R=me

[Kenneth Russell]

Committed r66784: <http://trac.webkit.org/changeset/66784>

[James Robinson]

Committed r66785: <http://trac.webkit.org/changeset/66785>