Re: [PATCH v2] glib/gpoll: Remove if conditional

2017-07-26 Thread Alistair Francis
On Wed, Jul 26, 2017 at 3:35 PM, Bastien Nocera wrote: > Patches are best attached to bugzilla. You can use "git-bz" to file the bug > and attach your path directly there. > > Don't hesitate to reply if you have trouble filing the bug or attaching the > patch. Thanks, after pinging this I did s

Re: [PATCH v2] glib/gpoll: Remove if conditional

2017-07-26 Thread Bastien Nocera
Patches are best attached to bugzilla. You can use "git-bz" to file the bug and attach your path directly there. Don't hesitate to reply if you have trouble filing the bug or attaching the patch. Cheers > On 26 Jul 2017, at 22:16, Alistair Francis wrote: > > On Fri, Jul 7, 2017 at 11:02 AM,

Re: [PATCH v2] glib/gpoll: Remove if conditional

2017-07-26 Thread Alistair Francis
On Fri, Jul 7, 2017 at 11:02 AM, Alistair Francis wrote: > The original ready < nhandles - 1 can be re-written as ready + 1 < nhandles > which is the same confition that we are checking on the first > itteration of the for loop. This means we can remove the if statement > and let the for loop chec