[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#57884: [PATCH] Flymake backend using the shellcheck program
From: |
Philip Kaludercic |
Subject: |
bug#57884: [PATCH] Flymake backend using the shellcheck program |
Date: |
Mon, 19 Sep 2022 08:03:35 +0000 |
Augusto Stoffel <arstoffel@gmail.com> writes:
> On Sun, 18 Sep 2022 at 21:18, Philip Kaludercic wrote:
>
>> BTW, this diff describes the changes required if you were to pull out
>> the sentinel definition into a named function:
>> [...]
>> I still don't think it looks that bad, but I don't insist on it.
>
> I still don't understand the motivation for this. Closures are the
> perfect tool for this job, and you seem to be just reinventing them. If
> I wanted to create a helper function (which I don't really find
> necessary from the code organization perspective in this case, although
> it might well be in the proselint case), then I'd pass this as :sentinel
> argument to make-process:
>
> (lambda (proc _event) (helper-function proc source report-fn))
It is just a optical-stylistic manner, nothing technical. Flatter
functions and less indentation, that is all.
- bug#57884: [PATCH] Flymake backend using the shellcheck program, (continued)
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Augusto Stoffel, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Augusto Stoffel, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Augusto Stoffel, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Augusto Stoffel, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/18
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Augusto Stoffel, 2022/09/19
- bug#57884: [PATCH] Flymake backend using the shellcheck program,
Philip Kaludercic <=
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Augusto Stoffel, 2022/09/24
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/24
- bug#57884: [PATCH] Flymake backend using the shellcheck program, Philip Kaludercic, 2022/09/18