[Python-checkins] cpython: Remove_uninitialized_compiler_warning

Amaury Forgeot d'Arc amauryfa at gmail.com
Sat May 12 15:19:27 CEST 2012


Hi,

The previous code looked wrong, but the new "break" will change behaviour.
>From a comment above, the LZMA1 filter needs a "dict_size" as well.
The "fallthrough" was probably intentional.

> diff --git a/Modules/_lzmamodule.c b/Modules/_lzmamodule.c
> --- a/Modules/_lzmamodule.c
> +++ b/Modules/_lzmamodule.c
> @@ -412,7 +412,11 @@
>              ADD_FIELD(options, lc);
>              ADD_FIELD(options, lp);
>              ADD_FIELD(options, pb);
> -        case LZMA_FILTER_LZMA2:
> +            ADD_FIELD(options, dict_size);
> +            break;
> +        }
> +        case LZMA_FILTER_LZMA2: {
> +            lzma_options_lzma *options = f->options;
>              ADD_FIELD(options, dict_size);
>              break;
>          }


-- 
Amaury Forgeot d'Arc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/python-checkins/attachments/20120512/bee80168/attachment-0001.html>


More information about the Python-checkins mailing list