Re: PATCH: Reindent assign_file_positions_for_non_load_sections

[David Daney <ddaney dot cavm at gmail dot com>]

On 06/11/2012 09:50 AM, H.J. Lu wrote:
> On Mon, Jun 11, 2012 at 09:44:51AM -0700, H.J. Lu wrote:
> > Hi,
> >
> > I checked in this patch to reindent _bfd_elf_map_sections_to_segments.
[...]
> diff --git a/patches/ChangeLog b/patches/ChangeLog
> index 2e62b52..3660f41 100644
> --- a/patches/ChangeLog
> +++ b/patches/ChangeLog
> @@ -1,3 +1,9 @@
> +2012-06-10  H.J. Lu<hjl.tools@gmail.com>
> +
> +	* binutils-pr14215.patch: New.
> +	* binutils-pr14207.patch: Likewise.
> +	* README: Apply them.
> +

Is there a reason not to just say you re-indented here?

I think adding a level of indirection in the ChangeLog it not a good thing.

David Daney

>   2012-06-09  H.J. Lu<hjl.tools@gmail.com>
>
>   	* binutils-sb.patch: Removed.
> diff --git a/patches/README b/patches/README
> index 667e473..b83a6f5 100644
> --- a/patches/README
> +++ b/patches/README
> @@ -23,6 +23,8 @@ patches="
>   	bfd-64k-2.patch
>   	binutils-lto-mixed-18.patch
>   	binutils-pr12639-2.patch
> +	binutils-pr14215.patch
> +	binutils-pr14207.patch
>   "
>
>   if [ -n "$clean" ]