907c5d387f
Originally, black_bindings found each contiguous "//|" block and sent it to black independently. This was slower than it needed to be. Instead, swap the comment prefix: when running black, take off "//|" prefixes and put "##|" prefixes on all un-prefixed lines. Then, after black is run, do the opposite operation This more than doubles the overall speed of "pre-commit run --all", from 3m20s to 55s CPU time on my local machine (32.5s to under 10s "elapsed" time) It also causes a small amount of churn in the bindings, because black now sees enough context to know whether one 'def' follows another or ends the 'def's in a 'class'. In the latter case, it adds an extra newline, which becomes a "//|" line. I'm less sure why a trailing comma was omitted before down in rp2pio/StateMachine.c but let's roll with it. |
||
|---|---|---|
| .. | ||
| DigitalInOut.c | ||
| DigitalInOut.h | ||
| Direction.c | ||
| Direction.h | ||
| DriveMode.c | ||
| DriveMode.h | ||
| Pull.c | ||
| Pull.h | ||
| __init__.c | ||
| __init__.h | ||