supervision icon indicating copy to clipboard operation
supervision copied to clipboard

fix: process video bug on unused else

Open AnonymDevOSS opened this issue 1 month ago • 1 comments

The loop never uses break, so the else clause always executes. However, since it iterates over a generator, the generator is already exhausted after the first pass... so the else block would iterate over nothing.

So, i believe the 'else' is redundant.

Type of change

Please delete options that are not relevant.

  • [X] Bug fix (non-breaking change which fixes an issue)

How has this change been tested, please provide a testcase or example of how you tested the change?

>>> def g():
...   for x in range(3):
...     yield x
... 
>>> gen = g()
>>> for x in gen:
...   print(x)
... else:
...   for y in gen:
...     print("y", y)
... 
0
1
2

Any specific deployment considerations

For example, documentation changes, usability, usage/costs, secrets, etc.

Docs

  • [ ] Docs updated? What were the changes:

AnonymDevOSS avatar Oct 26 '25 23:10 AnonymDevOSS

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 26 '25 23:10 CLAassistant