PS: in both cases, Cython files are not processed. autopep8 is actually able to process them, but the comparison wouldn't be apples-to-apples.
(that said, autopep8 gives suboptimal results on Cython files, for example it changes "&c_variable" to "& c_variable" and "void* ptr" to "void * ptr") Regards Antoine. Le 02/04/2020 à 15:30, Antoine Pitrou a écrit : > > Hello, > > I've put up two PRs to compare the effect of running black vs. autopep8 > on the Python codebase. > > * black: https://github.com/apache/arrow/pull/6810 > 65 files changed, 7855 insertions(+), 5215 deletions(-) > > * autopep8: https://github.com/apache/arrow/pull/6811 > 20 files changed, 137 insertions(+), 118 deletions(-) > > I've configured black to try and minimize changes (for example, avoid > normalizing string quoting style). Still, the number of changes is > humongous and they add 2600 lines to the codebase (which is a tangible > amount of vertical space). > > Regards > > Antoine. >
