On Thu, Nov 22, 2001 at 11:09:03AM +0100, Martin Diehl wrote:
>
> Hi Jean,
>
> thanks for the improvement - yeah, soon after sending this quick hack
> I've realized there was a thinko in the bits-handling :-(
> Because this wouldn't hurt people testing with their latency problems
> I decided to wait and see, whether it would help at all.
It doesn't matter, but I don't want to get bad code in the
stack, too many time I just cut'n'paste...
> No complains - applies (2.4.14, with few lines offset for most hunks) and
> compiles fine, nothing broken for me and irlap/qos seems to do the right
> thing with the min_tx_turn_time. Only tested with mtt=1000 peer device
> - I have no such mtt=0 guy.
Good.
> I would suggest to simply stay with the semantincs of the existing FAST_RR
> config option but just make it enabled by default. Which would still mean
> you have to say "No" in order to *disable* stuff, IMHO?
> "no need to say Yes to enable" != "Yes to disable"
I don't know if you can say "enabled by default". I'll look
into it.
> I believe the Configure.help entry should change from "if unsure, say N"
> to "if unsure, say Y" as well.
Of course.
> Martin
Jean
_______________________________________________
Linux-IrDA mailing list - [EMAIL PROTECTED]
http://www.pasta.cs.UiT.No/mailman/listinfo/linux-irda
