X-Git-Url: http://shamusworld.gotdns.org/cgi-bin/gitweb.cgi?p=rmac;a=blobdiff_plain;f=parmode.h;h=64c3b5c86981c3576a099d2a921d04328de219da;hp=89d53a7da60ad32e612259f185b66660171c30c3;hb=eace4e1b294ccec54a5c476619f616f5da0bf8a9;hpb=05d0350b35a6a6b255cb2a3fab7796f5d4ee4d02

diff --git a/parmode.h b/parmode.h
index 89d53a7..64c3b5c 100644
--- a/parmode.h
+++ b/parmode.h
@@ -78,6 +78,7 @@
 		{
 			//Since index register isn't used here, store register number in this field
 			AnIXREG = *tok++ & 7;                                // (Dn)
+
 			if (*tok == ')')
 			{
 				tok++;
@@ -85,21 +86,21 @@
 				AnEXTEN |= EXT_BS;          // Base register suppressed
 				AnEXTEN |= EXT_BDSIZE0;     // Base displacement null
 				AnEXTEN |= EXT_IISPOSN;     // Indirect Postindexed with Null Outer Displacement
-				AMn= MEMPOST;
+				AMn = MEMPOST;
 				AnREG = 6 << 3;		// stuff 110 to mode field
 				goto AnOK;
 			}
 			else if (*tok == 'L')
 			{
 				// TODO: does DINDL gets used at all?
-				AMn=DINDL;                                     // (Dn.l)
+				AMn = DINDL;                                     // (Dn.l)
 				AnEXTEN = 1 << 1;   // Long index size
 				tok++;
 			}
 			else if (*tok == 'W')                                // (Dn.w)
 			{
 				// TODO: does DINDW gets used at all?
-				AMn=DINDW;
+				AMn = DINDW;
 				AnEXTEN = 1 << 1;   // Word index size
 				tok++;
 			}
@@ -179,12 +180,13 @@
 				AMn = MEMPOST;
 				goto AnOK;
 			}
-			else if (*tok==',')
+			else if (*tok == ',')
 			{
 				tok++;  // eat the comma
 				// It might be (Dn[.wl][*scale],od)
 				// Maybe this is wrong and we have to write some code here
 				// instead of reusing that path...
+				AnEXTEN |= EXT_BDSIZE0;     // Base displacement null - suppressed
 				goto CHECKODn;
 			}
 			else
@@ -278,7 +280,11 @@
 
 			if (*tok == ',')
 			{
+				// If we got here we didn't get any [] stuff
+				// so let's suppress base displacement before
+				// branching off
 				tok++;
+				AnEXTEN |= EXT_BDSIZE0;     // Base displacement null - suppressed
 				goto CHECKODn;
 			}
 			if (*tok++ != ')')         // final ")"
@@ -289,23 +295,24 @@
 		else if (*tok == '[')
 		{                              // ([...
 			tok++;
-			AnEXTEN|=EXT_FULLWORD;     //Definitely using full extension format, so set bit 8
+			AnEXTEN |= EXT_FULLWORD;     // Definitely using full extension format, so set bit 8
+
 			// Check to see if base displacement is present
-			if (*tok!=CONST && *tok !=SYMBOL)
+			if (*tok != CONST && *tok != SYMBOL)
 			{
-				AnEXTEN|=EXT_BDSIZE0;
+				AnEXTEN |= EXT_BDSIZE0;
 			}
 			else
 			{
 				expr(AnBEXPR, &AnBEXVAL, &AnBEXATTR, &AnESYM);
-				if (CHECK_OPTS(OPT_BASE_DISP) && AnBEXVAL==0 && AnEXATTR!=0)
+				if (CHECK_OPTS(OPT_BASE_DISP) && AnBEXVAL == 0 && AnEXATTR != 0)
 				{
 					// bd=0 so let's optimise it out
 					AnEXTEN|=EXT_BDSIZE0;
 				}
-				else if (*tok==DOTL)
+				else if (*tok == DOTL)
 				{						// ([bd.l,...
-						AnEXTEN|=EXT_BDSIZEL;
+						AnEXTEN |= EXT_BDSIZEL;
 						tok++;
 				}
 				else
@@ -313,7 +320,7 @@
 					// Is .W forced here?
 					if (*tok == DOTW)
 					{
-    					AnEXTEN|=EXT_BDSIZEW;
+						AnEXTEN |= EXT_BDSIZEW;
 						tok++;
 					}
 					else
@@ -322,7 +329,7 @@
 						// to absolute short
 						if (CHECK_OPTS(OPT_ABS_SHORT)
 							&& ((AnBEXATTR & (TDB | DEFINED)) == DEFINED)
-							&& ((AnBEXVAL + 0x8000) < 0x10000))
+							&& (((uint32_t)AnBEXVAL + 0x8000) < 0x10000))
 						{
 							AnEXTEN |= EXT_BDSIZEW;
 							warn("absolute value in base displacement ranging $FFFF8000..$00007FFF optimised to absolute short");
@@ -350,7 +357,7 @@
 			}
 			else if ((*tok >= KW_A0) && (*tok <= KW_A7))
 			{					// ([bd,An,...
-				AnREG = (6<<3)|*tok & 7;
+				AnREG = (6 << 3) | *tok & 7;
 				tok++;
 			}
 			else if ((*tok >= KW_D0) && (*tok <= KW_D7))
@@ -364,23 +371,23 @@
 
 				// Check for size
 				{
-					// ([bd,An/PC],Xn.W/L...)
-					switch ((int)*tok)
-					{
-					// Index reg size: <empty> | .W | .L
-					case DOTW:
-						tok++;
-						break;
-					default:
-						break;
-					case DOTL:
-						AnEXTEN |= EXT_L;
-						tok++;
-						break;
-					case DOTB:
-						// .B not allowed here...
-						goto badmode;
-					}
+				// ([bd,An/PC],Xn.W/L...)
+				switch ((int)*tok)
+				{
+				// Index reg size: <empty> | .W | .L
+				case DOTW:
+					tok++;
+					break;
+				default:
+					break;
+				case DOTL:
+					AnEXTEN |= EXT_L;
+					tok++;
+					break;
+				case DOTB:
+					// .B not allowed here...
+					goto badmode;
+				}
 				}
 
 				// Check for scale
@@ -392,6 +399,7 @@
 					{
 						if (expr(AnEXPR, &AnEXVAL, &AnEXATTR, &AnESYM) != OK)
 							return error("scale factor expression must evaluate");
+
 						switch (AnEXVAL)
 						{
 						case 1:
@@ -569,7 +577,7 @@
 					tok++;
 					goto AnOK;
 				}
-				else if (*tok!=',')
+				else if (*tok != ',')
 					return error("comma expected");
 				else
 					tok++;	// eat the comma
@@ -586,6 +594,7 @@
 					tok++;
 					goto AnOK;
 				}
+
 				// ([bd,An/PC],Xn,od)
 				if (*tok == DOTL)
 				{
@@ -598,7 +607,7 @@
 					// optimized to absolute short
 					if (CHECK_OPTS(OPT_ABS_SHORT)
 						&& ((AnEXATTR & (TDB | DEFINED)) == DEFINED)
-						&& ((AnEXVAL + 0x8000) < 0x10000))
+						&& (((uint32_t)AnEXVAL + 0x8000) < 0x10000))
 					{
 						AnEXTEN |= EXT_IISPOSW; // Word outer displacement
 						AMn = MEMPOST;
@@ -628,7 +637,7 @@
 				else
 					return error("Closing parenthesis missing on addressing mode");
 
-				IS_SUPPRESSEDn:
+IS_SUPPRESSEDn:
 
 				// Check for od
 				if (*tok == ')')	// ([bd,An/PC],Xn)
@@ -671,18 +680,18 @@
 					// expr[.W][]
 					AnEXTEN |= EXT_IISNOIW; // Word outer displacement with IS suppressed
 					AMn = MEMPRE;
+
 					if (*tok == DOTW)
 					{
 						//AnEXTEN|=EXT_IISNOIW; // Word outer displacement
 						AMn = MEMPOST;
 						tok++;
 					}
-
 					// Defined, absolute values from $FFFF8000..$00007FFF get
 					// optimized to absolute short
 					else if (CHECK_OPTS(OPT_BASE_DISP)
 						&& ((AnEXATTR & (TDB | DEFINED)) == DEFINED)
-						&& ((AnEXVAL + 0x8000) < 0x10000))
+						&& (((uint32_t)AnEXVAL + 0x8000) < 0x10000))
 					{
 						//AnEXTEN|=EXT_IISNOIW; // Word outer displacement with IS suppressed
 						warn("outer displacement absolute value from $FFFF8000..$00007FFF optimised to absolute short");
@@ -718,23 +727,23 @@
 
 				// Check for size
 				{
-					// ([bd,An/PC],Xn.W/L...)
-					switch ((int)*tok)
-					{
-					// Index reg size: <empty> | .W | .L
-					case DOTW:
-						tok++;
-						break;
-					default:
-						break;
-					case DOTL:
-						tok++;
-						AnEXTEN |= EXT_L;
-						break;
-					case DOTB:
-						// .B not allowed here...
-						goto badmode;
-					}
+				// ([bd,An/PC],Xn.W/L...)
+				switch ((int)*tok)
+				{
+				// Index reg size: <empty> | .W | .L
+				case DOTW:
+					tok++;
+					break;
+				default:
+					break;
+				case DOTL:
+					tok++;
+					AnEXTEN |= EXT_L;
+					break;
+				case DOTB:
+					// .B not allowed here...
+					goto badmode;
+				}
 				}
 
 				// Check for scale
@@ -804,20 +813,20 @@
                 else if (*tok++ != ',')
 					return error("comma expected after ]");
 
-                if (*tok == SYMBOL || *tok == CONST)
-                {
-				if (expr(AnEXPR, &AnEXVAL, &AnEXATTR, &AnESYM) != OK)
-					goto badmode;
-
-                    if (CHECK_OPTS(OPT_BASE_DISP) && (AnEXVAL == 0) && (AnEXATTR & DEFINED))
+				if (*tok == SYMBOL || *tok == CONST)
 				{
-					// od=0 so optimise it out
-					AMn = MEMPRE;		 // let's say it's ([bd,An],Xn,od) with od=0 then
-					AnEXTEN |= EXT_IISPRE0; // No outer displacement
-					tok++;
-					goto AnOK;
+					if (expr(AnEXPR, &AnEXVAL, &AnEXATTR, &AnESYM) != OK)
+						goto badmode;
+
+					if (CHECK_OPTS(OPT_BASE_DISP) && (AnEXVAL == 0) && (AnEXATTR & DEFINED))
+					{
+						// od=0 so optimise it out
+						AMn = MEMPRE;		 // let's say it's ([bd,An],Xn,od) with od=0 then
+						AnEXTEN |= EXT_IISPRE0; // No outer displacement
+						tok++;
+						goto AnOK;
+					}
 				}
-                }
 
 				// ([bd,An/PC,Xn],od)
 				if (*tok == DOTL)
@@ -830,36 +839,35 @@
 				else
 				{
 					// expr.[W]
-                    int expr_size;
-
 					AMn = MEMPRE;
-                    expr_size = EXT_IISPREW; // Assume we have a .w value
+					int expr_size = EXT_IISPREW; // Assume we have a .w value
 
-                    if ((AnEXVAL + 0x8000) > 0x10000)
+					if ((AnEXVAL + 0x8000) > 0x10000)
 					{
-                        // Long value, so mark it as such for now
-                        expr_size = EXT_IISPREL;
-					// Defined, absolute values from $FFFF8000..$00007FFF get optimized
-					// to absolute short
+						// Long value, so mark it as such for now
+						expr_size = EXT_IISPREL;
+
+						// Defined, absolute values from $FFFF8000..$00007FFF
+						// get optimized to absolute short
                         if (CHECK_OPTS(OPT_BASE_DISP)
-						&& ((AnEXATTR & (TDB | DEFINED)) == DEFINED)
-						&& ((AnEXVAL + 0x8000) < 0x10000))
-					{
-                            expr_size = EXT_IISPREW;
-						warn("outer displacement absolute value from $FFFF8000..$00007FFF optimised to absolute short");
+							&& ((AnEXATTR & (TDB | DEFINED)) == DEFINED)
+							&& (((uint32_t)AnEXVAL + 0x8000) < 0x10000))
+						{
+							expr_size = EXT_IISPREW;
+							warn("outer displacement absolute value from $FFFF8000..$00007FFF optimised to absolute short");
+						}
 					}
-				}
 
-                    AnEXTEN |= expr_size; // Assume we have a .w value
+					AnEXTEN |= expr_size; // Assume we have a .w value
 
-                    // Is .W forced here?
+					// Is .W forced here?
 					if (*tok == DOTW)
 					{
 						tok++;
-                        if (expr_size == EXT_IISPREL)
-                            return error("outer displacement value does not fit in .w size");
-					}
 
+						if (expr_size == EXT_IISPREL)
+							return error("outer displacement value does not fit in .w size");
+					}
 				}
 
 				// Check for final closing parenthesis
@@ -1015,7 +1023,7 @@ CHK_FOR_DISPn:
 			// to absolute short
 			if (CHECK_OPTS(OPT_ABS_SHORT)
 				&& ((AnEXATTR & (TDB | DEFINED)) == DEFINED)
-				&& ((AnEXVAL + 0x8000) < 0x10000))
+				&& (((uint32_t)AnEXVAL + 0x8000) < 0x10000))
 			{
 				AMn = ABSW;