On Fri, Oct 11, 2024 at 6:19 AM Dehan Meng <dem...@redhat.com> wrote:
> v2: > Split v1 up to separate commits for each logically independent change > > Dehan Meng (4): > sscanf return values are checked to ensure correct parsing. > Proper initialization of n to 0 for getline to function correctly. > Avoiding freeing line prematurely. It's now only freed at the end of > the function. > For correcting code style: Variable declarations moved to the > beginning of blocks Followed the coding style of using snake_case > for variable names. And merged redundant route and networkroute > variables. > > First of all, don't use so long commit title. You can add more info in the commit message if needed. Commits 3 and 4 have very long titles. Do not set . at the end of the commit title. As in git, we do not see cover-messages, please add `qemu-ga:` suffix for each commit Best Regards, Konstantin Kostiuk. > Signed-off-by: Dehan Meng <dem...@redhat.com> > > qga/commands-linux.c | 128 ++++++++++++++++++++----------------------- > 1 file changed, 59 insertions(+), 69 deletions(-) > > -- > 2.40.1 > >
