Marton Balint <cus <at> passwd.hu> writes: > - res = shift_data(s); > + if ((res = shift_data(s)) < 0) It's not my code but "if (res < 0)" is imo easier to read, less error-prone and makes the patch smaller... Carl Eugen