[libvirt] [PATCH] C#Bindings patch, rename classes

Daniel P. Berrange berrange at redhat.com
Wed Oct 20 12:21:24 UTC 2010


On Wed, Oct 20, 2010 at 02:07:45PM +0200, arnaud.champion at devatom.fr wrote:
> ?Okay, I'll make the change... Currently at naming level, my problem is 
> around Error class. Do you have a suggestion ?

What's wrong with following the example I gave, eg

  namespace Libvirt [

    public class Error {
    ...

> >
> >IMHO
> >
> >namespace LibvirtBindings
> >{
> >    public class virDomain
> >    {
> >    ....
> >
> >Is somewhat redundant and better named
> >
> >namespace Libvirt
> >{
> >    public class Domain
> >    {
> >    ....
> >
> >
> >The 'vir' prefix on C library APIs/structs is just a hack because C
> >does not have any concept of namespaces. It shouldn't be copied into
> >languages which do have proper namespace support
> >

Daniel
-- 
|: Red Hat, Engineering, London    -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :|
|: http://autobuild.org        -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|




More information about the libvir-list mailing list