Xpra: Ticket #2082: fix for r21210 which breaks nvenc

A "for" loop was eliminated in r21210 but a "break" was left behind. This extra break breaks the wrong for loop.

rumba:~/src # svn diff --patch-compatible xpra/codecs/video_helper.py
Index: xpra/codecs/video_helper.py
===================================================================
--- xpra/codecs/video_helper.py (revision 21228)
+++ xpra/codecs/video_helper.py (working copy)
@@ -43,7 +43,6 @@
         module_name = CODEC_TO_MODULE[codec_name]
         if has_codec_module(module_name):
             names.append(codec_name)
-            break
     return names
 #all the codecs we know about:


Fri, 14 Dec 2018 22:41:08 GMT - Nathan Hallquist: priority, component changed

Forgot to mark the patch as code, and don't have permission to edit my original post.

Index: xpra/codecs/video_helper.py
===================================================================
--- xpra/codecs/video_helper.py (revision 21228)
+++ xpra/codecs/video_helper.py (working copy)
@@ -43,7 +43,6 @@
         module_name = CODEC_TO_MODULE[codec_name]
         if has_codec_module(module_name):
             names.append(codec_name)
-            break
     return names
 #all the codecs we know about:

Fri, 14 Dec 2018 22:44:35 GMT - Antoine Martin: status, description changed; resolution set

Oops, sorry about that. Applied in r21229.


Sat, 23 Jan 2021 05:41:44 GMT - migration script:

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/2082