[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#35062: [PATCH 0/4] Trivial code cleanups
From: |
Eli Zaretskii |
Subject: |
bug#35062: [PATCH 0/4] Trivial code cleanups |
Date: |
Mon, 01 Apr 2019 07:37:23 +0300 |
> From: Konstantin Kharlamov <Hi-Angel@yandex.ru>
> Date: Mon, 1 Apr 2019 01:37:38 +0300
>
> These are mostly fixes of some of LGTM warnings
> https://lgtm.com/projects/g/emacs-mirror/emacs/alerts/?mode=tree
>
> Except the second patch, where I initially wanted to fix one warning,
> and as part of it I had to constify a variable to see that it is indeed
> immutable. And then I figured I could search through the code and find
> more similar places, where variables weren't marked as const. I like
> this cleanup because it is simple and trivially testable (i.e. if it
> compiles, then it's fine). FTR: there's still lots of opportunities for
> constification, I just stopped at some point.
Thanks.
I think the general policy is not to fix those except when making
other changes in the same function, but I will let others comment.
- bug#35062: [PATCH 0/4] Trivial code cleanups,
Eli Zaretskii <=