> > On Wed, Jul 19, 2017 at 09:42:39AM +0100, Frediano Ziglio wrote: > > This will make easier to change code that handle images. > > > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > > Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > > --- > > Makefile.am | 2 ++ > > vdagent/image.cpp | 88 > > +++++++++++++++++++++++++++++++++++++++++++++++++++++ > > vdagent/image.h | 48 +++++++++++++++++++++++++++++ > > vdagent/vdagent.cpp | 57 +++++----------------------------- > > 4 files changed, 146 insertions(+), 49 deletions(-) > > create mode 100644 vdagent/image.cpp > > create mode 100644 vdagent/image.h > > > > diff --git a/Makefile.am b/Makefile.am > > index b60a718..868199e 100644 > > --- a/Makefile.am > > +++ b/Makefile.am > > @@ -42,6 +42,8 @@ vdagent_SOURCES = \ > > vdagent/vdagent.cpp \ > > vdagent/as_user.cpp \ > > vdagent/as_user.h \ > > + vdagent/image.cpp \ > > + vdagent/image.h \ > > $(NULL) > > > > vdagent_rc.$(OBJEXT): vdagent/vdagent.rc > > diff --git a/vdagent/image.cpp b/vdagent/image.cpp > > new file mode 100644 > > index 0000000..960058d > > --- /dev/null > > +++ b/vdagent/image.cpp > > @@ -0,0 +1,88 @@ > > +/* > > + Copyright (C) 2013-2017 Red Hat, Inc. > > + > > + This program is free software; you can redistribute it and/or > > + modify it under the terms of the GNU General Public License as > > + published by the Free Software Foundation; either version 2 of > > + the License, or (at your option) any later version. > > + > > + This program is distributed in the hope that it will be useful, > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > + GNU General Public License for more details. > > + > > + You should have received a copy of the GNU General Public License > > + along with this program. If not, see <http://www.gnu.org/licenses/>. > > +*/ > > + > > +#include <spice/macros.h> > > + > > +#include "vdcommon.h" > > +#include "image.h" > > + > > +#include "ximage.h" > > + > > +typedef struct ImageType { > > + uint32_t type; > > + DWORD cximage_format; > > +} ImageType; > > + > > +static const ImageType image_types[] = { > > + {VD_AGENT_CLIPBOARD_IMAGE_PNG, CXIMAGE_FORMAT_PNG}, > > + {VD_AGENT_CLIPBOARD_IMAGE_BMP, CXIMAGE_FORMAT_BMP}, > > +}; > > + > > +static DWORD get_cximage_format(uint32_t type) > > +{ > > + for (unsigned int i = 0; i < SPICE_N_ELEMENTS(image_types); i++) { > > + if (image_types[i].type == type) { > > + return image_types[i].cximage_format; > > + } > > + } > > + return 0; > > +} > > + > > +HANDLE get_image_handle(const VDAgentClipboard& clipboard, uint32_t size, > > UINT&) > > For what it's worth, I would have just removed that UINT& arg for now. > > Christophe > The interface was written that way Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel