2011/5/10 Eric Blake <eblake@xxxxxxxxxx>:
> On 05/10/2011 02:26 AM, Lai Jiangshan wrote:
>> ---
>> Âdaemon/remote_generator.pl  |  Â2 +-
>> Âsrc/remote/remote_driver.c  |  Â2 +-
>> Âsrc/remote/remote_protocol.x | Â Â8 +++++++-
>> Âsrc/remote_protocol-structs Â| Â Â4 ++++
>> Â4 files changed, 13 insertions(+), 3 deletions(-)
>>
>> diff --git a/daemon/remote_generator.pl b/daemon/remote_generator.pl
>> index 062ccc1..74fa769 100755
>> --- a/daemon/remote_generator.pl
>> +++ b/daemon/remote_generator.pl
>> @@ -35,7 +35,7 @@ sub name_to_ProcName {
>> Â Â Â@elems = map { $_ =~ s/Nwfilter/NWFilter/; $_ =~ s/Xml/XML/;
>> Â Â Â Â Â Â Â Â Â Â $_ =~ s/Uri/URI/; $_ =~ s/Uuid/UUID/; $_ =~ s/Id/ID/;
>> Â Â Â Â Â Â Â Â Â Â $_ =~ s/Mac/MAC/; $_ =~ s/Cpu/CPU/; $_ =~ s/Os/OS/;
>> - Â Â Â Â Â Â Â Â Â $_ } @elems;
>> + Â Â Â Â Â Â Â Â Â $_ =~ s/Nmi/NMI/; $_ } @elems;
>
> Do we really want yet another difference in spelling? ÂBut cleaning that
> up affects more than just your new API, so I'll leave it for a later patch.
>
Actually the problem is different here. name_to_ProcName takes the
procedure name (that is all uppercase) and splits it on underscores
into words, then turns all words into all lowercase and finally make
the first letter uppercase again. This is why NMI and up as Nmi and
needs to be fixed afterwards. If you have a better solution for this
I'd like to see it.
Matthias
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list