Re: [Libusbx-devel] [PATCH] Add topology calls

2012-05-12 Thread Ludovic Rousseau
2012/5/11 Pete Batard : > +/** \ingroup dev > + * Get the list of all port numbers from root for the specified device > + * \param dev a device > + * \param path the array that should contain the port numbers > + * \param path_len the maximum length of the array I propose to document that a "corr

Re: [Libusbx-devel] [PATCH] Add topology calls

2012-05-12 Thread Peter Stuge
Pete Batard wrote: > I'm puzzled as to why you're trying to blow such a minor API item > out of proportion Not at all out of proportion. The parameter is unneccessary and redundant since the depth limit of 7 is in fact not arbitrary. Every library should be convenient to use, and redundancy is not

Re: [Libusbx-devel] [PATCH] Add topology calls

2012-05-12 Thread Pete Batard
On 2012.05.12 12:30, Ludovic Rousseau wrote: > 2012/5/11 Pete Batard: > >> +/** \ingroup dev >> + * Get the list of all port numbers from root for the specified device >> + * \param dev a device >> + * \param path the array that should contain the port numbers >> + * \param path_len the maximum len

Re: [Libusbx-devel] [PATCH] Add topology calls

2012-05-12 Thread Pete Batard
On 2012.05.12 12:55, Peter Stuge wrote: > I was hoping that you would also comment on my API suggestion which > completely avoids this problem. Maybe someone else will. I don't see how returning numbers represented as a string instead of actual numbers is that helpful. This translates the actual