[latex3-commits] [l3svn] 02/04: "deprecated" here causes an error ... [ci skip]

[Bruno Le Floch]

On 05/13/2017 08:20 AM, Joseph Wright wrote:
> On 13/05/2017 13:18, Bruno Le Floch wrote:
>>
>>>     "deprecated" here causes an error ... [ci skip]
>>
>>> -% \begin{macro}[deprecated = 2017-04-06]{\box_resize:Nnn, \box_resize:cnn}
>>> +% \begin{macro}{\box_resize:Nnn, \box_resize:cnn}
>>
>> This is on purpose: the "deprecated" key expects the date at which the
>> function will be removed.  I chose to produce an error when typesetting
>> the doc so that we wouldn't forget to remove these functions come Dec 31.
>>
>> When should \box_resize:Nnn be deleted?  2018-12-31?
>>
>> Bruno
>>
> 
> Presumably.
> 
> I wanted to ship and the logic wasn't clear, so I've 'forced' the issue.
> To me, putting in the date _when something is deprecated_ seems logical:
> the date for removal is a policy decision that should appear at most
> once in any code (in l3doc).
> 
> Joseph
> 

While I agree "deprecated = <date-where-it-will-be-removed>" is
illogical, I don't understand in what sense anything is duplicated by
doing that.  What markup would you like?  I guess one option would be to do

   \begin{macro}[deprecated = 2017-04-05]{...}

and make l3doc complain if the current year is > (year+1).

By the way, I'm not complaining about what you did removing the date,
just trying to figure out how to set things up better.

Bruno