On Sat, Aug 24, 2013 at 4:47 PM, Jeremie Le Hen <j...@freebsd.org> wrote:
> On Sat, Aug 24, 2013 at 10:41:56AM -0400, Eitan Adler wrote:
>> On Thu, Aug 22, 2013 at 4:49 PM, Jeremie Le Hen <j...@freebsd.org> wrote:
>> > Well, whatever, if you have any concerns, objections or comments, please
>> > speak now :).
>>
>> This LGTM but please include a comment above the warning with a date /
>> release number when this compatibility can be removed.
>
> If the old variable names are deprecated in releng/10, they can be
> removed in releng/11, can't they?

Yes, and this should be indicated in a comment.  When I see
"deprecated" or "old hack" or similar terms in code it takes some
archaeology to figure out when it was added and when it could be
removed.   It would be nice to help the future reader a bit.

-- 
Eitan Adler
_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"

Reply via email to