[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make
From: |
Noam Postavsky |
Subject: |
bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME> |
Date: |
Sat, 17 Mar 2018 10:54:46 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.90 (gnu/linux) |
> From: Eric Skoglund <eric@pagefault.se>
>> More importantly, could you explain a bit how your change works/why it's
>> correct?
>
> I'm guessing you mean in the commit message and not here in the bug
> report right?
Either, both :)
(If you send a new patch with the explanation in the commit message,
it's automatically in the bug report as well anyway)
>> Not sure [about] this `make-symbol' call, should it rather be
>> `intern'? (Maybe we should update signal-process take a string as
>> well a symbol.)
>
> Saw that Eli also agreed, for future reference when do we want to use
> `make-symbol` vs `intern`?
Usually, `make-symbol' is only for macros, where you want a symbol that
is not `eq' to any other. Sometimes it's handy for making a unique
object at run-time. In this case, since we only care about the symbol
name, it doesn't really matter, it's just a bit surprising to see it
because it's usually a mistake in a non-macro context.
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Noam Postavsky, 2018/03/16
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Noam Postavsky, 2018/03/16
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Eli Zaretskii, 2018/03/17
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Eric Skoglund, 2018/03/17
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Eli Zaretskii, 2018/03/17
- Message not available
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Noam Postavsky, 2018/03/22
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Eric Skoglund, 2018/03/23
- bug#29156: 25.3; eshell/kill does not understand -<signal>, [PATCH] Make eshell/kill handle -<signal> and -<SIGNALNAME>, Noam Postavsky, 2018/03/25