> On Fri, Jul 26, 2019 at 4:17 PM Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
> >
> > >
> > > Register constructor for emulated CD device. Now creation
> > > of CD devices at startup is possible.
> > >
> > > Signed-off-by: Yuri Benditovich <yuri.benditovich@xxxxxxxxxx>
> > > ---
> > > src/meson.build | 6 ++++++
> > > src/usb-backend.c | 1 +
> > > 2 files changed, 7 insertions(+)
> > >
> > > diff --git a/src/meson.build b/src/meson.build
> > > index 49fec52..944b083 100644
> > > --- a/src/meson.build
> > > +++ b/src/meson.build
> > > @@ -124,6 +124,12 @@ spice_client_glib_sources = [
> > > 'usb-backend.c',
> > > 'usb-emulation.h',
> > > 'usb-backend.h',
> > > + 'usb-device-cd.c',
> > > + 'cd-scsi.c',
> > > + 'cd-scsi.h',
> > > + 'cd-scsi-dev-params.h',
> > > + 'cd-usb-bulk-msd.c',
> > > + 'cd-usb-bulk-msd.h',
> > > 'vmcstream.c',
> > > 'vmcstream.h',
> > > ]
> >
> > I would split this part of the patch and add the files to the
> > build as soon as they are added so to check that at least they
> > compiler, even if they are not used.
>
> No problem
>
> >
> > > diff --git a/src/usb-backend.c b/src/usb-backend.c
> > > index 02369d2..760dc16 100644
> > > --- a/src/usb-backend.c
> > > +++ b/src/usb-backend.c
> > > @@ -432,6 +432,7 @@ SpiceUsbBackend *spice_usb_backend_new(GError
> > > **error)
> > > #endif
> > > #endif
> > > be->own_devices_mask = 3; /* exclude addresses 0 and 1 */
> > > + spice_usb_device_register_cd(be);
> > > }
> > > SPICE_DEBUG("%s <<", __FUNCTION__);
> > > return be;
> >
> > This is adding a circular dependency.
> > usb-backend will depend on usb-device-cd which depends on usb-backend.
>
> I am not aware of deprecation of circular dependency between C files.
> For example, last time I was requested by Victor to create such
> dependency between usb-redir-channel and usb-backend.
>
Not specifically referring to link issue, I'm talking about Design Patterns,
Dependency Inversion Principle and such stuff.
If you like spaghetti code circular dependency is perfectly fine.
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel