This is the mail archive of the elfutils-devel@sourceware.org mailing list for the elfutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

ChangeLog entries (Was: [PATCH v3] Use fallthrough attribute)


On Fri, 2018-02-09 at 22:36 -0500, Frank Ch. Eigler wrote:
> On Sat, Feb 10, 2018 at 03:26:43AM +0100, Mark Wielaard wrote:
> 
> > I added ChangeLog entries and pushed it to master.
> 
> What's a ChangeLog entry?  :-)

Yeah, yeah, I know. I am the last one to care.
Really, it helps me review code (even my own - actually writing them
makes me happy - o, right I did that!). If only to know a change is
really intended to be there.

To me the problem with dropping them is that there is no description
anymore what changed. The commit message normally describes why
something changed, not how. You of course have the diff to see what
changed, but not in a summary form. And I really like to have a summary
to cross check that what changed was intended to change.

For people that don't like them, what exactly is it about them that you
don't like? And what would you suggest as replacement?

Thanks,

Mark


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]