[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#27016: possible bug in `defsetf'
From: |
Stefan Monnier |
Subject: |
bug#27016: possible bug in `defsetf' |
Date: |
Sat, 15 Jul 2017 21:03:26 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
>> + (push (cons name (eval fun t))
> What does this do? Should it be `lexical-binding' instead of `t'?
Sorry, part of an unrelated experiment.
>> + (push (cons fun `(,prop ,val
>> + . ,(assq fun byte-compile-plist-environment)))
> That should be alist-get instead of assq.
Good catch.
> + (let* ((fun (eval fun t))
> + (prop (eval prop t))
These should likely be just (eval fun) without the `t` either.
Tho it doesn't really matter in any case: all those `eval`s just strip
off the quote in front of a sexp and should do the same regardless of
lexical-binding.
Stefan
- bug#27016: possible bug in `defsetf', (continued)
- bug#27016: possible bug in `defsetf', npostavs, 2017/07/10
- bug#27016: possible bug in `defsetf', Stefan Monnier, 2017/07/11
- bug#27016: possible bug in `defsetf', npostavs, 2017/07/11
- bug#27016: possible bug in `defsetf', Stefan Monnier, 2017/07/11
- bug#27016: possible bug in `defsetf', npostavs, 2017/07/13
- bug#27016: possible bug in `defsetf', Stefan Monnier, 2017/07/13
- bug#27016: possible bug in `defsetf', npostavs, 2017/07/13
- bug#27016: possible bug in `defsetf', Stefan Monnier, 2017/07/13
- bug#27016: possible bug in `defsetf', Stefan Monnier, 2017/07/14
- bug#27016: possible bug in `defsetf', npostavs, 2017/07/15
- bug#27016: possible bug in `defsetf',
Stefan Monnier <=