Mesa (master): glsl: Use | action in the lexer source to avoid duplicating the float action
Neil Roberts
nroberts at kemper.freedesktop.org
Thu Nov 27 11:48:09 UTC 2014
Module: Mesa
Branch: master
Commit: c97cbd7e3d9faae1185a740d2a94c48e9a75d8b9
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c97cbd7e3d9faae1185a740d2a94c48e9a75d8b9
Author: Neil Roberts <neil at linux.intel.com>
Date: Wed Nov 26 17:57:42 2014 +0000
glsl: Use | action in the lexer source to avoid duplicating the float action
Flex and lex have a special action ‘|’ which means to use the same action as
the next rule. We can use this to reduce a bit of code duplication in the
rules for the various float literal formats.
Reviewed-by: Matt Turner <mattst88 at gmail.com>
---
src/glsl/glsl_lexer.ll | 15 +++------------
1 file changed, 3 insertions(+), 12 deletions(-)
diff --git a/src/glsl/glsl_lexer.ll b/src/glsl/glsl_lexer.ll
index 419a07b..57c46be 100644
--- a/src/glsl/glsl_lexer.ll
+++ b/src/glsl/glsl_lexer.ll
@@ -450,18 +450,9 @@ layout {
return LITERAL_INTEGER(8);
}
-[0-9]+\.[0-9]+([eE][+-]?[0-9]+)?[fF]? {
- yylval->real = _mesa_strtof(yytext, NULL);
- return FLOATCONSTANT;
- }
-\.[0-9]+([eE][+-]?[0-9]+)?[fF]? {
- yylval->real = _mesa_strtof(yytext, NULL);
- return FLOATCONSTANT;
- }
-[0-9]+\.([eE][+-]?[0-9]+)?[fF]? {
- yylval->real = _mesa_strtof(yytext, NULL);
- return FLOATCONSTANT;
- }
+[0-9]+\.[0-9]+([eE][+-]?[0-9]+)?[fF]? |
+\.[0-9]+([eE][+-]?[0-9]+)?[fF]? |
+[0-9]+\.([eE][+-]?[0-9]+)?[fF]? |
[0-9]+[eE][+-]?[0-9]+[fF]? {
yylval->real = _mesa_strtof(yytext, NULL);
return FLOATCONSTANT;
More information about the mesa-commit
mailing list