On Tue, Oct 15, 2013 at 09:42:17AM +0000, Paulo Matos wrote:
> Thanks, regarding the indentation I was convinced we used 2 space
> indentation with maximum line length of 80 characters.
We do, however 8 consecutive spaces in the indentation should be always
replaced by a tab.
> Should I also submit several change logs to gcc-patches like:
>
> For gcc/
> 2013-10-15 Paulo Matos <pma...@broadcom.com>
>
> * ...
>
> For gcc/cp/
> 2013-10-15 Paulo Matos <pma...@broadcom.com>
>
> * ...
If you touch just one ChangeLog file, just make the paths relative
to it's location (i.e. post what you will actually commit to the
ChangeLog file). Otherwise, either you can post multiple ChangeLog
entries as you write above, or just one date name email line
and
gcc/
* ...
gcc/cp/
* ...
or similar, then the individual pieces will be again the final text
you put into the ChangeLog and just add the leading line there.
In either way the reviewer can immediately see you do not mean to
put everything into one ChangeLog or keep the path prefixes that shouldn't
be added during your commit.
Jakub
