[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