Michael Buesch <mb@xxxxxxxxx> writes:
> On Saturday 21 February 2009 22:52:02 Alina Friedrichsen wrote:
>> Give slow hardware some time to do the TSF sync, to not run into an IBSS merging endless loop in some rarely situations.
>
> It would be great to put this into a source code comment. Without
> this knowledge one immediately gets a WTF-lookalike face when seeing
> this code.
I fully agree.
>> - if (beacon_timestamp > rx_timestamp) {
>> + if (beacon_timestamp > rx_timestamp && rx_timestamp > 0x400000) {
And no magic numbers, please. Create a define for that.
--
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
