Re: [RFA] monitor.c fix for rom68k target boards

[Andrew Cagney <ac131313@cygnus.com>]

> A possibility would be to change the test back to the original version
> rather than removing the test, like:
> 
>   if (val == 0 && valstr == p)
>      error ("monitor_supply_register (%d):  bad value from monitor: %s.",
>            regno, valstr);

Yes, that sounds better.  I think the function should still fail if it
does nothing - that would tend to suggest some sort of real problem with
the monitor.

> I'm willing to go either way, I just don't know what the intent was in
> changing the if test.

I don't think it was intentional.  Any way, assume your patch is
approved with the tweeked test.

enjoy,
	Andrew