Title: [94757] trunk/Source
- Revision
- 94757
- Author
- abe...@webkit.org
- Date
- 2011-09-08 06:52:04 -0700 (Thu, 08 Sep 2011)
Log Message
[Qt] Build fails with strict compiler
https://bugs.webkit.org/show_bug.cgi?id=67778
Reviewed by Csaba Osztrogonác.
Source/ThirdParty/ANGLE:
* src/compiler/glslang_lex.cpp: Regenerate with generate_glslang_lexer.sh using a newer flex
to suppress warning and fix the build when using [-Werror=unused-result]
Source/WebCore:
No new tests needed.
* platform/graphics/TiledBackingStore.cpp:
(WebCore::TiledBackingStore::resizeEdgeTiles): Remove unused contentsRect variable
which's usage was removed in r94681 to fix the build with [-Werror=unused-but-set-variable].
Modified Paths
Diff
Modified: trunk/Source/ThirdParty/ANGLE/ChangeLog (94756 => 94757)
--- trunk/Source/ThirdParty/ANGLE/ChangeLog 2011-09-08 13:44:19 UTC (rev 94756)
+++ trunk/Source/ThirdParty/ANGLE/ChangeLog 2011-09-08 13:52:04 UTC (rev 94757)
@@ -1,3 +1,13 @@
+2011-09-08 Andras Becsi <andras.be...@nokia.com>
+
+ [Qt] Build fails with strict compiler
+ https://bugs.webkit.org/show_bug.cgi?id=67778
+
+ Reviewed by Csaba Osztrogonác.
+
+ * src/compiler/glslang_lex.cpp: Regenerate with generate_glslang_lexer.sh using a newer flex
+ to suppress warning and fix the build when using [-Werror=unused-result]
+
2011-08-12 Mark Rowe <mr...@apple.com>
Be more forward-looking in the choice of compiler.
Modified: trunk/Source/ThirdParty/ANGLE/src/compiler/glslang_lex.cpp (94756 => 94757)
--- trunk/Source/ThirdParty/ANGLE/src/compiler/glslang_lex.cpp 2011-09-08 13:44:19 UTC (rev 94756)
+++ trunk/Source/ThirdParty/ANGLE/src/compiler/glslang_lex.cpp 2011-09-08 13:52:04 UTC (rev 94757)
@@ -859,6 +859,10 @@
void yyset_lineno (int line_number ,yyscan_t yyscanner );
+int yyget_column (yyscan_t yyscanner );
+
+void yyset_column (int column_no ,yyscan_t yyscanner );
+
YYSTYPE * yyget_lval (yyscan_t yyscanner );
void yyset_lval (YYSTYPE * yylval_param ,yyscan_t yyscanner );
@@ -909,7 +913,7 @@
/* This used to be an fputs(), but since the string might contain NUL's,
* we now use fwrite().
*/
-#define ECHO fwrite( yytext, yyleng, 1, yyout )
+#define ECHO do { if (fwrite( yytext, yyleng, 1, yyout )) {} } while (0)
#endif
/* Gets input and stuffs it into "buf". number of characters read, or YY_NULL,
@@ -920,7 +924,7 @@
if ( YY_CURRENT_BUFFER_LVALUE->yy_is_interactive ) \
{ \
int c = '*'; \
- int n; \
+ unsigned n; \
for ( n = 0; n < max_size && \
(c = getc( yyin )) != EOF && c != '\n'; ++n ) \
buf[n] = (char) c; \
Modified: trunk/Source/WebCore/ChangeLog (94756 => 94757)
--- trunk/Source/WebCore/ChangeLog 2011-09-08 13:44:19 UTC (rev 94756)
+++ trunk/Source/WebCore/ChangeLog 2011-09-08 13:52:04 UTC (rev 94757)
@@ -1,3 +1,16 @@
+2011-09-08 Andras Becsi <andras.be...@nokia.com>
+
+ [Qt] Build fails with strict compiler
+ https://bugs.webkit.org/show_bug.cgi?id=67778
+
+ Reviewed by Csaba Osztrogonác.
+
+ No new tests needed.
+
+ * platform/graphics/TiledBackingStore.cpp:
+ (WebCore::TiledBackingStore::resizeEdgeTiles): Remove unused contentsRect variable
+ which's usage was removed in r94681 to fix the build with [-Werror=unused-but-set-variable].
+
2011-09-08 Sheriff Bot <webkit.review....@gmail.com>
Unreviewed, rolling out r94695.
Modified: trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp (94756 => 94757)
--- trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp 2011-09-08 13:44:19 UTC (rev 94756)
+++ trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp 2011-09-08 13:52:04 UTC (rev 94757)
@@ -340,7 +340,6 @@
bool TiledBackingStore::resizeEdgeTiles()
{
- IntRect contentsRect = this->contentsRect();
bool wasResized = false;
Vector<Tile::Coordinate> tilesToRemove;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes