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:
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:
Oops, sorry about that. Applied in r21229.
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/2082