Thread: [pygccxml-development] Changing mdecl_wrapper_t.decls

Brought to you by: mbaas, roman_yakovenko

pygccxml-development

[pygccxml-development] Changing mdecl_wrapper_t.decls

From: Allen B. <al...@vr...> - 2006-08-23 20:42:42

I would like to propose that we change mdecl_wrapper_t.decls to be named 
something else.  Perhaps "decl_list" or something.

The reason is that the current name conflicts with the method of the 
same name in the decl interface from scopedef_t.

So for example:

classes = ns.decls("class")
classes.decls("method")

This will fail because it finds the attribute decls which is not a callable.

Comments?

-Allen

Re: [pygccxml-development] Changing mdecl_wrapper_t.decls

From: Roman Y. <rom...@gm...> - 2006-08-23 21:00:53

On 8/23/06, Allen Bierbaum <al...@vr...> wrote:
> I would like to propose that we change mdecl_wrapper_t.decls to be named
> something else.  Perhaps "decl_list" or something.

declarations ?

> The reason is that the current name conflicts with the method of the
> same name in the decl interface from scopedef_t.
>
> So for example:
>
> classes = ns.decls("class")
> classes.decls("method")
>
> This will fail because it finds the attribute decls which is not a callable.
>
> Comments?
>
> -Allen
>
>
> -------------------------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job easier
> Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> _______________________________________________
> pygccxml-development mailing list
> pyg...@li...
> https://lists.sourceforge.net/lists/listinfo/pygccxml-development
>


-- 
Roman Yakovenko
C++ Python language binding
http://www.language-binding.net/

Re: [pygccxml-development] Changing mdecl_wrapper_t.decls

From: Allen B. <al...@vr...> - 2006-08-23 21:16:43

Roman Yakovenko wrote:

> On 8/23/06, Allen Bierbaum <al...@vr...> wrote:
>
>> I would like to propose that we change mdecl_wrapper_t.decls to be named
>> something else.  Perhaps "decl_list" or something.
>
>
> declarations ?

Sounds fine with me.

-Allen

>
>> The reason is that the current name conflicts with the method of the
>> same name in the decl interface from scopedef_t.
>>
>> So for example:
>>
>> classes = ns.decls("class")
>> classes.decls("method")
>>
>> This will fail because it finds the attribute decls which is not a 
>> callable.
>>
>> Comments?
>>
>> -Allen
>>
>>
>> ------------------------------------------------------------------------- 
>>
>> Using Tomcat but need to do more? Need to support web services, 
>> security?
>> Get stuff done quickly with pre-integrated technology to make your 
>> job easier
>> Download IBM WebSphere Application Server v.1.0.1 based on Apache 
>> Geronimo
>> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
>> _______________________________________________
>> pygccxml-development mailing list
>> pyg...@li...
>> https://lists.sourceforge.net/lists/listinfo/pygccxml-development
>>
>
>

Re: [pygccxml-development] Changing mdecl_wrapper_t.decls

From: Roman Y. <rom...@gm...> - 2006-08-23 21:30:05

On 8/24/06, Allen Bierbaum <al...@vr...> wrote:
> Roman Yakovenko wrote:
>
> > On 8/23/06, Allen Bierbaum <al...@vr...> wrote:
> >
> >> I would like to propose that we change mdecl_wrapper_t.decls to be named
> >> something else.  Perhaps "decl_list" or something.
> >
> >
> > declarations ?
>
> Sounds fine with me.

Please commit the change. I will run unit tests tomorrow

-- 
Roman Yakovenko
C++ Python language binding
http://www.language-binding.net/

Re: [pygccxml-development] Changing mdecl_wrapper_t.decls

From: Allen B. <al...@vr...> - 2006-08-24 02:13:35

Roman Yakovenko wrote:

> On 8/24/06, Allen Bierbaum <al...@vr...> wrote:
>
>> Roman Yakovenko wrote:
>>
>> > On 8/23/06, Allen Bierbaum <al...@vr...> wrote:
>> >
>> >> I would like to propose that we change mdecl_wrapper_t.decls to be 
>> named
>> >> something else.  Perhaps "decl_list" or something.
>> >
>> >
>> > declarations ?
>>
>> Sounds fine with me.
>
>
> Please commit the change. I will run unit tests tomorrow
>
I haven't implemented the change.  I don't really know everywhere this 
will impact so I didn't try to make the change.

-Allen