Re: [PATCH][gdb] Update syscalls/{amd64,i386}-linux.xml

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

On 5/9/22 17:40, Andreas Schwab wrote:
> On Mai 09 2022, Tom de Vries via Gdb-patches wrote:
> 
>> +    tmp=$(mktemp)
>> +
>> +    echo '#include <sys/syscall.h>' \
>> +	| gcc -E - -dD "$@" \
>> +	| grep -E '#define __NR_' \
>> +		> "$tmp"
>> +
>> +    echo '<syscalls_info>'
>> +
>> +    while read -r line; do
>> +	name=$(echo "$line" | awk '{print $2}' | sed 's/^__NR_//')
>> +	nr=$(echo "$line" | awk '{print $3}')
>> +	echo "  <syscall name=\"$name\" number=\"$nr\"/>"
>> +    done < "$tmp"
>> +
>> +    echo '</syscalls_info>'
>> +
>> +    rm -f "$tmp"
> 
> There is no need for a temporary file, you can just pipe directly into
> the loop.
> 

True, but I like the temporary file there, it makes it easy to 
investigate intermediate results.

Thanks,
- Tom