Hi Joey,

I find some parts of the mr script quite hard to follow, and
consequently to write patches for.  For example, loadconfig() has 221
lines, and is 8 levels of indentation deep at certain points.  Would
you accept "extract subroutine" refactoring patches which do not
change the functionality in any way, but simply break down some of the
longer subroutines into more easily understandable chunks?

Thanks,
Adam
_______________________________________________
vcs-home mailing list
vcs-home@lists.madduck.net
http://lists.madduck.net/listinfo/vcs-home

Reply via email to