misc TODO review/cleanup

git-svn-id: https://smali.googlecode.com/svn/trunk@130 55b6fa8a-2a1e-11de-a435-ffa8d773f76a
This commit is contained in:
JesusFreke@JesusFreke.com 2009-06-08 01:24:40 +00:00
parent 0fa57d3fe1
commit 34a4a30e34
3 changed files with 3 additions and 8 deletions

View File

@ -923,9 +923,7 @@ fragment TYPE_FIELD_METHOD_LITERAL_EMITCHILDREN
MEMBER_NAME_EMIT
( METHOD_PROTOTYPE_EMITCHILDREN
| COLON_EMIT NONVOID_TYPE_DESCRIPTOR_EMITCHILD))?
//TODO: allow void and primitive type here?
| PRIMITIVE_TYPE_EMIT
| VOID_TYPE_EMIT;
| PRIMITIVE_TYPE_EMIT;
fragment ENUM_EMIT
: ENUM {emit($ENUM, ENUM);};

View File

@ -176,13 +176,11 @@ smali_file
EOF
{
if (!$smali_file::hasClassSpec) {
//TODO: throw correct exception type
throw new RuntimeException("The file must contain a .class directive");
throw new SemanticException(input, "The file must contain a .class directive");
}
if (!$smali_file::hasSuperSpec) {
//TODO: throw correct exception type
throw new RuntimeException("The file must contain a .super directive");
throw new SemanticException(input, "The file must contain a .super directive");
}
}
-> ^(I_CLASS_DEF

View File

@ -1061,7 +1061,6 @@ register_list[int totalMethodRegisters, int methodParameterRegisters] returns[by
: ^(I_REGISTER_LIST
(REGISTER
{
//TODO: shouldn't this be == 6?
if ($registerCount == 5) {
throw new SemanticException(input, "A list of registers can only have a maximum of 5 registers. Use the <op>/range alternate opcode instead.");
}