qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] topology.h: cleanup


From: Cao jin
Subject: Re: [Qemu-devel] [PATCH] topology.h: cleanup
Date: Wed, 24 Feb 2016 10:10:56 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0



On 02/24/2016 05:16 AM, Eduardo Habkost wrote:
On Tue, Feb 23, 2016 at 02:51:11PM +0800, Cao jin wrote:
remove useless parameter of several functions

Signed-off-by: Cao jin <address@hidden>

I believe this makes the API much easier to be misused. Currently
you can safely use (smp_cores, smp_threads) every time. With this
patch, you need to always check the function definition to find
out what's the correct argument.


Yes, you do have a point here, and very interesting to me. A caller need to know/check how to use a api if it want to use it, seems a nature thing to me.

Thanks for your explanation.

--
Yours Sincerely,

Cao jin





reply via email to

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