Changed the way Dalvik operands are handled.

2 files changed, 34 insertions, 34 deletions

diff --git a/plugins/dalvik/operand.c b/plugins/dalvik/operand.c
index e89bcab..3470736 100644
--- a/plugins/dalvik/operand.c
+++ b/plugins/dalvik/operand.c
@@ -164,7 +164,7 @@ static bool dalvik_read_basic_operands(GArchInstruction *instr, GDexFormat *form
 
     /* Choix des opérandes à charger */
 
-    switch (model & ~DALVIK_OP_EXTRA_MASK)
+    switch (DALVIK_OP_BASE_MASK(model))
     {
         case DALVIK_OPT_10T:
             types = (DalvikOperandID []) {
@@ -643,7 +643,7 @@ bool dalvik_read_operands(GArchInstruction *instr, GExeFormat *format, const GBi
 
     /* Récupération de la base ? */
 
-    if (DALVIK_OP_GET_MNEMONIC(model) == 'T')
+    if (DALVIK_OP_GET_MNEMONIC(model) == DALVIK_OP_MNEMONIC_1('t'))
     {
         extra = &base;
 
@@ -655,7 +655,7 @@ bool dalvik_read_operands(GArchInstruction *instr, GExeFormat *format, const GBi
 
     /* Bourrage : ØØ|op ? */
 
-    switch (model & ~DALVIK_OP_EXTRA_MASK)
+    switch (DALVIK_OP_BASE_MASK(model))
     {
         case DALVIK_OPT_10X:
         case DALVIK_OPT_20T:
@@ -671,7 +671,7 @@ bool dalvik_read_operands(GArchInstruction *instr, GExeFormat *format, const GBi
 
     /* Décodage... */
 
-    switch (model & ~DALVIK_OP_EXTRA_MASK)
+    switch (DALVIK_OP_BASE_MASK(model))
     {
         case DALVIK_OPT_10T:
         case DALVIK_OPT_11N:
@@ -702,8 +702,6 @@ bool dalvik_read_operands(GArchInstruction *instr, GExeFormat *format, const GBi
             break;
 
         case DALVIK_OPT_3RC:
-        case DALVIK_OPT_3RMS:
-        case DALVIK_OPT_3RFS:
             result = dalvik_read_variatic_operands(instr, dformat, content, pos, &low, endian, model);
             break;
 
diff --git a/plugins/dalvik/operand.h b/plugins/dalvik/operand.h
index 7b4275e..4673b85 100644
--- a/plugins/dalvik/operand.h
+++ b/plugins/dalvik/operand.h
@@ -55,7 +55,7 @@
 #define DALVIK_OP_POOL_OFF  20
 #define DALVIK_OP_POOL_MASK 0x00f00000
 
-#define DALVIK_OP_EXTRA_MASK (DALVIK_OP_POOL_MASK)
+#define DALVIK_OP_BASE_MASK(b) (b & ~DALVIK_OP_POOL_MASK)
 
 #define DALVIK_OP_LEN(l)        ((l) << DALVIK_OP_LEN_OFF)
 #define DALVIK_OP_GET_LEN(v)    (((v) & DALVIK_OP_LEN_MASK) >> DALVIK_OP_LEN_OFF)
@@ -66,50 +66,52 @@
 #define DALVIK_OP_POOL(p)       ((p) << DALVIK_OP_POOL_OFF)
 #define DALVIK_OP_GET_POOL(v)   (((v) & DALVIK_OP_POOL_MASK) >> DALVIK_OP_POOL_OFF)
 
-#define DALVIK_OP_GET_MNEMONIC(v) ((v) & 0xff)
+#define DALVIK_OP_MNEMONIC_1(v0)         ((v0) - 'a')
+#define DALVIK_OP_MNEMONIC_2(v0, v1)     (((v0) - 'a') | ((v1) - 'a') << 5)
+#define DALVIK_OP_MNEMONIC_3(v0, v1, v2) (((v0) - 'a') | ((v1) - 'a') << 5 | ((v2) - 'a') << 10)
+
+#define DALVIK_OP_GET_MNEMONIC(v) (v & 0x7fff)
 
 
 /* Types d'opérandes supportés */
 typedef enum _DalvikOperandType
 {
-    DALVIK_OPT_10T      = DALVIK_OP_LEN(1) | DALVIK_OP_REG(0) | 'T',
-    DALVIK_OPT_10X      = DALVIK_OP_LEN(1) | DALVIK_OP_REG(0) | 'X',
+    DALVIK_OPT_10T = DALVIK_OP_LEN(1) | DALVIK_OP_REG(0) | DALVIK_OP_MNEMONIC_1('t'),
+    DALVIK_OPT_10X = DALVIK_OP_LEN(1) | DALVIK_OP_REG(0) | DALVIK_OP_MNEMONIC_1('x'),
 
-    DALVIK_OPT_11N      = DALVIK_OP_LEN(1) | DALVIK_OP_REG(1) | 'N',
-    DALVIK_OPT_11X      = DALVIK_OP_LEN(1) | DALVIK_OP_REG(1) | 'X',
+    DALVIK_OPT_11N = DALVIK_OP_LEN(1) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('n'),
+    DALVIK_OPT_11X = DALVIK_OP_LEN(1) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('x'),
 
-    DALVIK_OPT_12X      = DALVIK_OP_LEN(1) | DALVIK_OP_REG(2) | 'X',
+    DALVIK_OPT_12X = DALVIK_OP_LEN(1) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('x'),
 
-    DALVIK_OPT_20T      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(0) | 'T',
+    DALVIK_OPT_20T = DALVIK_OP_LEN(2) | DALVIK_OP_REG(0) | DALVIK_OP_MNEMONIC_1('t'),
 
-    DALVIK_OPT_21C      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | 'C',
-    DALVIK_OPT_21H      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | 'H',
-    DALVIK_OPT_21S      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | 'S',
-    DALVIK_OPT_21T      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | 'T',
+    DALVIK_OPT_21C = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('c'),
+    DALVIK_OPT_21H = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('h'),
+    DALVIK_OPT_21S = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('s'),
+    DALVIK_OPT_21T = DALVIK_OP_LEN(2) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('t'),
 
-    DALVIK_OPT_22B      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | 'B',
-    DALVIK_OPT_22C      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | 'C',
-    DALVIK_OPT_22S      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | 'S',
-    DALVIK_OPT_22T      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | 'T',
-    DALVIK_OPT_22X      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | 'X',
+    DALVIK_OPT_22B = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('b'),
+    DALVIK_OPT_22C = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('c'),
+    DALVIK_OPT_22S = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('s'),
+    DALVIK_OPT_22T = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('t'),
+    DALVIK_OPT_22X = DALVIK_OP_LEN(2) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('x'),
 
-    DALVIK_OPT_23X      = DALVIK_OP_LEN(2) | DALVIK_OP_REG(3) | 'X',
+    DALVIK_OPT_23X = DALVIK_OP_LEN(2) | DALVIK_OP_REG(3) | DALVIK_OP_MNEMONIC_1('x'),
 
-    DALVIK_OPT_30T      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(0) | 'T',
+    DALVIK_OPT_30T = DALVIK_OP_LEN(3) | DALVIK_OP_REG(0) | DALVIK_OP_MNEMONIC_1('t'),
 
-    DALVIK_OPT_31C      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(1) | 'C',
-    DALVIK_OPT_31I      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(1) | 'I',
-    DALVIK_OPT_31T      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(1) | 'T',
+    DALVIK_OPT_31C = DALVIK_OP_LEN(3) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('c'),
+    DALVIK_OPT_31I = DALVIK_OP_LEN(3) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('i'),
+    DALVIK_OPT_31T = DALVIK_OP_LEN(3) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('t'),
 
-    DALVIK_OPT_32X      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(2) | 'X',
+    DALVIK_OPT_32X = DALVIK_OP_LEN(3) | DALVIK_OP_REG(2) | DALVIK_OP_MNEMONIC_1('x'),
 
-    DALVIK_OPT_35C      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(5) | 'C',
+    DALVIK_OPT_35C = DALVIK_OP_LEN(3) | DALVIK_OP_REG(5) | DALVIK_OP_MNEMONIC_1('c'),
 
-    DALVIK_OPT_3RC      = DALVIK_OP_LEN(3) | DALVIK_OP_REG(DALVIK_OP_REG_RANGE) | 'C',
-    DALVIK_OPT_3RMS     = DALVIK_OP_LEN(3) | DALVIK_OP_REG(DALVIK_OP_REG_RANGE) | 'M',
-    DALVIK_OPT_3RFS     = DALVIK_OP_LEN(3) | DALVIK_OP_REG(DALVIK_OP_REG_RANGE) | 'F',
+    DALVIK_OPT_3RC = DALVIK_OP_LEN(3) | DALVIK_OP_REG(DALVIK_OP_REG_RANGE) | DALVIK_OP_MNEMONIC_2('r', 'c'),
 
-    DALVIK_OPT_51L      = DALVIK_OP_LEN(5) | DALVIK_OP_REG(1) | 'L'
+    DALVIK_OPT_51L = DALVIK_OP_LEN(5) | DALVIK_OP_REG(1) | DALVIK_OP_MNEMONIC_1('l')
 
 } DalvikOperandType;