qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 3/4] util: Remove unused functions


From: Thomas Huth
Subject: Re: [Qemu-devel] [PATCH 3/4] util: Remove unused functions
Date: Tue, 17 Feb 2015 10:52:37 +0100

 Hi,

On Tue, 17 Feb 2015 09:55:33 +0100
Markus Armbruster <address@hidden> wrote:

> Thomas Huth <address@hidden> writes:
> 
> > Delete the unused functions qemu_opt_get_number_del(),
> > qemu_signalfd_available(), qemu_send_full() and qemu_recv_full().
> >
> > Signed-off-by: Thomas Huth <address@hidden>
> 
> qemu_opt_get_number_del() was added in commit 782730b along with
> qemu_opt_get_del(), qemu_opt_get_bool_del() and qemu_opt_get_size_del().
> It hasn't been used so far, but removing it makes the interface
> irregular.  I'd rather not.

Ok, then let's keep them.

> qemu_signalfd_available() is unused since commit 6d32717.  It's also the
> only user of CONFIG_SIGNALFD.  If we want to drop it, I recommend
> dropping CONFIG_SIGNALFD as well.

As far as I can see, CONFIG_SIGNALFD is still used in qemu_signalfd(),
so I think we've got to keep that.

 Thomas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]