Bug 29190 – commit-log-editor should move common prefixes to the top of the commit log

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Adam Roben (:aroben)]

Created attachment 39445 [details]
Patch v1

It's silly to have common prefixes in ChangeLogs (e.g., bug title/URL + reviewer) repeated in the commit log. Let's move common prefixes to the top of the log instead.

[David Kilzer (:ddkilzer)]

Comment on attachment 39445 [details]
Patch v1

> +    foreach my $key (keys %{$hashOfStrings}) {
> +        $hashOfStrings->{$key} = substr($hashOfStrings->{$key}, $lastDoubleNewline);

Don't you want to use ($lastDoubleNewline + 2) here as well?

> +    }
> +    return substr($prefix, 0, $lastDoubleNewline + 2);
> +}

Otherwise this looks great!

[Adam Roben (:aroben)]

(In reply to comment #1)
> (From update of attachment 39445 [details])
> > +    foreach my $key (keys %{$hashOfStrings}) {
> > +        $hashOfStrings->{$key} = substr($hashOfStrings->{$key}, $lastDoubleNewline);
> 
> Don't you want to use ($lastDoubleNewline + 2) here as well?

The current behavior is what we want. The leading double newline that's left on each ChangeLog entry ends up right after the label for that ChangeLog, like "WebCore:\n\nUnique part of the entry".

But even though this is the behavior we want, it does seem a bit subtle. It would probably be better to strip the leading double newline from each entry, then explicitly add the double newline after each label.

[Eric Seidel (no email)]

This was landed on Sept 11th as r48307.

/me points aroben at 'bugzilla-tool land-diff' as an svn ci replacement. :)  It will auto-close the bug for you. :)  You can even pass --no-build or --no-test if you don't want it to bother automatically doing one-last build and test cycle for you before landing. :)