python bindings and uint16 on big-endian platforms

John (J5) Palmieri johnp at redhat.com
Fri Nov 4 22:49:34 PST 2005


No sorry it is my fault missed the u.  A bit tired after the daylight
savings change.  Patch is good as is.

On Sat, 2005-11-05 at 02:34 +0200, Johan Hedberg wrote:
> On Fri, Nov 04, 2005, John (J5) Palmieri wrote:
> > Looks good to me though you have a duplicated elif:
> > 
> > +            elif type == TYPE_INT16:
> > +                num = iter.get_int16()
> > +                arg = 'int16:%d\n' % (num)
> > +            elif type == TYPE_UINT16:
> > +                num = iter.get_uint16()
> > +                arg = 'uint16:%u\n' % (num)
> > 
> > Can you repost the patch with one of those taken out?  Thanks.
> 
> Hmm, maybe I'm missing something, but how is that a duplicate? One is
> for INT16 and the other for UINT16.
> 
> Johan
-- 
John (J5) Palmieri <johnp at redhat.com>



More information about the dbus mailing list