This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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]

Re: [PATCH 1/2] gas: don't get confused by .asci{i,z} after .struct


On Mon, Dec 07, 2015 at 07:41:36AM -0700, Jan Beulich wrote:
> --- 2015-12-07/gas/read.c
> +++ 2015-12-07/gas/read.c
> @@ -5329,8 +5329,9 @@ stringer (int bits_appendzero)
>    if (now_seg == absolute_section)
>      {
>        as_bad (_("strings must be placed into a section"));
> -      c = 0;
> -      ignore_rest_of_line ();
> +      if (c)
> +	ignore_rest_of_line ();
> +      return;
>      }

I think it would be better to move this block above the code setting
'c' (and of course not test 'c').  OK with that change.

-- 
Alan Modra
Australia Development Lab, IBM


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