[v2,06/65] aarch64: use is_whitespace()

Message ID 7c2a9f3c-2927-4bd6-8e48-e6d5da5a6726@suse.com
State New
Headers
Series gas: whitespace handling |

Commit Message

Jan Beulich Jan. 27, 2025, 3:50 p.m. UTC
  Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).
  

Comments

Richard Earnshaw (lists) Jan. 27, 2025, 4:31 p.m. UTC | #1
On 27/01/2025 15:50, Jan Beulich wrote:
> Wherever blanks are permissible in input, tabs ought to be permissible,
> too. This is particularly relevant when -f is passed to gas (alongside
> appropriate input).
> 
> --- a/gas/config/tc-aarch64.c
> +++ b/gas/config/tc-aarch64.c
> @@ -634,7 +634,7 @@ const char FLT_CHARS[] = "rRsSfFdDxXeEpP
>  
>  /* Separator character handling.  */
>  
> -#define skip_whitespace(str)  do { if (*(str) == ' ') ++(str); } while (0)
> +#define skip_whitespace(str)  do { if (is_whitespace (*(str))) ++(str); } while (0)
>  
>  static inline bool
>  skip_past_char (char **str, char c)
> 

OK.

R.
  

Patch

--- a/gas/config/tc-aarch64.c
+++ b/gas/config/tc-aarch64.c
@@ -634,7 +634,7 @@  const char FLT_CHARS[] = "rRsSfFdDxXeEpP
 
 /* Separator character handling.  */
 
-#define skip_whitespace(str)  do { if (*(str) == ' ') ++(str); } while (0)
+#define skip_whitespace(str)  do { if (is_whitespace (*(str))) ++(str); } while (0)
 
 static inline bool
 skip_past_char (char **str, char c)