On Fri, Oct 23, 2015 at 3:54 PM, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote: > >> >> On Fri, Oct 23, 2015 at 3:25 PM, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote: >> > >> >> >> >> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxx> >> >> >> >> Just move some declarations around >> >> --- >> >> server/Makefile.am | 2 + >> >> server/cursor_channel.h | 62 +++++++++++ >> >> server/display-channel.h | 211 +++++++++++++++++++++++++++++++++++++ >> >> server/red_worker.c | 264 >> >> +---------------------------------------------- >> >> server/red_worker.h | 20 ++++ >> >> 5 files changed, 297 insertions(+), 262 deletions(-) >> >> create mode 100644 server/cursor_channel.h >> >> create mode 100644 server/display-channel.h >> >> >> >> diff --git a/server/Makefile.am b/server/Makefile.am >> >> index cd4686a..a701170 100644 >> >> --- a/server/Makefile.am >> >> +++ b/server/Makefile.am >> >> @@ -106,6 +106,8 @@ libspice_server_la_SOURCES = \ >> >> red_time.h \ >> >> red_worker.c \ >> >> red_worker.h \ >> >> + display-channel.h \ >> >> + cursor_channel.h \ >> > >> > I think this file should cursor-channel.h as renamed some patches later >> >> Good catch, but here I have to disagree. >> I would not start having two "nomenclatures" for files at this point. >> The most part of the files are using "_" instead of "-". I would go >> for "_" for all of them, including display-channel. :-) >> > > This is part of the refactory. The big rename patch was removed just temporarily > to make it easier for cherry picking and similar but new files should have dash > and old files will be renamed later. Okay, makes sense. > > Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel