On 12/25/2009 02:25 PM, Marcelo Tosatti wrote:

One small comment on the naming:

kvm->memslots->memslots[i]
kvm->aliases->aliases[i]

Doesn't look good...
Hi Sheng,

Yes its somewhat confusing but i could not come up with
anything better. Suggestions?

Could all it 'mem', and put both ->memslots and ->aliases as members. This might simplify things a bit.

However, I don't recommend doing so now, it's just churn. We can always to it later.

--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to