danmar/cppcheck — comment on commit 9f42ce91a104d87f555b511d6b9abe18b81e963e
File:  line 0
By lioncash:

I don't mean to be 'that guy', especially when I've only done as much as contribute an update to the Japanese language translation, but I do agree with kimmov.

I may not contribute to the code directly, but I do read the commits themselves to see where I can possibly help out in the future.

Which brings me to the point that some of the commit messages are rather vague or ambiguous to what they actually change.

An example of this is this commit, as well as https://github.com/danmar/cppcheck/commit/b1ff900aaaf6092ef1524a84d6a452e96d8b2338, and https://github.com/danmar/cppcheck/commit/2ba2a4e6ae52e2fb7442c57b03093cc6a64fc461; the only message in the whole commit is "Some Refactorizations".

While I can read the code itself to figure out what you're changing, it doesn't really provide an explanation to, 'why?'. As in, "Why did you change it?".