WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
44707
Add thirdparty directory and incorporate GLU tessellator
https://bugs.webkit.org/show_bug.cgi?id=44707
Summary
Add thirdparty directory and incorporate GLU tessellator
Kenneth Russell
Reported
2010-08-26 12:32:45 PDT
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.
Attachments
Patch
(273.34 KB, patch)
2010-08-26 12:39 PDT
,
Kenneth Russell
kbr
: commit-queue-
Details
Formatted Diff
Diff
Revised patch
(273.73 KB, patch)
2010-08-26 13:59 PDT
,
Kenneth Russell
fishd
: review+
kbr
: commit-queue-
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Kenneth Russell
Comment 1
2010-08-26 12:39:06 PDT
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
Comment 2
2010-08-26 12:43:12 PDT
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
Comment 3
2010-08-26 13:59:43 PDT
Created
attachment 65616
[details]
Revised patch Renamed 3rdparty directory to thirdparty on suggestion from hyatt.
WebKit Review Bot
Comment 4
2010-08-26 14:05:04 PDT
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
Comment 5
2010-08-26 23:30:31 PDT
> 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 6
2010-09-03 17:00:35 PDT
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
Comment 7
2010-09-03 17:48:54 PDT
(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
Comment 8
2010-09-03 18:20:17 PDT
Committed
r66784
: <
http://trac.webkit.org/changeset/66784
>
James Robinson
Comment 9
2010-09-03 18:24:45 PDT
Committed
r66785
: <
http://trac.webkit.org/changeset/66785
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug