> I don't want to beat the issue to death (too late?), but I think you'll > agree that as implemented this is a hack. A sometimes useful hack that > hasn't caused any serious problems so far, but still a hack. I'd like us to > get rid of hacks by getting rid of the code or turning them into non-hack > features. In this case, turning it into a feature would add a lot of code > and complexity for not a lot of extra functionality (which I think you've > said as well), so I'd prefer it got axed. I don't think it's as bad of a hack as you're making it out to be. I'm also not sure that I agree that it can't be a useful feature. But even if it were a hack and always would be, removing it necessary? If it's useful for me and not causing anyone else any problems at all, why cause me (and anyone else that may use it) pain? There are lots of M5 features that aren't well documented, but that are useful for people. If they're benign, why not leave them in? Certainly if they make code appreciably more confusing or complex, they could be removed, but in this case, you're talking about 4 lines of code (that even has two lines of comments before it).
> Then again I didn't even know the > code was there until Ali brought it up and I actively looked for it, so it's > more a philosophical question whether to keep it or not (vs. expanding it > which is a different sort of issue). If it helps you and it doesn't hurt > anybody then I wouldn't mind leaving it alone. I agree. :) I still don't think it would be that huge of an issue to make it easier to work with. I could even factor out the interactive prompt stuff (nearly trivial), so people can just put interact() into options.py and then get a >>> prompt there with the ability to do dir() and find out what all is available at the interactive prompt. If you have IronPython installed, you could then do something like "foo?" and find out what foo is all about and get its docstring. I also just looked and I do indeed set M5_CONFIG=$HOME/m5/config and I also have something in options.py to change the default output directory. Nate _______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
