INTERNAL ERROR: Black produced different code on the second pass of the formatter.

[ccie18643]

error: cannot format /root/PyTCP/fpp/icmp6.py: INTERNAL ERROR: Black produced different code on the second pass of the formatter. Please report a bug on https://github.com/psf/black/issues. This diff might be helpful: /tmp/blk_74vkg8wd.log

diff

(python3.8) root@ubuntu-1:~/PyTCP# cat /tmp/blk_74vkg8wd.log
Mode(target_versions=set(), line_length=160, string_normalization=True, experimental_string_processing=False, is_pyi=False)
--- source
+++ first pass
@@ -275,22 +275,34 @@
     def _read_nd_options(self, optr):
         """ Read ND options """
 
         nd_options = []
         while optr < len(self._frame):
-            nd_options.append({ps.icmp6.ND_OPT_SLLA: NdOptSLLA, ps.icmp6.ND_OPT_TLLA: NdOptTLLA, ps.icmp6.ND_OPT_PI: NdOptPI}.get(self._frame[optr], NdOptUnk)(self._frame, optr))
+            nd_options.append(
+                {ps.icmp6.ND_OPT_SLLA: NdOptSLLA, ps.icmp6.ND_OPT_TLLA: NdOptTLLA, ps.icmp6.ND_OPT_PI: NdOptPI}.get(self._frame[optr], NdOptUnk)(
+                    self._frame, optr
+                )
+            )
             optr += self._frame[optr + 1] << 3
 
         return nd_options
 
     @property
     def nd_options(self):
         """ Read ND options  """
 
         if self.__nd_options is self.__not_cached:
             assert self.type in {ps.icmp6.ROUTER_SOLICITATION, ps.icmp6.ROUTER_ADVERTISEMENT, ps.icmp6.NEIGHBOR_SOLICITATION, ps.icmp6.NEIGHBOR_ADVERTISEMENT}
-            optr = self._hptr + {ps.icmp6.ROUTER_SOLICITATION: 12, ps.icmp6.ROUTER_ADVERTISEMENT: 16, ps.icmp6.NEIGHBOR_SOLICITATION: 24, ps.icmp6.NEIGHBOR_ADVERTISEMENT: 24}[self.type]
+            optr = (
+                self._hptr
+                + {
+                    ps.icmp6.ROUTER_SOLICITATION: 12,
+                    ps.icmp6.ROUTER_ADVERTISEMENT: 16,
+                    ps.icmp6.NEIGHBOR_SOLICITATION: 24,
+                    ps.icmp6.NEIGHBOR_ADVERTISEMENT: 24,
+                }[self.type]
+            )
             self.__nd_options = self._read_nd_options(optr)
         return self.__nd_options
 
     @property
     def nd_opt_slla(self):
@@ -588,6 +600,5 @@
 
     def __eq__(self, other):
         """ Compare two records """
 
         return self.raw_record == other.raw_record
-
--- first pass
+++ second pass
@@ -290,19 +290,16 @@
     def nd_options(self):
         """ Read ND options  """
 
         if self.__nd_options is self.__not_cached:
             assert self.type in {ps.icmp6.ROUTER_SOLICITATION, ps.icmp6.ROUTER_ADVERTISEMENT, ps.icmp6.NEIGHBOR_SOLICITATION, ps.icmp6.NEIGHBOR_ADVERTISEMENT}
-            optr = (
-                self._hptr
-                + {
-                    ps.icmp6.ROUTER_SOLICITATION: 12,
-                    ps.icmp6.ROUTER_ADVERTISEMENT: 16,
-                    ps.icmp6.NEIGHBOR_SOLICITATION: 24,
-                    ps.icmp6.NEIGHBOR_ADVERTISEMENT: 24,
-                }[self.type]
-            )
+            optr = self._hptr + {
+                ps.icmp6.ROUTER_SOLICITATION: 12,
+                ps.icmp6.ROUTER_ADVERTISEMENT: 16,
+                ps.icmp6.NEIGHBOR_SOLICITATION: 24,
+                ps.icmp6.NEIGHBOR_ADVERTISEMENT: 24,
+            }[self.type]
             self.__nd_options = self._read_nd_options(optr)
         return self.__nd_options
 
     @property
     def nd_opt_slla(self):

[ichard26]

Hello!

All reproduction cases in this issue format without error on master. The fixing commit was 8672af3 from PR GH-2126. I'll be marking this issue as a duplicate of GH-1629 since that's what GH-2126 aimed to fix and it's highly likely this issue falls under GH-1629.

Since we use the issue tracker as a reflection of what's on master, I'll be closing this issue. If you have any issues, especially with the new (but stable) output, please open a new issue. Oh and the fix should be available in a published release soon, see GH-2125 for more info.

Thank you for reporting!