[Spice-devel] [spice-gtk][PATCH v4] Added INFO messages about a file transfer
Christophe Fergeau
cfergeau at redhat.com
Mon Oct 6 07:25:46 PDT 2014
On Mon, Oct 06, 2014 at 08:35:05AM -0400, Marc-André Lureau wrote:
>
>
> ----- Original Message -----
> > When a file transfer starts / finishes an information message is printed (in
> > INFO log level).
> > Also INFO messages about the transfer progress are periodically printed.
>
> Why info level and not debug?
>
> I am not fond adding more code to the "Xfer" code limited by design, I would rather work on shared folder and dnd support.
>
> > ---
> > changes since v4:
> > - logging is enabled when transferring more files simultaneously
> >
> > gtk/channel-main.c | 42 +++++++++++++++++++++++++++++++++++++++++-
> > 1 file changed, 41 insertions(+), 1 deletion(-)
> >
> > diff --git a/gtk/channel-main.c b/gtk/channel-main.c
> > index 1ad090f..1e0f940 100644
> > --- a/gtk/channel-main.c
> > +++ b/gtk/channel-main.c
> > @@ -70,6 +70,8 @@ typedef struct SpiceFileXferTask {
> > char buffer[FILE_XFER_CHUNK_SIZE];
> > uint64_t read_bytes;
> > uint64_t file_size;
> > + GDateTime *start_time;
> > + GDateTime *last_update;
>
> GDateTime requires glib 2.26 (currently only 2.22 required), you need to make this code conditional.
Or we can raise the requirement in configure.ac to 2.26.
Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20141006/dfa1cf46/attachment.sig>
More information about the Spice-devel
mailing list