On Thu, 20 Nov 2003, Leonard den Ottolander wrote:
> /* calculate the rest if there is any */
> - num_bytes_new = num_bytes_read -
> nb_cells*ATM_CELL_TOTAL_SIZE;
> - num_bytes_read -= nb_cells*ATM_CELL_TOTAL_SIZE;
> + num_bytes_new = num_bytes_read -= nb_cells *
> ATM_CELL_TOTAL_SIZE;
>
It's a style thing but this I really don't like. It is far too easy
to overlook the - in the -= in the changed version. IMnsHO, the
original was better.
Wouldn't
num_bytes_read -= nb_cells * ATM_CELL_TOTAL_SIZE;
num_bytes_new = num_bytes_read;
be better?
Tim.
--
God said, "div D = rho, div B = 0, curl E = - @B/@t, curl H = J + @D/@t,"
and there was light.
http://tjw.hn.org/ http://www.locofungus.btinternet.co.uk/
Liste de diffusion modem ALCATEL SpeedTouch USB
Pour se désinscrire : mailto:[EMAIL PROTECTED]
