2 Mar
2014
2 Mar
'14
9:09 a.m.
On Sat, Feb 22, 2014 at 12:36:02AM +0100, Jacob Erlbeck wrote:
Dear Jacob,
let me publicly say I think you did great work. Going from the problem
description and proposal of just dropping frames you have read both the
code and the specification and came up with a way better solution.
I was about to merge your branch but noticed one thing.
+/* Table 3.1 TS 04.08: Values of parameter S */
+static const uint8_t tx_integer[] = {
+ 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 14, 16, 20, 25, 32, 50,
+};
...
+ for (i = 0; i < 15; i++) {
+ if (tx_integer[i] == T) {
I didn't have breakfast yet but I think ARRAY_SIZE(tx_integer) is 16 but
the last element is never looked at. Can you confirm/reject?