[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#5950: defvaralias after defvar should be warned in runtime
From: |
Clément Pit-Claudel |
Subject: |
bug#5950: defvaralias after defvar should be warned in runtime |
Date: |
Fri, 3 Aug 2018 17:00:33 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-08-03 16:23, Noam Postavsky wrote:
> On 2 August 2018 at 23:33, Clément Pit-Claudel <cpitclaudel@gmail.com> wrote:
>> On 2018-08-02 13:03, Stefan Monnier wrote:
>>>>> The typical failure case goes like this, user does this:
>>>>>
>>>>> (setq the-package-setting 'foo)
>>>>> (require 'the-package)
>>>>>
>>>>> And the-package.el does this:
>>>>>
>>>>> (defvar the-package-real-setting 'bar)
>>>>> ;; Oops! User's setting of `foo' is overwritten here:
>>>>> (defvaralias 'the-package-setting 'the-package-real-setting)
>>>>
>>>> Thanks. Should the warning be disabled when both variables are
>>>> already `defvar'd, then?
>>>
>>> Why? Replace `setq` with `defvar` in the above scenario and you have
>>> the same problem.
>>
>> Do you? I thought the problem was that you were overwriting a user setting…
>> if you change the first setq into a defvar, where's the user setting that
>> you're overwriting?
>
> If the user writes defvar instead of setq, it has the exact same
> effect: they've set the-package-setting to foo, which will be
> overwritten to bar, same as before.
Right, I understand that. But why would a user write 'defvar' to set a user
variable?
I guess I'm just a bit confused by the warning. What's the proper way to
resolve it? Should I create the alias before loading flycheck?
- bug#5950: defvaralias after defvar should be warned in runtime, Noam Postavsky, 2018/08/01
- bug#5950: defvaralias after defvar should be warned in runtime, Clément Pit-Claudel, 2018/08/02
- bug#5950: defvaralias after defvar should be warned in runtime, Noam Postavsky, 2018/08/02
- bug#5950: defvaralias after defvar should be warned in runtime, Clément Pit-Claudel, 2018/08/02
- bug#5950: defvaralias after defvar should be warned in runtime, Stefan Monnier, 2018/08/02
- bug#5950: defvaralias after defvar should be warned in runtime, Clément Pit-Claudel, 2018/08/02
- bug#5950: defvaralias after defvar should be warned in runtime, Noam Postavsky, 2018/08/03
- bug#5950: defvaralias after defvar should be warned in runtime,
Clément Pit-Claudel <=
- bug#5950: defvaralias after defvar should be warned in runtime, Noam Postavsky, 2018/08/03
- bug#5950: defvaralias after defvar should be warned in runtime, Clément Pit-Claudel, 2018/08/03
- bug#5950: defvaralias after defvar should be warned in runtime, Stefan Monnier, 2018/08/03