On Fri, 2017-12-08 at 08:56 -0500, Dennis Dalessandro wrote:
> On 12/8/2017 8:33 AM, Hal Rosenstock wrote:
> > > int create_ah(struct ud_resources *ud_res)
> > > @@ -641,7 +649,7 @@ static int register_to_trap(struct sync_resources *sync_res,
> > > pthread_mutex_unlock(res->mad_buffer_mutex);
> > > } while (rc == 2); // while old response.
> > >
> > > - } while (rc == 0 && ++counter < 3);
> > > + } while (rc == 0 && ++counter < 5);
> > >
> > > if (counter==3) {
> >
> > This should be changed from 3 to 5 also. Change both to some define ?
>
> Yes please change these to a #define with a some what descriptive name.
> That would make the code much better to read.
It would be even better to change the loop such that it counts down instead
of up. Then the if-statement after the loop can test whether the counter has
reached zero and the constant that represents the number of iterations will
only occur once.
Bart.��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
