Remove timestamp again and some fixes for ufixed parsing.

chriseth · chriseth · commit 9b00290d7479 · 2016-03-11T17:50:09.000+01:00
diff --git a/libsolidity/parsing/Token.cpp b/libsolidity/parsing/Token.cpp
@@ -147,21 +147,25 @@ tuple<Token::Value, unsigned int, unsigned int> Token::fromIdentifierOrKeyword(s
 		}
 		else if (keyword == Token::UFixed || keyword == Token::Fixed)
 		{
-			auto positionN = find_if_not(++positionX, _literal.end(), ::isdigit);
-			int n = parseSize(++positionX, positionN);
 			if (
-				0 < m + n && 
-				m + n <= 256 && 
-				m % 8 == 0 && 
-				n % 8 == 0 && 
-				positionN == _literal.end() &&
-				*positionX == 'x'
-			)
-			{
-				if (keyword == Token::UFixed)
-					return make_tuple(Token::UFixed, m, n);
-				else
-					return make_tuple(Token::Fixed, m, n);
+				positionM < positionX &&
+				positionX < _literal.end() &&
+				*positionX == 'x' &&
+				all_of(positionX + 1, _literal.end(), ::isdigit)
+			) {
+				int n = parseSize(positionX + 1, _literal.end());
+				if (
+					0 < m && m < 256 &&
+					0 < n && n < 256 &&
+					m + n <= 256 &&
+					m % 8 == 0 &&
+					n % 8 == 0
+				) {
+					if (keyword == Token::UFixed)
+						return make_tuple(Token::UFixed, m, n);
+					else
+						return make_tuple(Token::Fixed, m, n);
+				}
 			}	
 		}
 		return make_tuple(Token::Identifier, 0, 0);
diff --git a/libsolidity/parsing/Token.h b/libsolidity/parsing/Token.h
@@ -231,7 +231,6 @@ namespace solidity
 	K(Type, "type", 0)                                                 \
 	K(TypeOf, "typeof", 0)                                             \
 	K(Using, "using", 0)                                               \
-	T(Timestamp, "timestamp", 0)                                       \
 	/* Illegal token - not able to scan. */                            \
 	T(Illegal, "ILLEGAL", 0)                                           \
 	\

Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,6 @@ namespace solidity`
`231`	`231`	`K(Type, "type", 0) \`
`232`	`232`	`K(TypeOf, "typeof", 0) \`
`233`	`233`	`K(Using, "using", 0) \`
`234`		`- T(Timestamp, "timestamp", 0) \`
`235`	`234`	`/* Illegal token - not able to scan. */ \`
`236`	`235`	`T(Illegal, "ILLEGAL", 0) \`
`237`	`236`	`\`