Re: [pushed] [gdb/testsuite] Handle attributes.h for remote host

[Tom de Vries <tdevries@suse.de>]

On 3/18/23 11:12, Andreas Schwab wrote:
> On Mär 18 2023, Tom de Vries via Gdb-patches wrote:
> 
>> +set flags {}
>> +lappend flags debug
> 
> aka set flags debug
> 

Hi,

yes, though I prefer the form I'm using, for instance because it allows 
me to do:
...
  set flags {}
-lappend flags debug
+#lappend flags debug
...
while this:
...
-set flags debug
+#set flags debug
...
breaks the test-case because flags is no longer set.

Besides, for readability I prefer that each flag is added individually, 
and in the same way, so not:
...
set flags {opt-a opt-b opt-c}
...
and not:
...
set flags opt-a
lappend flags opt-b
lappend flags opt-c
...
but:
...
set flags {}
lappend flags opt-a
lappend flags opt-b
lappend flags opt-c
...

Thanks,
- Tom