Apply suggestions from code review

Exirel · dgw · web-flow · commit 5c974c7a5ff4 · 2021-01-28T23:40:38.000+01:00
Co-authored-by: dgw &lt;dgw@technobabbl.es&gt;
diff --git a/sopel/modules/find.py b/sopel/modules/find.py
@@ -137,11 +137,11 @@ def kick_cleanup(bot, trigger):
              [:,]\s+)?        # Followed by optional colon/comma and whitespace
              s(?P<sep>\|)     # The literal s and a separator | as group 2
              (?P<old>         # Group 3 is the thing to find
-               (?:\\\||[^|])+  # One or more non-slashes or escaped slashes
+               (?:\\\||[^|])+  # One or more non-pipe or escaped pipe
              )
              \|               # The separator again
              (?P<new>         # Group 4 is what to replace with
-               (?:\\\||[^|])* # One or more that isn't the separator
+               (?:\\\||[^|])* # One or more non-pipe or escaped pipe
              )
              (?:|             # Optional separator followed by group 5 (flags)
                 (?P<flags>\S+)

Original file line number	Diff line number	Diff line change
`@@ -137,11 +137,11 @@ def kick_cleanup(bot, trigger):`
`137`	`137`	`[:,]\s+)? # Followed by optional colon/comma and whitespace`
`138`	`138`	`s(?P<sep>\\|) # The literal s and a separator \| as group 2`
`139`	`139`	`(?P<old> # Group 3 is the thing to find`
`140`		`- (?:\\\\|\|[^\|])+ # One or more non-slashes or escaped slashes`
	`140`	`+ (?:\\\\|\|[^\|])+ # One or more non-pipe or escaped pipe`
`141`	`141`	`)`
`142`	`142`	`\\| # The separator again`
`143`	`143`	`(?P<new> # Group 4 is what to replace with`
`144`		`- (?:\\\\|\|[^\|])* # One or more that isn't the separator`
	`144`	`+ (?:\\\\|\|[^\|])* # One or more non-pipe or escaped pipe`
`145`	`145`	`)`
`146`	`146`	`(?:\| # Optional separator followed by group 5 (flags)`
`147`	`147`	`(?P<flags>\S+)`