Message163147
| Author |
nadeem.vawda |
| Recipients |
Sam.Rushing, eric.araujo, jcea, nadeem.vawda |
| Date |
2012年06月19日.08:57:14 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1340096235.88.0.343437247271.issue14684@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
> To restate my position: the need is for an immutable string of bytes, [...]
I disagree that we should require the dictionary to be immutable - if the
caller wishes to use a mutable buffer here, it is their responsibility to
ensure that it is not modified until the compressor is finished with it
("consenting adults" and all that). The documentation can inform users of
this requirement.
> I believe the argument for aesthetics does not apply, as the constant
> dictionary constructor argument is a morally different kind of
> parameter, comparable to (say) the compression level.
Even so, the surrounding code sets a precedent for how it accepts binary
data buffers, and deviating from this existing convention should not be
taken lightly.
Nitpicking about the API aside, thanks for the patch :-) |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2012年06月19日 08:57:16 | nadeem.vawda | set | recipients:
+ nadeem.vawda, jcea, eric.araujo, Sam.Rushing |
| 2012年06月19日 08:57:15 | nadeem.vawda | set | messageid: <1340096235.88.0.343437247271.issue14684@psf.upfronthosting.co.za> |
| 2012年06月19日 08:57:15 | nadeem.vawda | link | issue14684 messages |
| 2012年06月19日 08:57:14 | nadeem.vawda | create |
|