[m-dev.] for review: workaround for the large term problem in the debugger

Zoltan Somogyi zs at cs.mu.OZ.AU
Sun Jun 14 14:46:01 AEST 1998


Tyson wrote:
> Not entirely true -- '*' or any other non-number will indicate all
> variables.  Might be confusing to type 'p X' and have it print all
> variables.  This isn't a big deal, but at least add an XXX to say
> this should be fixed.

Thanks for pointing out this oversight. What I committed has the fix:

@@ -281,10 +281,13 @@
 					printf("There is no variable #%d.\n",
 						n);
 				}
-			} else {
+			} else if streq(words[1], "*") {
 				MR_trace_browse_all((int)
 					layout->MR_sll_var_count,
 					&layout->MR_sll_var_info);
+			} else {
+				printf("The argument of this command should be,\n");
+				printf("a variable number or a '*' indicating all variables.\n");
 			}
 		} else {
 			printf("This command expects one argument,\n");

Zoltan.



More information about the developers mailing list