mbox series

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

Message ID 160320977352.266767.5622969302437522296.stgit@bahia.lan
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

> 

>