[PATCH] tools: variables clean-up in libvirt-guests script
Prathamesh Chavan
pc44800 at gmail.com
Sun Mar 29 08:57:29 UTC 2020
> >For my project, 'Introducing job control to the storage driver',
> >right now I'm going through the `/src/qemu/THREADS.txt` as well
> >as previous work done by Tucker DiNapoli during GSoC'14, as well
> >as the proposal of Taowei Luo for GSoC'15. Their email threads
> >are surely helping me get a better picture of the project.
> >Apart from this, please let me know if there is anything else,
> >which could help me with this project. Thansk!
> >
>
> Discussing the project would be a good start. You're doing well that you're
> reading what was done before, but also make sure to check out what is happening
> lately.
>
I started looking up the changes which happened after that using git-log,
the release notes and also by looking up into the mailing list.
> >[1]: https://travis-ci.org/github/pratham-pc/libvirt/builds/668209157
...
> >- name=$(guest_name "$uri" "$guest")
> >+ local name=$(guest_name "$uri" "$guest")
>
> It would be nice to have the info about the variable being local in one place,
> at the top of the function. Most of us are used to our old C ways here, so
> keeping this in a similar fashion could be nice.
>
> I understand that this is not an interesting task and it is also one of the
> easiest ones, but the idea for that was created when this actually solved a real
> bug. This will probably be a one-off since not much is happening in the script,
> but we could have a calmer sleep if the variables are marked properly.
>
Thanks, Martin for the suggestions. I'll be sending an updated version
of this soon.
Thanks,
Prathamesh Chavan
More information about the libvir-list
mailing list