Re: [Libusb-win32-devel] =?iso-8859-15?q?=5BPATCH=5D_Add_recipient_su?= =?iso-8859-15?q?pport_for_get/set_descriptor?=

[Stephan M. <ste...@we...>]

The first change was intended, but not the second one!
I'll reverse it.

Thanks for the hint,
Stephan


> On Thu, Jul 20, 2006 at 08:04:41PM +0200, Stephan Meyer wrote:
> >=20
> > I applied your patch, but in driver=5Fapi.h I moved the 'recipient' memb=
er=20
> > to the end of the structure so that the new dll remains downward=20
> > compatible and works with older driver versions.
>=20
> It looks like a couple unintentional changes might have crept into your
> commit of windows.c. Just in case, I've highlighted them here...
>=20
> Thanks!
> --Adam
>=20
> > diff -u -r1.47 -r1.48
> > --- windows.c	18 Jul 2006 20:13:40 -0000	1.47
> > +++ windows.c	20 Jul 2006 17:58:57 -0000	1.48
> > @@ -161,18 +161,18 @@
> >      }
> >   =20
> >    /* now, retrieve the device's current configuration, except from hu=
bs */
> > -  if(dev->device->config && dev->device->config->interface
> > -     && dev->device->config->interface[0].altsetting
> > -     && dev->device->config->interface[0].altsetting[0].bInterfaceCla=
ss=20
> > -     !=3D USB=5FCLASS=5FHUB)
> > -    {
> > -      config =3D usb=5Fget=5Fconfiguration(dev);
> > +/*   if(dev->device->config && dev->device->config->interface */
> > +/*      && dev->device->config->interface[0].altsetting */
> > +/*      && dev->device->config->interface[0].altsetting[0].bInterface=
Class  */
> > +/*      !=3D USB=5FCLASS=5FHUB) */
> > +/*     { */
> > +/*       config =3D usb=5Fget=5Fconfiguration(dev); */
> >       =20
> > -      if(config > 0)
> > -        {
> > -          dev->config =3D config;
> > -        }
> > -    }
> > +/*       if(config > 0) */
> > +/*         { */
> > +/*           dev->config =3D config; */
> > +/*         } */
> > +/*     } */
> > =20
> >    return 0;
> >  }
> > @@ -768,7 +770,7 @@
> >      {
> >        usb=5Ferror("usb=5Fcontrol=5Fmsg: sending control message failed, "
> >                  "win error: %s", usb=5Fwin=5Ferror=5Fto=5Fstring());
> > -      return -usb=5Fwin=5Ferror=5Fto=5Ferrno();
> > +      ret =3D -usb=5Fwin=5Ferror=5Fto=5Ferrno();
> >      }
> > =20
> >    /* out request=3F */
>=20
> ------------------------------------------------------------------------=
-
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share =
your
> opinions on IT & business topics through brief surveys -- and earn cash
> http://www.techsay.com/default.php=3Fpage=3Djoin.php&p=3Dsourceforge&CID=3DDEVDE=
V
> =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F
> Libusb-win32-devel mailing list
> Lib...@li...
> https://lists.sourceforge.net/lists/listinfo/libusb-win32-devel


=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=

Erweitern Sie FreeMail zu einem noch leistungsst=E4rkeren E-Mail-Postfach!	=09
Mehr Infos unter http://freemail.web.de/home/landingpad/=3Fmc=3D021131