On Thu, Mar 03, 2016 at 10:12:25AM -0500, Frediano Ziglio wrote:
> >
> > On Tue, Mar 01, 2016 at 12:42:20PM -0500, Frediano Ziglio wrote:
> > > > with QXLState an internal structure as well (defined in reds.h), and
> > > > RedDispatcher/RedQXL is one of the few members of this QXLState
> > > > structure. Should they be merged?
> > > >
> > >
> > > I think you should check patch 4/5 :)
> >
> > Ah indeed, missed that one. Don't you mean QXLState rather than
> > RedsState in the shortlog though?
> > With this commit, we end up with
> > server/red-qxl.h:typedef struct QXLState RedQXL;
> > server/spice-qxl.h:typedef struct QXLState QXLState;
> > and
> > struct QXLState {} in red-qxl.c which holds the data for what we call
> > RedQXL throughout the file. I think it would be better to just use the
> > QXLState name everywhere and avoid introducing yet another name (we
> > already have QXLState, QXLInstance, QXLInterface, not having RedQXL in
> > addition to all of this sounds better to me.
> >
> > Christophe
> >
>
> So would you like to have functions like qxl_state_set_mm_time and similar
> and files like qxl-state.[ch] ?
> I was lookin at similar implementation (the main one is record/play).
> Here you have the struct SpiceRecordState which is defined "private" in
> sound.c and functions like snd_record_/snd_playback_.
> Would then make sense if the functions where just named qxl_ (like
> qxl_set_mm_time) ?
> So perhaps file names qxl.[ch] ?
Oh, actually I was mainly talking about the type name (QXLState vs
RedQXL). I think I won't mind too much if the methods operating on it are
void red_qxl_do_foo(QXLState *);
Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel