SL Baur <steve(a)vmailer.xemacs.org> writes:
I propose the following addition to the rules of approving patches
on xemacs-review:
If a ChangeLog-less patch appears that you wish to approve because
it is in your area of expertise of XEmacs, you include appropriate
ChangeLog entries with the approval.
OK with me.
However, there is one more thing where I disagree with Kyle: I don't
think ChangeLogs should be exact descriptions of changes. We have CVS
for that. In my opinion, something like "would return the wrong
value", or "fixed counter incrementing" are perfectly fine ChangeLog
entries.
ChangeLog should be regarded as a "conceptual undo list" of changes,
not as a plain-English replacement for `diff -u'.
--
Hrvoje Niksic <hniksic(a)srce.hr> | Student at FER Zagreb, Croatia
--------------------------------+--------------------------------
Unspeakable horrors from outer space paralyze the living and
resurrect the dead!