Re: [sage-devel] Opinions on mostly superfluous refactoring

2016-05-30 Thread Erik Bray
On Mon, May 30, 2016 at 6:42 PM, William Stein wrote: > On Mon, May 30, 2016 at 9:22 AM, Erik Bray wrote: >> Hi all, >> >> I recently needed to dive into the sage_setup.autogen.interpreters >> module in order to make some small changes. > > Quick link: > > https://github.com/sagemath/sage/blob

Re: [sage-devel] Opinions on mostly superfluous refactoring

2016-05-30 Thread William Stein
On Mon, May 30, 2016 at 9:22 AM, Erik Bray wrote: > Hi all, > > I recently needed to dive into the sage_setup.autogen.interpreters > module in order to make some small changes. Quick link: https://github.com/sagemath/sage/blob/master/src/sage_setup/autogen/interpreters.py It's the fast_callab

[sage-devel] Opinions on mostly superfluous refactoring

2016-05-30 Thread Erik Bray
Hi all, I recently needed to dive into the sage_setup.autogen.interpreters module in order to make some small changes. The file is over 4000 lines long, which is a bit on the long side for your typical Python file, though not egregious by any means. That said, when trying to understand some rela