[PATCH 1/1] bash-completion: fix variable leaks of "IFS" and "word"

[Michal Prívozník]

On 11/23/21 13:50, Koichi Murase wrote:
> Thank you for reviewing the patch.
> 
> 2021年11月23日(火) 21:37 Michal Prívozník <mprivozn at redhat.com>:
>> [...]
>>
>> The word variable is also used only inside a while loop (not visible in
>> the context). So it can be declared local there instead of here. I'll
>> fix that before pushing.
> 
> OK.
> 
>> However, we do require developers to provide signoff for their patches
>> to signal their compliance with DCO:
>>
>> https://libvirt.org/hacking.html#developer-certificate-of-origin
>>
>> No need to resend the whole patch, it's sufficient if you reply with
>> your SoB line.
> 
> OK. Are just the following lines sufficient?
> --
> [PATCH 1/1] bash-completion: fix variable leaks of "IFS" and "word"
> 
> Signed-off-by: Koichi Murase <myoga.murase at gmail.com>

Perfect.

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

and pushed. Congratulations on your first libvirt contribution!

Michal