[m-rev.] for review: add a "suggested replacement" arg to obsolete pragmas

Julien Fischer jfischer at opturion.com
Mon Sep 9 13:06:21 AEST 2019


On Mon, 9 Sep 2019, Peter Wang wrote:

> On Sun, 08 Sep 2019 22:53:56 +1000 (AEST), "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
>> For review by anyone.
>> 
>> The main change in {Log,DIFF}.obs adds the optional new argument
>> to obsolete pragmas.
>
> That looks fine.
>
>> Once that diff reaches the rotds and is installed
>> on each of our systems, the second diff can be committed.
>
> I disagree with deprecating *.from_list, *.from_sorted_list list.
>
> Preferences aside, the naming patterns
>
>    <module>.to_<type>
>    <module>.from_<type>
>
> and related
>
>    <module>.cast_to_<type>
>    <module>.cast_from_<type>
>
> are widely used in the standard library compared to <type>_from_<type>
> and <type>_to_<type>.

Seconded, do not deprecate the from_* versions.

Julien.


More information about the reviews mailing list