qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH RFC 2/4] range: Eliminate direct Range member ac


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH RFC 2/4] range: Eliminate direct Range member access
Date: Mon, 20 Jun 2016 09:26:10 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

"Michael S. Tsirkin" <address@hidden> writes:

> On Wed, Jun 15, 2016 at 10:41:48PM +0200, Markus Armbruster wrote:
>> Users of struct Range mess liberally with its members, which makes
>> refactoring hard.  Create a set of methods, and convert all users to
>> call them instead of accessing members.  The methods have carefully
>> worded contracts, and use assertions to check them.
>> 
>> To help with tracking down the places that access members of struct
>> Range directly, hide the implementation of struct Range outside of
>> range.c by trickery.  The trickery will be dropped in the next commit.
>> 
>> Signed-off-by: Markus Armbruster <address@hidden>
>
> Reviewed-by: Michael S. Tsirkin <address@hidden>

Thanks!

> I guess you want me to merge this because of the changes in pc and pci?

Yes, please (whole series, once respun without the RFC).



reply via email to

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