Re: [PATCH 08/33] codegen: Remove old ptr32 attribute

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

 



On Wed, Jul 01, 2015 at 06:10:00PM +0100, Frediano Ziglio wrote:
> This attribute is not use in code.

'used'
This was removed in commit 2e8aecc2a56e5b7690d77516e41436b697921b1b I
think. I'd move this commit before the preceding one as you are removing
code that you just added. Or did you use these new checks to make sure
that this is really not used?

Christophe

> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  python_modules/ptypes.py | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/python_modules/ptypes.py b/python_modules/ptypes.py
> index 3a307ed..f94fd24 100644
> --- a/python_modules/ptypes.py
> +++ b/python_modules/ptypes.py
> @@ -105,8 +105,6 @@ valid_attributes={
>      # for a switch this indicate that on network
>      # will occupy always same size (maximum size required for all members)
>      'fixedsize',
> -    # use 32 bit pointer
> -    'ptr32',
>  }
>  
>  attributes_with_arguments={
> @@ -613,8 +611,6 @@ class Member(Containee):
>          self.container = container
>          self.member_type = self.member_type.resolve()
>          self.member_type.register()
> -        if self.has_attr("ptr32") and self.member_type.is_pointer():
> -            self.member_type.set_ptr_size(4)
>          for i in propagated_attributes:
>              if self.has_attr(i):
>                  self.member_type.attributes[i] = self.attributes[i]
> -- 
> 2.1.0
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: pgpJUD5NaWJtx.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]