[Python-Dev] SRE recursion removed
Guido van Rossum
guido at python.org
Sat Oct 18 15:37:12 EDT 2003
> > What is the purpose of the USE_RECURSION #define? It looks to me like
> > you have added a lot of dead code; I recommend to remove all this code.
>
> If you enable USE_RECURSION it will become recursive again, so it's
> nice to see if some problem is related to the non-recursive algorithm
> or not, and makes it easy to understand to change made.
That's okay.
> The "dead" code you're talking about is probably the unused macros,
> right? I've used them in some ideas, and gave up later. OTOH, they may
> be used in further extensions. If you don't mind, I'd rather leave them
> there, than thinking about it again if I need it. But if they're really
> a problem, well, I'll remove. Just let me know.
That is *not* okay. Dead code is a distraction for future
maintainers.
--Guido van Rossum (home page: http://www.python.org/~guido/)
More information about the Python-Dev
mailing list