mbox series

[v2,0/5] tests/9pfs: Code refactoring

Message ID 160320977352.266767.5622969302437522296.stgit@bahia.lan (mailing list archive)
Headers show
Series tests/9pfs: Code refactoring | expand

Message

Greg Kurz Oct. 20, 2020, 4:09 p.m. UTC
Some code refactoring to have a clear distinction between top level
test functions and helper functions.

v2: - use "do_" prefix instead of "do_fs_"

---

Greg Kurz (5):
      tests/9pfs: Factor out do_version() helper
      tests/9pfs: Turn fs_readdir_split() into a helper
      tests/9pfs: Set alloc in fs_create_dir()
      tests/9pfs: Factor out do_attach() helper
      tests/9pfs: Turn fs_mkdir() into a helper


 tests/qtest/virtio-9p-test.c |   62 +++++++++++++++++++++++-------------------
 1 file changed, 34 insertions(+), 28 deletions(-)

--
Greg

Comments

Christian Schoenebeck Oct. 20, 2020, 5:23 p.m. UTC | #1
On Dienstag, 20. Oktober 2020 18:09:07 CEST Greg Kurz wrote:
> Some code refactoring to have a clear distinction between top level
> test functions and helper functions.
> 
> v2: - use "do_" prefix instead of "do_fs_"
> 
> ---
> 
> Greg Kurz (5):
>       tests/9pfs: Factor out do_version() helper
>       tests/9pfs: Turn fs_readdir_split() into a helper
>       tests/9pfs: Set alloc in fs_create_dir()
>       tests/9pfs: Factor out do_attach() helper
>       tests/9pfs: Turn fs_mkdir() into a helper
> 
> 
>  tests/qtest/virtio-9p-test.c |   62
> +++++++++++++++++++++++------------------- 1 file changed, 34
> insertions(+), 28 deletions(-)
> 
> --
> Greg

Queued on 9p.next:
https://github.com/cschoenebeck/qemu/commits/9p.next

I just tweaked the order of the patches to prevent intermediate compiler 
errors.

Thanks!

Best regards,
Christian Schoenebeck
Greg Kurz Oct. 20, 2020, 6 p.m. UTC | #2
On Tue, 20 Oct 2020 19:23:19 +0200
Christian Schoenebeck <qemu_oss@crudebyte.com> wrote:

> On Dienstag, 20. Oktober 2020 18:09:07 CEST Greg Kurz wrote:
> > Some code refactoring to have a clear distinction between top level
> > test functions and helper functions.
> > 
> > v2: - use "do_" prefix instead of "do_fs_"
> > 
> > ---
> > 
> > Greg Kurz (5):
> >       tests/9pfs: Factor out do_version() helper
> >       tests/9pfs: Turn fs_readdir_split() into a helper
> >       tests/9pfs: Set alloc in fs_create_dir()
> >       tests/9pfs: Factor out do_attach() helper
> >       tests/9pfs: Turn fs_mkdir() into a helper
> > 
> > 
> >  tests/qtest/virtio-9p-test.c |   62
> > +++++++++++++++++++++++------------------- 1 file changed, 34
> > insertions(+), 28 deletions(-)
> > 
> > --
> > Greg
> 
> Queued on 9p.next:
> https://github.com/cschoenebeck/qemu/commits/9p.next
> 
> I just tweaked the order of the patches to prevent intermediate compiler 
> errors.
> 

Oops, sorry... thanks ! :)

> Thanks!
> 
> Best regards,
> Christian Schoenebeck
> 
>