[ernst_klim]

Indeed. I have a few colleages and they constantly try to push these long convoluted functions which look like

    is_done = False
    while not is_done:
      if pattern1:
        ...
        if pattern2:
          ...
          if matched == "SUCCESS":
             is_done = True
             break
        if pattern3:
          ...

It's usually correct but extremely hard to follow and reminds of the good old asm code with convoluted goto's.

And the colleages tend to do reviews with the help of the agents so they don't even care to read this mess.