format skipping (fmt:on/off, autopep8:on/off) only works once

[alinsavix]

For some reason, it looks like the functionality added in #557, to enable blocks of code to be selectively skipped during an autopep8 run, only works if those directives appear a single time in a file. If they appear two or more times, the skip functionality is completely disabled for all instances.

For example, this works as you'd expect:

# fmt: off
test_struct = (
    ("Diffuse",                   "MapObjDiffuse_T1",           "MapObjDiffuse"),
)

test_struct_two = (
    ("TwoLayerDiffuseMod2xNA",    "MapObjDiffuse_Comp",         "MapObjTwoLayerDiffuseMod2xNA"),
)
# fmt: on

$ autopep8 --diff fmt_off_test.py
[no output]

But, if you instead use this input file:

# fmt: off
test_struct = (
    ("Diffuse",                   "MapObjDiffuse_T1",           "MapObjDiffuse"),
)
# fmt: on

# fmt: off
test_struct_two = (
    ("TwoLayerDiffuseMod2xNA",    "MapObjDiffuse_Comp",         "MapObjTwoLayerDiffuseMod2xNA"),
)
# fmt: on

... then both blocks get reformatted:

$ autopep8 --diff fmt_off_test.py
--- original/fmt_off_test.py
+++ fixed/fmt_off_test.py
@@ -1,11 +1,11 @@
 # fmt: off
 test_struct = (
-    ("Diffuse",                          "MapObjDiffuse_T1",           "MapObjD
iffuse"),
+    ("Diffuse", "MapObjDiffuse_T1", "MapObjDiffuse"),
 )
 # fmt: on

 # fmt: off
 test_struct_two = (
-    ("TwoLayerDiffuseMod2xNA",    "MapObjDiffuse_Comp",                "MapObjT
woLayerDiffuseMod2xNA"),
+    ("TwoLayerDiffuseMod2xNA", "MapObjDiffuse_Comp", "MapObjTwoLayerDiffuseMod2
xNA"),
 )
 # fmt: on

This is, needless to say, somewhat problematic when it comes to actually using this feature. (and it would definitely be a nice to use feature, for code that needs large data tables where you want a human to have any hope whatsoever of being able to look at a table be able to understand anything about it)

Environment

• Python version: 3.9.6
• autopep8 version: autopep8 1.5.7 (pycodestyle: 2.7.0)
• Platform: Windows 10 20H2 (build 19042.1202)

[hhatto]

Thanks for reporting.

duplicate #603.
fixed in version 1.6.