On Wed, Dec 08, 2021 at 02:27:34PM -0700, Shuah Khan wrote: > > } > > } > > + > > + snd_config_delete(config); > > } > > /* > This open comment at the end of the patch looks odd. Does this compile? Yes, it's the start of the comment describing the next function, more complete context is: } snd_config_delete(config); } /* * Check that we can read the default value and it is valid. Write * tests use the read value to restore the default. */ void test_ctl_get_value(struct ctl_data *ctl) {
Attachment:
signature.asc
Description: PGP signature